mein Objekt ist wie folgt aufgebaut

int main(int argc, char *argv[]) {

   std::vector<mStructure> structureList; // Vector from my class

   ...
}

class mStructure{
   ...
private:
   int structureID_;
   int numTraces_;
   std::string name_;
   int color_;
   mTrace trace_;
};

class mTrace{
   ...
private:
   int traceID_;
   int color_;
   int numPoints_;
   std::vector<mPoint> points_;
};

Problem ist nun, dass ich den Vektor vom Typ mStrucutere am Anfang anlege und später erst die Daten einlese. Beim einlesen erfahre ich aber erst wieviele Punkte in mTrace eingelesen werden. Doch der erste Eintrag von dem Vektor points_ verändert sich immer. Ich vermute, dass beim Programmstart Speicherbereich für structureList festgelegt wird und wenn ich im späteren Verlauf Punkte mittels push_back() hinzufüge, es eigentlich zu einer Bereichsüberschreitung kommt. Mich irritiert nur, dass die anderen Elemente vom Vektor points_ in Ordnung sind, nur der erste Eintrag ist falsch. Hat jmd eine Idee, wie ich das beheben kann? Oder ggf ein anderes Design?

<SectionTracer numStructures="6">
  <Structure structureID="0" numTraces="1" name="2a_outer" color="-256">
    <Trace traceID="1" color="-256" numPoints="45">
      <Point num="0">4949 1751</Point>
      <Point num="1">4977 1676</Point>
      <Point num="2">5005 1608</Point>

TraceID: 1 Color: -256 NumPoints: 45
X: 17 Y: 0
X: 4949 Y: 1751
TraceID: 1 Color: -256 NumPoints: 45
X: 210 Y: 0
X: 4949 Y: 1751
X: 4977 Y: 1676
TraceID: 1 Color: -256 NumPoints: 45
X: 152 Y: 0
X: 4949 Y: 1751
X: 4977 Y: 1676
X: 5005 Y: 1608
...

Achso, zu erwähnen wäre, dass ich auf Grund des Fehlers ein leeres Objekt hinzufüge.

if (result->points().empty()) {// TODO Speicherkorrektur
		result->addPoint(mPoint(0, 0));
	}

void read_xml(const char *pFilename, vector<mStructure> &structureList,	mPNGinfo &fi) {

	//open file
	TiXmlDocument doc(pFilename);
	if (!doc.LoadFile())
		throw runtime_error(string("can't read ") + pFilename);

	//check correctness of file
	TiXmlElement *root = doc.RootElement();
	if (strcmp(root->Value(), "SectionTracer") != 0)
		throw runtime_error(string("wrong root ") + root->Value());

	//read filename
	TiXmlElement* e = root->FirstChildElement();
	if (e && strcmp(e->Value(), "Filename") == 0) {
		fi.setName(textValue(e));
	} else {
		throw runtime_error("error filename");
	}

   ...

	//read Structures
	for (TiXmlElement* element = e->NextSiblingElement(); element; element = element->NextSiblingElement()){
		structureList.push_back(nodeToStructure(element));
	}

}

mStructure nodeToStructure(TiXmlElement* e) {

	if (strcmp(e->Value(), "Structure") != 0)
		throw runtime_error(string("wrong structure ") + e->Value());

	mStructure *result = new mStructure;

   ...

	result->setTrace(nodeToTrace(e->FirstChildElement()));

	return *result;
}

mTrace nodeToTrace(TiXmlElement* e) {

	if (strcmp(e->Value(), "Trace") != 0)
		throw runtime_error(string("wrong trace ") + e->Value());

	mTrace *result = new mTrace;
	const char *traceID, *color, *numPoints;

	//read Parameter
	traceID = e->Attribute("traceID");
	if (traceID != NULL)
		result->setTraceID(atoi(traceID));
	else
		throw runtime_error(string("wrong traceID ") + e->Value());

	color = e->Attribute("color");
	if (color != NULL)
		result->setColor(atoi(color));
	else
		throw runtime_error(string("wrong color ") + e->Value());

	numPoints = e->Attribute("numPoints");
	if (numPoints != NULL)
		result->setNumPoints(atoi(numPoints));
	else
		throw runtime_error(string("wrong numPoints ") + e->Value());

	if (result->points().empty()) {// TODO Speicherkorrektur
		result->addPoint(mPoint(0, 0));
	}

	for (TiXmlElement* element = e->FirstChildElement(); element; element = element->NextSiblingElement()) {
		result->addPoint(nodeToPoint(element));
		cout << *result;
	}

	return *result;
}

void addPoint(const mPoint &pt){points_.push_back(pt);};

new hatte ich gemacht, weil mir das jmd empfohlen hatte. Das sollte den Speicherfehler beheben. Hat aber wie man sieht nichts gebracht.

new hatte ich gemacht, weil mir das jmd empfohlen hatte. Das sollte den Speicherfehler beheben. Hat aber wie man sieht nichts gebracht.

Dann hat dich da aber jemand ziemlich an der Nase rumgeführt.. Mit new forderst du Speicher an, den du selbst verwalten musst. Und wenn du ihn nicht freigibst, hast du erst Recht ein Speicherleck..