proggen.org

Diddi hat geschrieben: In der Funktion parse_line() erkennt man, dass ich ein Dataset dynamisch anlege und mit setVector.push_back(*dataset) dem Vektor hinzufüge. Wenn ich in die Konsole "exit" tippe, wird außerdem setVector.clear() gemacht. Ist das nicht in Ordnung so?

Diddi hat geschrieben:Zur Auswertung von Messdaten schreibe ich ein Programm, das diese zunächst einliest. Jeder eingelesene Datensatz wird in einem Objekt des Typs Dataset gespeichert. Die Datasets werden von der Hauptanwendungsklasse Cpw in einem vector <Dataset> abgelegt.
Es kompiliert nun ohne Fehler, aber zur Laufzeit gibt es Speicherzugriffsfehler, wenn ich entweder mindestens drei Datensätze lade oder wenn ich das Programm beende und mindestens zwei Datensätze geladen sind.

		Dataset *dataset = new Dataset();
		if (dataset == NULL)

		try
		{  
			Dataset *dataset = new Dataset();
		}
		catch( std::badalloc & badalloc )
		{
			cout << "Konnte keinen Speicher reservieren\n";
			return 1;
		}

Diddi hat geschrieben:Ich hänge mal die Dateien cpw.cpp, cpw.h und main.cpp an.
In der Funktion parse_line() erkennt man, dass ich ein Dataset dynamisch anlege und mit setVector.push_back(*dataset) dem Vektor hinzufüge. Wenn ich in die Konsole "exit" tippe, wird außerdem setVector.clear() gemacht. Ist das nicht in Ordnung so?

		setVector.push_back(*dataset);
		activeSet = setVector.size() - 1;

Diddi hat geschrieben:Damit habe ich nun keine Schwierigkeiten mehr. Ich vermute, dass ich einen Kopierkonstruktor oder solches Gedöns hätte definieren müssen.

Diddi hat geschrieben: Ich meine, nun einen angemessenen Destruktor von Cpw zu haben:

Code: Alles auswählen

Cpw::~Cpw()
{
	for (unsigned int i = 0; i < setVector.size(); i++)
		delete setVector[i];
}

In der Klasse Dataset gehe ich entsprechend vor, nur dass ich dort eine Liste habe.

Cpw::~Cpw()
{
	for (unsigned int i = 0, n = setVector.size(); i < n; i++)
		delete setVector[i];
}

Cpw::~Cpw()
{
	for (auto it=setVector.begin(), eit=setVector.end(); it!=eit; it++)
		delete *it;
}

Cpw::~Cpw()
{
	for (Dataset * value : setVector)
		delete value;
}

Diddi hat geschrieben:Xin, vielen Dank für deine Kommentare zur Exception und zur Subtraktion

Architekt hat geschrieben:Ich empfehle hierbei ganz stark smart pointer wie std::unique_ptr. Die nehmen dir diese lästige manuelle Freigabe ab.

Architekt hat geschrieben:Ich empfehle hierbei ganz stark smart pointer wie std::unique_ptr. Die nehmen dir diese lästige manuelle Freigabe ab.