ich muss eineige Aufgaben programmieren und weiß echt nicht mehr weiter. Die müssen bis Montag fertig sein und ich weiß nichtmal wo ich anfangen soll

Wäre echt super wenn mir da jemand helfen könnte. Ich bin mir sicher das ist eigentlich nur eine sache von 5 min. Hätte ich mehr zeit würde ich auch versuchen mich da reinzuarbeiten aber die zeit fehlt leider

Hier die Aufgabe die ich lösen muss:

Schreiben Sie einen Kopierkonstruktor, der die Zeichenkette des zu kopierenden Objektes übernimmt, aber nicht seine Adresse, sondern die Adresse des neuen Strings in dem neuen Objekt ablegt.

#include <iostream>
#include <string>
#include <vector>

using namespace std;

class Test {

private:
	string str;
	string * pStr;

public:
	Test (string);
	string getString() {
		return str;
	}

};

Test::Test (string str) : str(str) {}

int main() {

return 0;
}

Hätte ich mehr zeit würde ich auch versuchen mich da reinzuarbeiten aber die zeit fehlt leider

Hätten wir mehr Zeit würden wir dir vielleicht auch helfen. Es sind deine Hausaufgaben nicht unsere. Wer sich noch nicht mal selbst Mühe macht, soll sich nicht wundern wenn andere die Bereitschaft ebensowenig haben.

cu André

...

...

class Test {
...
	Test (string);
...

Das ist kein Kopierkonstruktor, weil er nicht ein Testobjekt in ein anderes (neu zu erstellendes) kopiert, sondern ein "normaler" Konstruktor.

Außerdem scheinst Du ziemlich unmotivert einen string* in Dein Objekt aufgenommen zu haben ... vermutlich weil in der der Aufgabenbeschreibung das Wort "Zeiger" auftaucht.

Bei Deinem Beispiel könntest Du einfach alle Konsturktoren weglassen .. dann würde der COmpiler (so wie es der Standard fordert) automatisch einen Kopierkonstruktor erstellen, wie Deine Aufgabe ihn fordert.
Du könntest ihn auch per Ahand erstellen (so ähnlich, wie Du's schon versucht hast ... nur, dass der Konstruktor halt keinen "string" als Parameter haben muss, sondern ein "Test const&".

Ich vermute allerdings, dass Dein Prof (shame on him!!) von Dir "Zeichenkettenverarbeitung via char*" (mit allen Fingerbrechereien und Gefahrenquellen) erwartet und da ich sowas garstig finde, kann ich mich kaum überwinden da weiterzuhelfen.

(OK, kleines Stichwort: str als char[] mit fixer Größe deklarieren, Inhalt mittels strcpy rüberkopieren und Prof mit einem nassen Lappen sachlagen).

Gruß,

Simon2.

Der Code ist wohl eher der Rohbau (die Vorgabe) der Aufgabe.
Hui99 hat vermutlich noch nichts geändert.

Was ich bis jetzt habe ist:

Test(Test & const kette) {	
string = new str;
str = kette.getString();
}

und natürlich das was ganz oben steht.

Aber da bekomme ich gut viele fehler vor allem
error C2143: Syntaxfehler : Fehlendes ';' vor '}'

aber mir wird nicht ersichtlich wo

Das kann gar nicht gut gehen (eine Referenz ist immer konstant). Schreib eines der folgenden Dinge:

Test(const Test& kette)
Test(Test const& kette)

string = new str;
str = kette.getString();

Hier wieder: Ist str nun ein Typ oder ein Member? Man versteht nicht, was du machen willst. Ausserdem kann man im Konstruktor Initialisierungslisten verwenden. Und vergiss nicht, einen entsprechenden Destruktor und Zuweisungsoperator selber zu implementieren, sonst hast du relativ schnell Probleme mit den Zeigern...

Aber da bekomme ich gut viele fehler vor allem
error C2143: Syntaxfehler : Fehlendes ';' vor '}'

Die Häufigkeit der Fehler ist meistens nicht relevant, da aus einem viele andere resultieren können. Schau dir am besten immer den ersten Fehler an.

Hier mal das was ich habe:

#include <iostream>
#include <string>
#include <vector>

using namespace std;

class Test {

private:
	char str;
	char * pStr;

public:
	Test (char);
	char getString() {
		return str;
	}

Test(Test &const);

void datenAus();

};

Test::Test(char iWert): str(iWert), pStr(&str){

}

/*Hier bekomme ich einen fehler mit dem ich aber nicht anfangen kann: 
error C2351: Veraltete Syntax fuer die Initialisierung eines C++-Konstruktors
*/
Test::Test(Test & objekt):(objekt.str), pStr(&str){
}

void Test::datenAus() {
	cout <<"str = " << str << ", pStr = " << pStr << endl;
}

int main() {

	return 0;
}

Macht das Porgamm jetzt eigentlich das was die aufgabe von mir will?

Hier nochmal die Aufgabestellung?
Schreiben Sie einen Kopierkonstruktor, der die Zeichenkette des zu kopierenden Objektes übernimmt, aber nicht seine Adresse, sondern die Adresse des neuen Strings in dem neuen Objekt ablegt.

Jetzt bleibt nur noch die frage ob die aufgabe richtig gelöst ist ^^

Ansonsten bist du auf dem richtigen Weg.

Jetzt die Aufgabe richtig gelöst?

#include <iostream>
#include <string>
#include <vector>

using namespace std;

class Test {

private:
	string str;
	string * pStr;

public:
	Test (string);
	string getString() {
		return str;
	}

Test(Test &const);

void datenAus();

};

Test::Test(string iWert): str(iWert), pStr(&str){

}

Test::Test(Test & objekt):str(objekt.str), pStr(&str){
}

void Test::datenAus() {
	cout <<"str = " << str << ", pStr = " << pStr << endl;
}

int main() {

Test k21("Hallo");
Test k22(k21);
k21.datenAus();
k22.datenAus();	
	return 0;
}

Du hast den Tipp von Nexus nicht beachtet.

Sowohl innerhalb der Klassendefinition als auch in der Definition des Copy-Konstruktors ist der const-Qualifier nicht ganz korrekt angewendet.

Ansonsten ist das OK. Hat sich doch gelohnt es selbst zu versuchen, was?

wie kann es sein das dein Beispiel läuft

Naja, egal. Für mich stellen sich hier ein paar Fragen. Die erste ist wohl, ob du den Unterschied zwischen char, char*, char[], string und string* kennst

Test(Test &const);

@kritiker: Wieso speichert er eigentlich den Zeiger und den Typ ab?

Das mit dem const habe ich eh noch nicht so ganz verstande. Naja dann mal weiter zur nächsten aufgabe

Mal sehen, wie weit du mit der Einstellung kommst, bzw. wie oft du den gleichen Fehler noch machen wirst...

Wenn es ein gutes Buch ist, sollte es schon erklärt haben, was es mit dem const auf sich hat. Und ansonsten hindert dich niemand daran, weitere Recherchen anzustellen. Zumindest hat es keinen Wert, eine Klasse zu schreiben, von der man über die Hälfte nicht versteht (wie Siassei sagte, auch die Typen solltest du noch einmal genauer anschauen)...

Aber die ist jetzt auch behoben.

Hier noch mal der Quellcode. Hoffe jetzt ist wirklcih alles richtig.

#include <iostream>
#include <string>
#include <vector>

using namespace std;

class Test {

private:
	string str;
	string * pStr;

public:
	Test (string);
	string getString() {
		return str;
	}

Test(const Test& objekt);

void datenAus();

};

Test::Test(string iWert): str(iWert), pStr(&str){

}

Test::Test(const Test & objekt):str(objekt.str), pStr(&str){
}

void Test::datenAus() {
	cout <<"str = " << str << ", pStr = " << pStr << endl;
}

int main() {

Test k21("Hallo");
Test k22(k21);
k21.datenAus();
k22.datenAus();	
	return 0;
}

@Siassei
Wus? Wo?