Строки - ООП (C++)

[Пепел Феникса]

почему это копирование должно быть const ?

конструктор копии не имеет права изменять исходный обьект.

[trashem]

Оценил дохлого страуса =)

[sidestep]

#include <stdlib.h>
#include <string.h>
#include <conio.h>
#include <iostream>

using namespace std;

class String {
private: 
	char* data;
public:

	String () {
		//конструктор по умолчанию (создает пустую строку)
		data = new char[1];
		data = "";
	}

	String (char* my_str /*= 0*/) { 
		// конструктор с параметром
		data = new char[strlen(my_str) + 1];
		strcpy(data, my_str);			
	}
	String (const String& my_str) {
		// конструктор копирования (в качестве параметра передается ссылка на существующий объект)
		data = new char[strlen(my_str.data) + 1];
		strcpy(data, my_str.data);
	}

	~String () {
		// деструктор
		cout << "destructor"<< endl; // просто для наглядности, что видно было, что запускается наш, а не дофолтный деструктор
		delete[] data;				 // в конце блока использования объектов класса String (в нашем случае, в конце программы и в методах, где создаются временные объекты класса) будет
	}								 // выводится строчка destructor для каждого "уничтоженного" объекта.

	void display () const {			 // функция просто выводит содержимое объекта на экран
		cout << data << endl;
	}

	// просто объявление части функций (методов) в классе
	
	String& operator=  (const String&);			// перегрузка оператора присваивания
	String& operator=  (const char*);			// перегрузка оператора присваивания (-//-)

	
	operator char* () const {			 // оператор привидения объектов класса стринг к переменным типа char*
		return data;
	}
};
//------------------------------------------------------------------------------------------------------------------
//---------------------------------------- Описание методов класса -------------------------------------------------




String& String::operator= (const String& my_str) {
	// короче в c++ достаточно специфичный случай для равенства объектов obj = obj, поэтому его надо "обойти"
	if(this != &my_str && my_str.data) { // this - e
		data = new char[strlen(my_str.data) + 1];
		strcpy(data, my_str.data);
	}
	return *this;
}

String& String::operator= (const char* my_str) {
	if (my_str) {
		data = new char[strlen(my_str) + 1];
		strcpy(data, my_str);
	}
	return *this;
}



int main() {
	String str1; // пустая строка, используется конструктор по умолчанию
	String str2 = "Privet!"; // используется конструктор с одним параметром
	String str3("Dobrui Den!"); // также используется конструктор с одним параметром 
	String str4 = str2; // конструктор копирования
	String str5(str3);  // конструктор копирования

	//----------------------------
	cout << "proverka rabotu konstruktorov:" << endl;
	str1.display();
	str2.display();
	str3.display();
	str4.display();
	str5.display();
	//----------------------------

	//----------------------------
	cout << "\nproverka operatora privideniya k tipu \"char\": " << endl;
	cout << "data obiekta str2 = ";
	str2.display();
	char* gavnoCHAROVSKOE = str2;
	cout << "peremenaya gavnoCHAROVSKOE posle prispoeniya znacheniya str2 = " << gavnoCHAROVSKOE << endl;
	//----------------------------

	//----------------------------
	cout << "\nproverka rabotu operatora prisvaivania:" << endl;
	str2.display();
	str5.display();
	cout << "teper prisvaivaem str2 = str5..." << endl;
	str2 = str5;
	str2.display();
	str5.display();
	
	

	cout << "\n\n\n\n\n";

	getch();
	return 0;
}

Все равно косячит именно деструктор

[pproger]

2sidestep

data = new char[1];
data = "";

чтож ты творишь то

[sidestep]

создает пустую строку по умолчанию
все работает
что не так ?

[Demon123123]

Действительно нормальный конструктор по умолчанию....
Кстати в классе заместо метода display надо было бы перегрузить операцию <<,ну можно было бы еще и операцию >>

[pproger]

2sidestep

создает пустую строку по умолчанию

2Demon123123

Действительно нормальный конструктор по умолчанию....

язык подучите

[waleri]

Сообщение от sidestep

создает пустую строку по умолчанию
все работает
что не так ?

Если бы работало, вы бы не жаловались на конструктор/деструктор. Я же вам писал: "А вы выделяете память для data а потом *меняете* указатель"
Копировать строки надо через strcpy а не присваиванием.

[sidestep]

pproger

напиши нормальный

[pproger]

2sidestep
по-нормальному все переписывать надо. для твоего случая можно ограничиться

data = new char[1];
*data = '\0';