string_klasse.h

class String {
private:
	char *buffer;
	unsigned len;
public:
	// Deklaration der Methoden(Fähigkeiten) von String

	// Zugriffsmethode 
	char *get_String() const;

	// Operator Überladung
	String operator+=(const String &str1);
};

string_klasse.cpp

// Definition der Methoden(Fähigkeiten) von String

// Zugriffsmethode
char *String::get_String() const { return buffer; }

// Operator Überladung
String String::operator+=(const String &str1) {
	// tmp gleich mit aktuellen Objekt initialisieren
	String tmp(*this);

	// Aktuelles Objekt löschen
	delete [] buffer;

	// Gesamtlänge beider Objekte ermitteln
	len = tmp.len + str1.len;

	// Speicher reservieren
	buffer = new char[len]+1;

	// linker Operand in buffer kopieren
	strcpy(buffer, tmp.buffer);

	// rechten Operand anhängen
	strcat(buffer, str1.buffer);

	// Zurück und fertig
	return buffer;
}

main.cpp

String str("Adam");

	str += " und Eva";

	cout << str.get_String();

zum Problem
Schaut bitte mal auf den main.cpp Abschnitt. Ich möchte gerne an den Text Adam mit dem neuen überladenen Operator "+=" den Text " und Eva" anhängen.

Die Ausgabe stimmt soweit. Allerdings kommt auch ein Fehler, den ich als JPG Bild festgehalten habe. Wie ist dieser zu beheben?

http://home.tiscali.de/myflash/operator_ueberladen.JPG

Laut meinem C++ Buch ist die obige Zeile total in Ordnung.

Wenn man das so umschreibt

// Speicher reservieren
buffer = new char[len+1];

klappts ohne Fehlermeldung. Das scheint wohl der Fehler zu sein.

Trotzdem danke.

MfG
#smY

Im Zweifelsfall bitte auch folgende Hinweise beachten:
C/C++ Forum :: FAQ - Sonstiges :: Wohin mit meiner Frage?

Dieses Posting wurde automatisch erzeugt.

also get_string sollte schon einen const char zurück geben ... sonnst kann es dir passieren das len nachher nichtmehr stimmt! (falls jemand was an deinem String dann verändert)

#include <iostream>

int main()
{
    std::cout << basic_string<char>("Hallo Welt!") << std::endl;
}

Bearbeitung
Hab dir mal kurz die großen 3 reingepackt

// Speicher reservieren
buffer = new char[len]+1;

Laut meinem C++ Buch ist die obige Zeile total in Ordnung.

syntaktisch ist die in Ordnung, technisch nicht. Du unterschlägst erstens den Platz für den Null-Terminator und überpringst zweitens das erste Byte des angeforderten Speicherbereiches.

PS: Ich hoffe für dich, deine String-Klasse hat die "großen drei" implementiert (Copy-Ctor, op= und Dtor)

PPS: Was spricht dagegen, std::string zu verwenden?