Une question sympa

Bonsoir,

Alors voila je vous expose mon problème qui est résolu mais dont j'aimerai comprendre quel en est le mécanisme.

Admettons un Objet:

Code:

---

1 2 3 4 5 6 7 8 9

class Object { private : char *chaine;   public: Object (char *c); // Je réalise une copie(mémoire) de c dans chaine ... Je gère le \0 char *get () const; // Je récupère chaine via un pointeur }

---

j'ai essayer de l'expoiter de deux manières :

Code:

---

1 2	Object test; test = Object ("chaine");

---

et aussi de cette façon

Code:

---

1 2	Object *test; test = new Object ("chaine");

---

Le problème: lorsque j'exploite la variable chaine avec la première manière, j'obtiens des problème de contenu (rien ou n'importe quoi).
Alors qu'en exploitant aprés le deuxième cas, tout se passe bien.

Pourquoi ?
Le problème viendrait-il d'autre part !?
Si besoin je peux mettre le code en détails, mais j'ai essayer d'aller à l'essentiel!

Bonjour
Dans le premier cas, le compilo fait appel au constructeur de recopie, qui copie les données membres sans se poser de questions dessus : il copie le pointeur mais pas son contenu.
Pour résoudre le problème, il suffit d'ajouter la méthode suivante à ton objet :

Code:

---

1 2 3 4 5 6

Object(Object & v) { /* verifs */ this->chaine = malloc(strlen(v.chaine)); strcpy(this->chaine, v.chaine); }

---

pour info, le constructeur de recopie par défaut devrait ressembler à

Code:

---

1 2 3 4

Object(Object & v) { this->chaine = v.chaine; }

---

Voilà, j'espère avoir été compréhensible

Mabu

Non, le code suivant :

Code:

---

Object test = Object ("chaine");

---

Utiliserait bien le constructeur par recopie, mais le code :

Code:

---

1 2 3

Object test; test = Object ("chaine");

---

Utilise l'opérateur d'affectation.

Mea culpa,

effectivement, je devrai attendre d'etre bien réveillé avant de causer
:oops:

En gros, si tu veux que ta classe soit correcte, il faut lui définir correctement le constructeur par défaut, le destructeur, le constructeur par copie, et l'opérateur d'affectation.

Donc, pour éviter de te prendre la tête avec ça et d'avoir une classe buggée, utilise plutôt std::string pour gérer les chaînes. Voir la FAQ C++.

Merci pour vos conseils