Помгите найти ошибку (MV C++)

[JOFRIF]

Помогите пожалуйста найти ошибку в коде.
Скорее всего дело в деструкторе или с удалением объектов...но я не уверен...
Внизу кода изображение ошибки.
код:

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

class Candybar
{
private:
	char *manufacturer_name;
	double weight;
	int calorie_content;
public:
	Candybar();
	Candybar(char *m_name,double wt,int cc);
	void print();
	~Candybar();
};
Candybar::Candybar(){};

Candybar::Candybar(char *m_name,double wt,int cc)
{
	manufacturer_name = new char[strlen(m_name)+1];
	strcpy(manufacturer_name,m_name);

	weight = wt;
	calorie_content = cc;
}

void Candybar::print()
{
	cout<<"The manufacturer name is: "<< manufacturer_name<<endl;
	cout<<"The weight is: "<<weight<<endl;
	cout<<"The calorie content is: "<<calorie_content<<endl;
}

Candybar::~Candybar()
{
	cout<<"Deliting manufacturer name "<<manufacturer_name<<endl;
	delete []manufacturer_name;
	cout<<"Manufacturer name has been deleted successfully               "<<manufacturer_name<<endl;
}

void main()
{
	Candybar snack[3];

	snack[0] = Candybar("Bounty",5.5,350);
	snack[0].print();
	
	snack[1] = Candybar("Snickers",4.5,450);
	snack[1].print();

	snack[2] = Candybar("Mars",3.5,550);
	snack[2].print();	
}

[pproger]

во-первых:

Candybar::~Candybar()
{
	cout<<"Deliting manufacturer name "<<manufacturer_name<<endl;
	delete []manufacturer_name;
	cout<<"Manufacturer name has been deleted successfully               "<<manufacturer_name<<endl;
}

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

во-вторых:

Candybar snack[3];
snack[0] = Candybar("Bounty",5.5,350);

это называется конструктор копирования. твоя ошибка именно в этом. Ща объясню, что у тя тут происходит. ты имеешь 2 объекта класса Candybar (snack[0] и Candybar в правой части). т.е вызывается 2 конструктора, для левой части пустой, для правой части - с выделением памяти и копированием строки. НО! Когда срабатывает оператор =, по уполчанию он просто присваивает каждое значение из правой части значению в левой части! А у тебя там находится указатель. И получается так, что 2 указателя указывают на одну и ту же область памяти. Когда выполняется первый деструктор (какой - не важно), все норм. Но когда выполняется второй (а указатель на строку указывает на область памяти, которая удалена первым деструктором) - ошибка.
что делать? книги читать)
либо используй std::string, либо перегрузи оператор =, и точно так же выделяй память под свой указатель (но тут надо проверить, не выделил ли ты уже ее. если да - удалить, если нет - выделить) и копируй строку и 2 твоих значения.

[JOFRIF]

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

Но я же удаляю указатель delete []manufacturer_name;.
В книге написано что то вроде:"При первой возможности удаляйте/высвобождайте динамическую память чтобы иметь возможность использовать её заново ".Вот я и хотел ипользовать её заново.

Я не совсем понял что это за "конструктор копирования".
Объясните пожалуйста своими словами. Насколько я понял "конструктор копирования" это замысловатое название для обычного конструктора который инициализирует элементы. Поправте если я не прав.

Ща объясню, что у тя тут происходит. ты имеешь 2 объекта класса Candybar (snack[0] и Candybar в правой части).

3 объекта )

т.е вызывается 2 конструктора, для левой части пустой, для правой части - с выделением памяти и копированием строки.

Что это значит? Для какой левой части он пустой?
Возможно вы имели ввиду верхний и нижний(1-й и второй) конструкторы т.е( 1)Пустой конструктор 2)с параметрами )

И получается так, что 2 указателя указывают на одну и ту же область памяти.

хмм...Тогда, насколько я понял удаляется память на которую указывает указатель а не сам указатель.

что делать? книги читать)

Разумеется)

зы я не понимаю чем стринг может помочь...

[pproger]

блин, я жопой чувствовал, что надо тебе семплы кидать было...

Но я же удаляю указатель delete []manufacturer_name;.

ты вообще посмотрел на сноску, которую я тебе дал?

cout<<"Deliting manufacturer name "<<manufacturer_name<<endl;
delete []manufacturer_name;
cout<<"Manufacturer name has been deleted successfully "<<manufacturer_name<<endl; - в этом месте указатель manufacturer уже удален! строчкой выше

В книге написано что то вроде:"При первой возможности удаляйте/высвобождайте динамическую память чтобы иметь возможность использовать её заново ".Вот я и хотел ипользовать её заново.[

чтобы потом опять сделать new и использовать ее, а не удалить и использовать недействительный указатель.

3 объекта )

нет, 2!

snack[0] = Candybar("Bounty",5.5,350);

тут используется !!!2!!! объекта класса Candybar, в левый и правый операнды.

на остальное мне сейчас влом отвечать. если желаешь, отпишись, утром с работы накидаю примеры

[JOFRIF]

ты вообще посмотрел на сноску, которую я тебе дал?
Цитата:cout<<"Deliting manufacturer name "<<manufacturer_name<<endl;
delete []manufacturer_name;
cout<<"Manufacturer name has been deleted successfully "<<manufacturer_name<<endl; - в этом месте указатель manufacturer уже удален! строчкой выше

Я думаю что не в этом ошибка, потому что я уже так делал для проверки(освободилась память или нет).
Если в строке

cout<<"Manufacturer name has been deleted successfully "<<manufacturer_name<<endl;

печаталось
Manufacturer name has been deleted successfully [][][][][][][]
то это означало что память освобожденна, а иначе нет.

чтобы потом опять сделать new и использовать ее, а не удалить и использовать недействительный указатель.

Как опять сделать new?! если этот new делается внутри коструктора??

Цитата:3 объекта )

нет, 2!
Цитата:snack[0] = Candybar("Bounty",5.5,350);

тут используется !!!2!!! объекта класса Candybar, в левый и правый операнды.

Я так понял что мы о разных объектах говорим. Я имел ввиду что три объекта это массив объектов snack[0]
snack[1]
snack[2]
А ты вообще говоришь про
snack[0] = Candybar("Bounty",5.5,350);
Только я не въехал с чего ты взял что Candybar("Bounty",5.5,350); это объект??? Это вообще конструктор а объектом является snack[0].

на остальное мне сейчас влом отвечать. если желаешь, отпишись, утром с работы накидаю примеры

Ок. Накидай.

[Stilet]

Вместо того чтоб развозить демагогию просто нужно создавать обьекты динамически:

void main()
{
	Candybar* snack[3];

	snack[0] = new Candybar("Bounty",5.5,350);
	snack[0]->print();

	snack[1] = new Candybar("Snickers",4.5,450);
	snack[1]->print();

	snack[2] =new Candybar("Mars",3.5,550);
	snack[2]->print();

	delete[] snack;
cin.get();
}

Теперь понятно?

[pproger]

2Stilet

Вместо того чтоб развозить демагогию просто нужно создавать обьекты динамически:

это не дамагогия, а реальная проблема. естественно можно сделать через дин память, как ты показал, но раз человек сделал по другому, надо указать на его ошибки, а не навязывать свой вариант.

2JOFRIF
ты меня просто убил этой строкой:

cout<<"Manufacturer name has been deleted successfully "<<manufacturer_name<<endl;
Manufacturer name has been deleted successfully [][][][][][][]
то это означало что память освобожденна, а иначе нет.

ты вообще адекватен? НЕЛЬЗЯ ИСПОЛЬЗОВАТЬ УКАЗАТЕЛЬ СРАЗУ ПОСЛЕ ТОГО, КАК БЫЛ ПРИМЕНЕН delete! Он стал указывать на область памяти, которая под него не выделена!

А ты вообще говоришь про
snack[0] = Candybar("Bounty",5.5,350);
Только я не въехал с чего ты взял что Candybar("Bounty",5.5,350); это объект??? Это вообще конструктор

а что является результатом выполнения конструктора? в данном случае - временный объект, который удалится сразу после присваивания

вобщем, вот пример с перегрузкой оператора=

#include <iostream>
#include <string>

using namespace std;

class Candybar {
private:
	char *manufacturer_name;
	double weight;
	int calorie_content;
public:
	Candybar();
	Candybar(char *m_name,double wt,int cc);
	void operator=(const Candybar &obj);
	void print();
	~Candybar();
};

Candybar::Candybar() {

}

void Candybar::operator=(const Candybar &obj) {
	this->manufacturer_name = new char[strlen(obj.manufacturer_name)+1];
	strcpy(this->manufacturer_name, obj.manufacturer_name);
	this->weight = obj.weight;
	this->calorie_content = obj.calorie_content;
}

Candybar::Candybar(char *m_name,double wt,int cc) {
	manufacturer_name = new char[strlen(m_name)+1];
	strcpy(manufacturer_name,m_name);
	weight = wt;
	calorie_content = cc;
}

void Candybar::print() {
	cout<<"The manufacturer name is: " << manufacturer_name<<endl;
	cout<<"The weight is: " << weight << endl;
	cout<<"The calorie content is: " << calorie_content << endl;
}

Candybar::~Candybar() {
	cout << "Deliting manufacturer name " << manufacturer_name << endl;
	delete [] manufacturer_name;
	cout << "Manufacturer name has been deleted successfully" << endl;
}

void main() {
	Candybar snack[3];

	snack[0] = Candybar("Bounty",5.5,350);
	snack[0].print();

	snack[1] = Candybar("Snickers",4.5,450);
	snack[1].print();

	snack[2] = Candybar("Mars",3.5,550);
	snack[2].print();	
}

на остальные вопросы не хочу отвечать, ибо ты некомпетентен

[Stilet]

а не навязывать свой вариант.

Но это ведь проще . Да и общепринято с обьектами работать через динамическую память все равно.
Впрочем автор пусть решает конечно же.

[JOFRIF]

Всем спасибо, буду разбираться.