eine Klasse hat ein Attribut

char* str;

Wie würde man dafür einen Copy-Konstruktor schreiben?

so??

T(const& T t)
{
    str = new char[32];
    strcpy(str, t.str);
}

T( const T& other )
{
  // ...
}

Beim kopieren musst du darauf achten das keine Memoryleaks entstehen (bei Exceptions oder Selbstzuweisung).

Eine Idee die Selbstzuweisung zu unterbinden ist folgende:

T( const T& other )
{
  if ( this == &other )
    return;

  // ...
}

Andererseits kann man sich diese Abfrage auch sparen:

T( const T& other )
{
  char* tmp = str;
  str = new char[ 32 ];
  strcpy( str, other.str );
  delete [] tmp; // speicher freigeben ist wichtig
}

Wobei du darauf achten solltest das str gültig oder NULL ist. Außerdem solltest du evtl die Länge des Strings von "other" berücksichtigen.

Und um sich den ganzen ärger zu sparen könntest du einfach std::string verwenden statt char*.

grüße

Eine Idee die Selbstzuweisung zu unterbinden ist folgende:

T( const T& other )
{
  if ( this == &other )
    return;

  // ...
}

Zuweisung ist was anderes als Kopie. Eine Selbstkopie gibt es nicht, so ein Test ist überflüssig.

@sly: Die Frage ist im Allgemeinen nicht zu beantworten, da du nicht schreibst, welche Assoziation zwischen str und deiner Klasse besteht. Nehmen wir mal an, dass die Klasse str besitzt. Dann ist dein Ansatz schon fast gut, allerdings solltest du dafür sorgen, dass dein String auch in den allozierten Speicher reinpasst. Pauschal 32 ist irgendwann zu wenig und ein potentielles Sicherheitsleck.

David_pb schrieb:

Eine Idee die Selbstzuweisung zu unterbinden ist folgende:

T( const T& other )
{
  if ( this == &other )
    return;

  // ...
}

Zuweisung ist was anderes als Kopie. Eine Selbstkopie gibt es nicht, so ein Test ist überflüssig.

@sly: Die Frage ist im Allgemeinen nicht zu beantworten, da du nicht schreibst, welche Assoziation zwischen str und deiner Klasse besteht. Nehmen wir mal an, dass die Klasse str besitzt. Dann ist dein Ansatz schon fast gut, allerdings solltest du dafür sorgen, dass dein String auch in den allozierten Speicher reinpasst. Pauschal 32 ist irgendwann zu wenig und ein potentielles Sicherheitsleck.

Arg, hast natürlich recht... Hm, is das spät heute! :-))

Für Strings gibt es (wie schon gesagt wurde) zB std::string

Dann ist dein Ansatz schon fast gut, allerdings solltest du dafür sorgen, dass dein String auch in den allozierten Speicher reinpasst. Pauschal 32 ist irgendwann zu wenig und ein potentielles Sicherheitsleck.

das stimmt.

Wäre es so besser?

T(const& T other)
{
    str = new char[strlen(other.str) + 1];
    strcpy(str, t.str);
}

Und um sich den ganzen ärger zu sparen könntest du einfach std::string verwenden statt char*.

Das würde ich auch gerne, allerdings verbietet mir die Vorgabe mit STL zu arbeiten. ( -> low level Programmierung).