un bug introuvable

Voila
ce code se compile correctement, mais à la fin l'invite de commande gèle:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70

#include<iostream> #include<stdio.h> #include<stdlib.h> #include<string.h> #include<strings.h>       using namespace std;   class Personne { private: char * nom; char * pnom; int age; public: Personne(); Personne(char * n, char * p, int a); ~Personne(); //void Modif(Personne& p); void Affich(); };   Personne::Personne() { nom = new char[6]; nom = "userN"; pnom = new char[5]; pnom = "userP"; age = 10; }   Personne::Personne(char * n, char * p, int a) { nom = new char[strlen(n)]; strcpy(nom, n);   pnom = new char[strlen(p)]; strcpy(pnom, p); age = a;   }   Personne::~Personne() { delete[] nom; delete[] pnom; } void Personne::Affich() { cout << "\nNom: " << nom << endl; cout << "Prenom: " << pnom << endl; cout << "Age: " << age << endl; } int main() { char *uname; uname = new char[12]; cin >> uname;   Personne p(uname, "toto", 24); Personne a;   a.Affich(); p.Affich();   delete[] uname; getchar(); }

---

d'où ça peut venir?

merci

Lae,

Sans avoir tout lu:

Code:

---

1 2 3 4 5 6 7 8 9 10

Personne::Personne(char * n, char * p, int a) { nom = new char[strlen(n)]; strcpy(nom, n);   pnom = new char[strlen(p)]; strcpy(pnom, p); age = a;   }

---

il faut réserver strlen(n) +1 pour la nouvelle chaîne (ne jamais oublier le 0 final :)).

d'où l'utilité d'utiliser des std::string !

Gao,

Citation:

---

Envoyé par Davidbrcz

d'où l'utilité d'utiliser des std::string !

---

Évidemment :)

Salut,

pour moi ça marche mais ça doit être un coup de chance.

En effet, ici:

Code:

---

1 2 3 4 5

nom = new char[6]; nom = "userN"; pnom = new char[5]; pnom = "userP"; age = 10;

---

Apparemment tu n'as pas tout saisi, désolé.
Pour copier une chaine dans une autre, ça ferait:

Code:

---

1 2 3 4 5

nom = new char[6]; strcpy(nom, "userN"); pnom = new char[6]; strcpy(pnom, "userP"); age = 10;

---

Ce que tu faisais, c'est que tu mettais l'équivalent d'un const char * dans ton pointeur, et donc ton pointeur pointe vers une autre zone maintenant, celle où il y a écrit "userN". Du coup plus rien ne pointe vers la mémoire allouée dynamiquement, et quand tu fais un delete [] sur un const char * ça a de grandes chances de planter...

Je ne suis peut-être pas très clair, mais regarde dans ton bouquin les const char *, et la recopie d'une chaîne dans une autre, il expliquera ça mieux que moi.

Sinon les autres remarques sont valides, (strlen+1 au lieu de strlen), et il manque un 'return 0' à la fin de ton main.

Une fois tout ça fait, tout va bien:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69

#include<iostream> #include<string.h>   using namespace std;   class Personne { private: char * nom; char * pnom; int age; public: Personne(); Personne(char * n, char * p, int a); ~Personne(); //void Modif(Personne& p); void Affich(); };   Personne::Personne() { nom = new char[6]; strcpy(nom, "userN"); pnom = new char[6]; strcpy(pnom, "userP"); age = 10; }   Personne::Personne(char * n, char * p, int a) { nom = new char[strlen(n)+1]; strcpy(nom, n);   pnom = new char[strlen(p)+1]; strcpy(pnom, p); age = a;   }   Personne::~Personne() { delete[] nom; delete[] pnom; } void Personne::Affich() { cout << "\nNom: " << nom << endl; cout << "Prenom: " << pnom << endl; cout << "Age: " << age << endl; } int main() { cout << "Entrez votre nom: ";   char *uname; uname = new char[12]; cin >> uname;   Personne p(uname, "toto", 24); Personne a;   a.Affich(); p.Affich();   delete[] uname; getchar();   return 0; }

---

Meo,

Citation:

---

Envoyé par coyotte507

pour moi ça marche mais ça doit être un coup de chance.

---

Moi, j'appelle ça de la malchance, car c'est un programme buggé, au comportement indéfini, et qui semble fonctionner normalement, alors que... :)

Citation:

---

Envoyé par coyotte507

Code:

---

1 2 3 4 5

nom = new char[6]; nom = "userN"; pnom = new char[5]; pnom = "userP"; age = 10;

---

Apparemment tu n'as pas tout saisi, désolé.
Pour copier une chaine dans une autre, ça ferait:

Code:

---

1 2 3 4 5

nom = new char[6]; strcpy(nom, "userN"); pnom = new char[6]; strcpy(pnom, "userP"); age = 10;

---

---

Ah oui, j'avais confondu la syntaxe de char avec celle de string..
Quoi qu'il arrive merci à vous tous! ;-)