Создание функции присваивания перемещения, получение освобождаемого указателя не было выделено

Этот:

delete[] str;

удаляет str. Но потом:

str[i] = move.str[i];

str удален. Так что это неопределенное поведение.

Во всяком случае, это не то, как делать ход. Весь смысл переезда состоит в том, чтобы избежать копирования строки. Если предположить, что str - это char*, тогда правильная реализация будет следующей (общее имя аргумента rhs, что означает «правая сторона»):

MyString& MyString::operator=(MyString&& rhs) noexcept
{
    std::swap(len, rhs.len);
    std::swap(str, rhs.str);
    return *this;
}

Опять же: это правильная реализация, только если str - просто указатель. Обратите внимание, что реализация ничего не удаляет. Удаление произойдет в деструкторе rhs. Спецификатор noexcept не требуется, но поскольку эта реализация никогда не может генерировать исключение, его отметка noexcept позволяет компилятору выполнить еще несколько оптимизаций.

Дополнение к Никосу С. ответ:

Обмен - не единственное решение, но оно довольно элегантное: вы сохраняете память целевой строки для повторного использования в исходной строке. Пока все в порядке, вы можете захотеть начать заново с пустой строкой после перемещения. Опять же, вы не должны удалять память, она отлично подходит для повторного использования. Итак, вы просто установите длину равной 0.

Однако тогда вам нужно будет отдельно запомнить, сколько символов еще уместилось в памяти. Но это все равно полезно. Подумайте, хотите ли вы перераспределять память строки каждый раз, когда вы добавляете один символ?

Скорее всего, нет. Таким образом, вы должны добавить дополнительную память (например, удвоить емкость, если у вас закончится память). Все вместе:

class MyString
{
    size_t length;
    size_t capacity;
    char* data;
public:
    MyString& operator=(MyString&& other)
    {
        if(&other != this)
        {
            std::swap(data, other.data);         // as in Nikos' answer
            std::swap(capacity, other.capacity);
            length = other.length;
            other.length = 0;                    // restart with empty string
                                                 // still you have quite a bit of
                                                 // memory already reserved
        }
        return *this;
    }
};

Имейте в виду, что это необязательно, и на самом деле вы можете заставить людей платить за то, что им может не понадобиться - если они не будут повторно использовать перемещенный объект ...