ich bin noch ziemlich neu in C++ und hab ein Problem:

Ich soll über cin mehrere Objekte(Punkt, Polygon und Gruppe) einlesen, wobei diese dann natürlich auch erst angelegt werden sollen. Punkt und Polygon haben eine eigene überladung des istream.

Gruppe auch, aber hier komm ich nicht weiter:
Eine Gruppe kann mehrere Objekte beinhalten(auch wieder Gruppen)
Daher wollte ich versuchen in der istream-Methode je nach eingelesenem Objekt wieder die istream Methode aufzurufen, nur leider weiß ich nicht wie.

Hier mal mein Code:

istream& operator>>(istream& is, GeoGroup& g)
{
	// Eingabe in der Form: 
	// GeoGroup ID: nGG  Objekte: 
	// <
	// GeoPoint ID: pointtemp  Geometrie: (6,6)
	// GeoGroup ID: mGG  Objekte: 
	// <
	// GeoPoint ID: hPoi  Geometrie: (8,10)
	// GeoPolygon ID: jPol  Geometrie: {(-1,8)(0,10)(-4,11)(3,6)(-1,8)}
	// >
	// GeoPolygon ID: polygontemp  Geometrie: {(4,6)(3,4)(2,5)(4,6)}
	// >

	string id;
	string nix;
	string zeile;
	string objekt;

	is.ignore(13);
	// Extrahieren der ID
	getline(is, id, ' ');
	// Setzen der eingelesenen ID
	g.setID(id.c_str());
	// Bis zum Ende der Zeile lesen
	getline(is, nix, '\n');
	// Die öffnende spitze Klammer einer Geogroup überlesen
	getline(is, nix, '\n');
	// Solange weiterlesen bis eine schließende spitze Klammer auftaucht
	while(is.peek() != '>')
	{
	  // Auslesen der Objektbezeichnung
	  getline(is, zeile, '\n');
	  // Prüfen ob GeoPoint
	  if(!zeile.find("GeoPoint"))
	  {
		  GeoPoint temp("0");
		  is >> temp;
		  cout << "Punkt" << endl;
	  }
	  // Prüfen ob GeoPolygon
	  if(!zeile.find("GeoPolygon"))
		  cout << "Polygon" << endl;
	  // Prüfen auf GeoGroup
	  if(!zeile.find("GeoGroup"))
		  cout << "Gruppe" << endl;
	}

	return is;
}

Das Problem taucht in der While-Schleife auf, wo ich versuche die zeile über den istream in temp einzulesen.

Kann mir jemand helfen? Ist sowas überhaupt möglich?

Hoffe man konnte mein Problem verstehen

Kann mir jemand helfen?

Mit Sicherheit.

Gloem schrieb:

Ist sowas überhaupt möglich?

Bestimmt.

Gloem schrieb:

Hoffe man konnte mein Problem verstehen

Nein, kann man nicht. Du hast dein Problem nämlich gar nicht geschildert.

der obige Code ist die Überladung des istream >> Operators in der Klasse GeoGroup.

Die Klasse GeoGroup hat ein vector-Objekt in dem Geopoints, GeoPolygone und auch GeoGroups selber wieder enthalten sein können. Möchte ich nun ein GeoGroup-Element über cin einlesen wird die obige Methode aufgerufen.

Wenn aber in dieser GeoGroup wieder andere Objekte(z.B. ein GeoPoint) enthalten sind, möchte ich aus der obigen Funktion heraus die istream-Methode des GeoPoints aufrufen. Und genau hieran scheiter ich momentan.
Mit cin >> point kann ich ein GeoPoint einlesen(wenn point ein GeoPoint-Objekt ist) und mit cin >> polygon ein GeoPolygon

Als Code nochmal

// Hier prüfe ich, das die eingelesene Zeile einen GeoPoint enthält
if(!zeile.find("GeoPoint"))
      {
          // Anlegen eines Geopoint-Objektes
          GeoPoint temp("0");

          // PROBLEM: Hier soll nun die Zeile die den GeoPoint enthält mit dem istream des GeoPoints eingelesen werden nur weiß ich nicht wie
          is >> temp; // Funktioniert nicht
          // im Hauptprogramm wäre der Aufruf hier cin >> temp (aber cin hab ich ja schon als string Zeile eingelesen)

          //Geometrie zur GeoGroup hinzufügen
          this.addGeom(temp);
      }

(btw, was soll die Bedingung if(!zeile.find("GeoPoint"))... eigentlich bedeuten? find() liefert keinen Boolean zurück, sondern die Position, an der der Teilstring gefunden wurde.)

(btw, was soll die Bedingung if(!zeile.find("GeoPoint"))... eigentlich bedeuten? find() liefert keinen Boolean zurück, sondern die Position, an der der Teilstring gefunden wurde.)

Der Compiler sieht aber 0 als wahr und ungleich 0 als false an. Auf jedenfall funktioniert das und im Falle eines Punktes komm ich auch in den angegebenen Teil.

Ich versuch jetzt nochmal mein Problem zu erklären:

Ich habe eine Klasse GeoShape von der die Klassen GeoPoint, GeoPolygon und GeoGroup erben. Ein Geopoint ist nur ein Punkt, ein GeoPolygon ist ein Polygon und eine GeoGoup ist eine Sammlung von Punkten, Polygonen oder wieder Gruppen.

Der Benutzer soll nun über die Kommandozeileneingabe eine Geometrie eingeben können, wofür in jeder Klasse der globale istream überladen wurde

Wenn ich z.B. einen Punkt von der Kommandozeile einlesen will, muss der Benutzer den nach diesem Schema eingeben:

GeoPoint ID: pointtemp  Geometrie: (6,6)

mit GeoPoint p();
cin >> p;
wird er dann eingelesen

Wenn ich ein Polygon von der Kommandozeile einlesen will, muss der Benutzer den nach diesem Schema eingeben:

GeoPolygon ID: jPol  Geometrie: {(-1,8)(0,10)(-4,11)(3,6)(-1,8)}

wieder mit GeoPolygon poly();
cin >> poly;

Diese beiden Fälle funktionieren ohne Probleme

Das Problem ist nun die GeoGruppe
**Ich möchte hier in der Überladung des istreams der GeoGroup die istreams der anderen Klassen aufrufen.
**

Der Benutzer gibt die GeoGroup dann so ein:

GeoGroup ID: nGG  Objekte: 
<
GeoPoint ID: pointtemp  Geometrie: (6,6)
GeoGroup ID: mGG  Objekte: 
<
GeoPoint ID: hPoi  Geometrie: (8,10)
GeoPolygon ID: jPol  Geometrie: {(-1,8)(0,10)(-4,11)(3,6)(-1,8)}
>
GeoPolygon ID: polygontemp  Geometrie: {(4,6)(3,4)(2,5)(4,6)}
>

Code aus der Überladung des istreams der GeoGroup

// Sollange keine spitze schließende Klammer an Ende einer GeoGroup kommt
while(is.peek() != '>')
    {
      // Auslesen der Objektbezeichnung
      getline(is, zeile, '\n');

      // Prüfen ob GeoPoint Ojekt eingelesen wurde
      if(!zeile.find("GeoPoint"))
      {
Hier soll nun der Aufruf des überladenen istream operators des GeoPoints kommen
          cout << "Punkt" << endl;
      }
      // Prüfen ob GeoPolygon
      if(!zeile.find("GeoPolygon"))
Hier soll nun der Aufruf des überladenen istream operators des GeoPolygons kommen
          cout << "Polygon" << endl;
      // Prüfen auf GeoGroup
      if(!zeile.find("GeoGroup"))
Hier soll nun der Aufruf des überladenen istream operators der GeoGroup kommen
          cout << "Gruppe" << endl;
    }

Geht's ein wenig präziser als "funktioniert nicht"?

Leider kann ich es nicht genauer schreiben, da ich nicht weiß wie der Code auszusehen hat und ob ein Aufruf des istream operators>> im Quellcode ohne Kommandozeile überhaupt möglich ist.

(btw, was soll die Bedingung if(!zeile.find("GeoPoint"))... eigentlich bedeuten? find() liefert keinen Boolean zurück, sondern die Position, an der der Teilstring gefunden wurde.)

Der Compiler sieht aber 0 als wahr und ungleich 0 als false an.

Andersum.

Auf jedenfall funktioniert das und im Falle eines Punktes komm ich auch in den angegebenen Teil.

Du suchst nicht, sondern prüfst nur, ob zeile mit dem gesuchten String anfängt. Nur dann liefert dir find nämlich 0 zurück.

Geht's ein wenig präziser als "funktioniert nicht"?

Leider kann ich es nicht genauer schreiben, da ich nicht weiß wie der Code auszusehen hat und ob ein Aufruf des istream operators>> im Quellcode ohne Kommandozeile überhaupt möglich ist.

Klar ist das möglich. Aber du musst den Fehler besser beschreiben. Mit "funktioniert nicht" kann niemand etwas anfangen.

Was ist das beobachtete Verhalten genau? Fehlermeldung, falsche Werte, Absturz usw. Irgendwie glaube ich nicht, dass da nur eine Meldung "Funktioniert nicht" auf dem Bildschirm erscheint.

GeoPoint p;
istringstream l(zeile);
l>>p;
...

(nur bei der mehrzeiligen Darstellung einer GeoGroup könnte es da ein Problem geben - weil du dort die erste Zeile in der Hand und den Rest der Daten im Stream hast. Als Lösungsansatz könntest du deren operator>> so anpassen, daß er auch Eingaben akzeptiert, die mit ">" beginnen (da mußt du die Gruppen-ID nachträglich ergänzen).)

PS: Eine Alternative Lösung wäre es, GeoShape eine read-Methode zu geben:

void read(string first_line,istream& input)
{
  if(first_line=="") getline(input,first_line);
  ...
}

istream& operator>>(istream& input,Geoshape&data)
{
  data.read("",input);
  return input;
}

Das erste funktioniert schonmal, jetzt muss ich nur noch an die GeoGroup machen

GeoPoint *pGeoPoint = dynamic_cast<GeoPoint*>(*first);

Dann eben doch die zweite wahrscheinlich auch bessere Variante:

Aber wofür brauch ich in der read-Methode den String? Geht das nicht auch ohne?

GeoGroup.h

class GeoGroup : public GeoShape
{
public:
	...

	void read(istream& input);

	...
};

GeoGroup.cpp

void GeoGroup::read(istream& input)
{
	// Hier den ganzen Rotz aus der alten istream-Methode
        if(!zeile.find("GeoGroup"))
		  read(is);
}

// Überladen des globalen Output-Streams für die Eingabe einer GeoGroup
istream& operator>>(istream& is, GeoGroup& g)
{
	g.read(is);

	return is;
}

Ok, es funktioniert doch nicht, er wirft mich dann bei der Ausgabe raus wenn ich versuche mittels dynamic_cast mein Objekt aus dem Vektor wieder herauszubekommen

GeoPoint *pGeoPoint = dynamic_cast<GeoPoint*>(*first);

Mit was für einer Meldung wirft er dich heraus? Und was genau steht in first? Und wie wird es gefüllt?

Dann eben doch die zweite wahrscheinlich auch bessere Variante:

Aber wofür brauch ich in der read-Methode den String? Geht das nicht auch ohne?

Damit wollte ich das Problem aushebeln, daß du möglicherweise den ersten Teil deines Objekts schon ausgelesen hast, wenn du den Stream weitergibst (IOStream's bieten zwar die Möglichkeit, eingelesene Zeichen zurückzuschreiben, aber der Standard garantiert nicht, daß das bei so langen Folgen fehlerfrei funktioniert). Die GeoGroup::read()-Methode kann damit die bereits eingelesene Zeile des Sub-Objekts an dessen read()-Methode weitergeben:

if(zeile.find("GeoPoint")==0)
{
  GeoPoint p;
  p.read(zeile,input);
  ...
}

Der Fehler sieht so aus:
PopUp mit Unhandled exception at 0x7c812a5b in Pflichtblatt2.exe: Microsoft C++ exception: std::__non_rtti_object at memory location 0x0013f9ac..

Auftreten passiert während des cout << GeoGroup Aufrufs
Das liegte daran, das er die Geometrien nicht korrekt in den Vector speichert.

Mein Vector sieht so aus:

typedef vector<GeoShape*> Geovektor;

sprich er legt die Objekte Polymorph zur abstrakten Oberklasse GeoShape ab

Eingefügt werden die mit dieser Methode:

// Methode um GeoObjekte zur GeoGroup hinzuzufügen
void GeoGroup::addGeoObject(GeoShape* geo)
{
		itsgeos.push_back(geo);
}

aus der Mainmethoder heraus funktioniert auch super, nur aus der neuen read-Methode nicht, da ich es nicht hinbekomme, einen GeoShape zu erzeugen

if(!zeile.find("GeoPoint"))
		{
			GeoPoint temp("0");
			istringstream l(zeile);
			l >> temp;
			this->addGeoObject(&temp);
		}

aus der Mainmethoder heraus funktioniert auch super, nur aus der neuen read-Methode nicht, da ich es nicht hinbekomme, einen GeoShape zu erzeugen

Du erzeugst das Objekt schon - es lebt nur nicht allzu lange, nämlich nur bis zur schließenden geschweiften Klammer. Danach hast du einen Zeiger auf ein nicht mehr existierendes Objekt in deinem vector.

Leg die Objekte mit new an.

if(!zeile.find("GeoPoint"))
{
  GeoPoint* temp= new GeoPoint("0");
  istringstream l(zeile);
  l >> *temp;
  this->addGeoObject(temp);
}

^* Aber ich bin mir hier nicht ganz sicher über die Besitzverhältnisse (kann der selbe Punkt in mehreren Gruppen enthalten sein?) - besser wäre es womöglich, Smart-Pointer zu verwenden.

Hatte es auch schon mit new() erzeugt aber vergessen das Referenzzeichen bei

this->addGeoObject(temp);

zu entfernen und dann hatte ich an der Baustelle erstmal aufgegeben

Der Fehler sieht so aus:
PopUp mit Unhandled exception at 0x7c812a5b in Pflichtblatt2.exe: Microsoft C++ exception: std::__non_rtti_object at memory location 0x0013f9ac..

Ist GeoShape dynamisch? Wenn nicht füge einfach mal virtual ~GeoShape() { } hinzu.

meine beiden Methoden istream& operator>>(..) und read(...) sehen nun so aus:

// Methode zum Einlesen einer GeoGroup
void GeoGroup::read(string first_line, istream& is)
{
	// Eingabe in der Form: 
	// GeoGroup ID: nGG  Objekte: 
	// <
	// GeoPoint ID: pointtemp  Geometrie: (6,6)
	// GeoGroup ID: mGG  Objekte: 
	// <
	// GeoPoint ID: hPoi  Geometrie: (8,10)
	// GeoPolygon ID: jPol  Geometrie: {(-1,8)(0,10)(-4,11)(3,6)(-1,8)}
	// >
	// GeoPolygon ID: polygontemp  Geometrie: {(4,6)(3,4)(2,5)(4,6)}
	// >

	string id;
	string nix;
	string zeile;

	// Wenn keine Zeile übergeben wurde -> erster Aufruf(keine Rekursion)
	if(first_line=="")
	{
		getline(is,first_line, '\n');
	}
	// Abschneiden der ersten Teils des String bis zur ID
	id = first_line.substr(12, first_line.length());
	// Abschneiden des hinteren Teils des Strings zur ID
	id = id.substr(0, id.find(" "));
	// Setzen der eingelesenen ID
	this->setID(id.c_str());
	// Bis zum Ende der Zeile lesen
	getline(is, nix, '\n');
	// Die öffnende spitze Klammer einer Geogroup überlesen
	getline(is, nix, '\n');
	// Solange weiterlesen bis eine schließende spitze Klammer auftaucht
	while(is.peek() != '>')
	{
		// Auslesen der Objektbezeichnung
		getline(is, zeile, '\n');
		// Prüfen ob GeoPoint
		if(!zeile.find("GeoPoint"))
		{
			GeoPoint* temppoi = new GeoPoint("0");
			istringstream l(zeile);
			l >> *temppoi;
			this->addGeoObject(temppoi); 
		}
		// Prüfen ob GeoPolygon
		if(!zeile.find("GeoPolygon"))
		{
			GeoPolygon* temppol = new GeoPolygon("0");
			istringstream l(zeile);
			l >> *temppol;
			this->addGeoObject(temppol);
		}
		// Prüfen auf GeoGroup
		if(!zeile.find("GeoGroup"))
		{
			GeoGroup* tempgg = new GeoGroup("0");
			// Rekursiver Aufruf der read-Methode mit der Übergabe des GeoGroup-Anfangstrings
			(*tempgg).read(zeile, is);
			this->addGeoObject(tempgg);
		}
	}
}

// Überladen des globalen Output-Streams für die Eingabe einer GeoGroup
istream& operator>>(istream& is, GeoGroup& g)
{
	// Aufruf der Methode zum Einlesen einer GeoGroup
	g.read("", is);

	return is;
}

Das Problem ist nun, das wohl nicht mehr alles verarbeitet wird und ich weiß nicht warum:

Eingabe:

GeoGroup ID: nGG  Objekte: 
<
GeoPoint ID: pointtemp  Geometrie: (6,6)
GeoGroup ID: mGG  Objekte: 
<
GeoPoint ID: hPoi  Geometrie: (8,10)
GeoPolygon ID: jPol  Geometrie: {(-1,8)(0,10)(-4,11)(3,6)(-1,8)}
>
GeoPolygon ID: polygontemp  Geometrie: {(4,6)(3,4)(2,5)(4,6)}
>

Ausgabe:

Ausgabe: GeoGroup ID:   Objekte:
<
GeoGroup ID:   Objekte:
<
GeoPolygon ID: jPol  Geometrie: {(-1,8)(0,10)(-4,11)(3,6)(-1,8)}
>
>

Kann mir jemand sagen, warum nicht mehr alle Objekte erzeugt werden? Irgendwie scheint mein this-Zeiger nicht mehr zu funktionieren

Vielen Dank an alle, jetzt funktionierts. So viele und so schnelle Hilfe hatte ich bislang selten in einem Forum