Kein Zugriff auf private Member

[kenner der dummköpfe]

Dein allererstes problem liegt in Zeile 3: Dein lustiger astring hat keinen Zuweisungsoperator...

[kenner der kenner]

kenner der dummköpfe schrieb:

Dein allererstes problem liegt in Zeile 3: Dein lustiger astring hat keinen Zuweisungsoperator...

quatsch in zeile 3 wird der konstruktor aufgerufen

[MFK]

physikus schrieb:

Muss bei der Ausgabe nicht sowas wie

return s << (*beginn).buchstabe

hin?

Eine Schleife wäre auch nicht schlecht. Oder willst du nur das erste Zeichen ausgeben?

nur das nimmt er nicht an

Das ist keine ausreichende Fehlerbeschreibung.

[physikus]

hallo

ja, ich will zuerst mal den ersten buchstaben ausgeben damit ich seh ob es klapptt. danach kommt die schleife.
Und diese Fehlermeldungen werden mir angezeigt

29. : error C2065: 'beginn': nichtdeklarierter Bezeichner
    error C2228: Links von ".buchstabe" muss sich eine Klasse/Struktur/Union befinden.

[MFK]

Du darfst natürlich das "str." nicht weglassen.

[physikus]

return s << str.*beginn.buchstabe;

genau die gleichen fehler

return s << str.(*beginn).buchstabe;

error C2059: Syntaxfehler: '('

[LordJaxom]

Nimm doch einfach das Pfeilchen...

[physikus]

sollte es jemand interessieren, die Lösung:

ostream& operator<<( ostream& s, const astring& str )     //ausgeben 
{ 
		astring::buchstabenliste *temp=str.beginn;

		return s << (*temp).buchstabe;

[inp]

ich habe dir schon in deinem ersten thread gesagt das das speichern eines strings als zeigerverkette liste wohl der schlechteste ansatz ist.

es sei denn du musst einen lehrer befriedigen.

[physikus]

inp schrieb:

es sei denn du musst einen lehrer befriedigen.

Gut erkannt

allesdings läuft meine schleife jetzt auch nicht.

astring::buchstabenliste *temp=str.beginn;

	while ((*temp).buchstabe!='\0')			//
	{
		return s << (*temp).buchstabe;
		temp=temp->next;

Das "H" bekomme ich zwarausgegeben, aber dann ist schluss.
kein zweiter schleifendurchlauf

[LordJaxom]

Das liegt daran dass die Funktion bei "return" beendet wird.

[physikus]

ja, und wie mach ich das ohne return????

[LordJaxom]

Nun, indem Du nicht "return" schreibst. Return beendet nunmal die Funktion und legt den Rückgabewert fest. Vermutlich solltest Du in der Schleife einfach nur die Ausgabe auf den Stream machen und anschließend, nach Abarbeitung der Schleife, die Rückgabe mit "return s;" auslösen.

[MFK]

Edit: Unsinn.

[Simon2]

physikus schrieb:

ja, und wie mach ich das ohne return????

Hi,

überleg' doch mal kurz, was Du eigentlich machen möchtest bei der Ausgabe: Möchtest Du EINMAL einen Buchstaben ausgeben ? Oder möchtest Du ALLE Buchstaben ausgeben ?

In Deinem Statement s << (*temp).buchstabe; wird nur auf EINEN Buchstaben verwiesen ... (*)

physikus schrieb:

sollte es jemand interessieren, die Lösung:

ostream& operator<<( ostream& s, const astring& str )     //ausgeben 
{ 
		astring::buchstabenliste *temp=str.beginn;

		return s << (*temp).buchstabe;

BTW: temp->buchstabe hätte es auch getan. ... oder gar direkt (temp braucht man nicht): str.beginn->buchstabe

Gruß,

Simon2.

(*) HINT: Stichwort ist "Schleife".

[Helium]

struct buchstabenliste { 
   char buchstabe; 
   buchstabenliste *next;

   friend std::ostream & operator<<(std::ostream & os, buchstabenliste const * bs)
   {
      if (bs != 0) {
         os << bs->buchstabe << bs->next;
      }
      return os;
   }
};

//------------

ostream& operator<<( ostream& os, const astring& str )
{ 
   return os << *str.beginn;
}

[Simon2]

Hi Helium,

das löst aber noch nicht physikus' "Problem des ersten Buchstabens"...

Gruß,

Simon2.

[Helium]

Muss ich den ganzen scheiß jetzt ernsthaft lesen, statt nur zu überfliegen? Was ist denn das Problem des ersten Buchstabens? Ich dachte es geht einfach darum alle Buchstaben auszugeben, verdammt.

[Simon2]

Helium schrieb:

...Was ist denn das Problem des ersten Buchstabens? Ich dachte es geht einfach darum alle Buchstaben auszugeben, verdammt.

Hoppla --- da habe ich mich "verlesen".

Sorry, ziehe alles zurück.

Gruß

Simon2.

[physikus]

Alles klar, jetzt funktionierts.
Danke an alle die geholfen haben.

while ((*temp).buchstabe!='#')			//
	{
		s << (*temp).buchstabe;
		temp=temp->next;
	}