C++ Определенное пользователем преобразование - неявное преобразование

Question

У меня вопрос о пользовательском преобразовании.

class String { 
    char* m_data; 

public: 
    String(): m_data(NULL) {} 
    String(const char* cstr): m_data(new char[strlen(cstr)+1]) { 
     strcpy(m_data, cstr); 
    } 
    ~String() { 
     delete[] m_data; 
    } 
    String& operator=(const char* cstr) { 
     delete[] m_data; 
     m_data = new char[strlen(cstr)+1]; 
     strcpy(m_data, cstr); 
     return *this; 
    } 
    operator const char*() const { 
     return m_data; 
    } 
}; 

В то время как это работает:

int main(int argc, char** argv) { 
    String a; 
    String b; 
    a = "aaa"; 
    b = (const char *)a; 
    return 0; 
} 

Это не:

int main(int argc, char** argv) { 
    String a; 
    String b; 
    a = "aaa"; 
    b = a; 
    return 0; 
} 

я получаю сообщение об ошибке выполнения double free or corruption. Valgrind говорит о некорректном удалении.

Почему я должен явно указывать его? Я думал, что так будет работать с explicit operator const char*(). Я делаю что-то неправильно?

Answer 1

Вы забыли определить copy assignment operator:

String& operator=(const String& other) { 
    if(this != &other) { 
     char* new_data = new char[strlen(other.m_data)+1]; 
     strcpy(new_data, other.m_data); 
     delete[] m_data; 
     m_data = new_data; 
    } 
    return *this; 
} 

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

String& operator=(const String& other) { 
    m_data = other.m_data; 
    return *this; 
} 

Так у вас есть два указателя на то же m_data в a и b и на выходе из main, delete[] будет называться дважды.