Discussion :

[kinta]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

et aussi de cette façon

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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!

[Invité(e)]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
Object(Object & v)
{
   this->chaine = v.chaine;
}

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

Mabu

[JolyLoic]

Non, le code suivant :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Object test = Object ("chaine");

Utiliserait bien le constructeur par recopie, mais le code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Utilise l'opérateur d'affectation.

[Invité(e)]

Mea culpa,

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

[Laurent Gomila]

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++.

[kinta]

Merci pour vos conseils