Discussion :

[pouic06]

Bonjour ... je voudrais savoir pourquoi ce code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
int main () {
  cout << "On cree les personnes suivantes\n";
  const char* prenoms1[] = {"Marcel", "Firmin", "Gaston", 0};
  const Personne marcel ("Meharcele",  prenoms1, 78);
  Afficher (marcel);
 
  const char* prenoms2[] = {"Lea", "Germaine", 0};
  const Personne lea ("Tralala",  prenoms2, 45);
  Afficher (lea);
 
  // et si vous aviez raté quelque chose sur le constructeur
  prenoms1[0] = "Claudia"; prenoms1[1] = 0;
  Personne claudia ("Choufleur",  prenoms1, 31);
  Afficher (claudia);

l'entete de la classe Personne :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
class Personne {
 
public :
Personne(const char* ,const char** ,int );
const char* Nom() const;
const char* PremierPrenom() const ;
const char** TousLesPrenoms() const ;
int Age() const ;
 
private :
const char* nom ;
const char** prenoms;
int age ;
 
};

avec ce constructeur dans la classe personne :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
Personne::Personne(char* leNom , const char** lesPrenoms , int lage) {
 
 
nom = strdup(leNom) ;
 
int i=0;
while(lesPrenoms[i] != 0) { i++;}
 
int j=0;
while (j != i) { 
prenoms[j] = strdup(lesPrenoms[j]) ;;
j++;
}
prenoms[j]=0;
 
age = lage ;
 
}

provoque cette erreur :

Code :

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

1
2
3
4
5
 
On cree les personnes suivantes
 Firmin (Firmin, Gaston) Meharcele, 78 ans
 Germaine (Germaine) Tralala, 45 ans
Erreur de segmentation (core dumped)

Merci d'avance

[MatRem]

Déjà le nom de l'erreur c'est "Erreur de segmentation", Core dump ça doit vouloir dire, mais je ne suis pas sur, qu'un fichier core (fichier d'erreur) a été écrit.

Ensuite les erreurs de segmentation apparraissent trés souvent quand on commence à s'amuser avec les pointeurs: donc essaye de les éviter quand c'est possible.

Tu peux les éviter en utilisant:
- string à la place de char*
- vector à la place des tableaux de base du langage

Mais bon tout ça a déjà été dit des centaines de fois sur ce forum .

[coca25]

Code :

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

1
2
3
4
5
6
 
while (j != i) { 
prenoms[j] = strdup(lesPrenoms[j]) ;;
j++;
}
prenoms[j]=0;

tu n'alloue nulle part de la mémoire pour prenoms.
et je crois sauf erreur de ma part qu'il y a des const en trop

[pouic06]

tu n'alloue nulle part de la mémoire pour prenoms.
et je crois sauf erreur de ma part qu'il y a des const en trop

Je crois que strdup alloue lui meme la memoire pour la chaine de caracteres .

Tu peux les éviter en utilisant:
- string à la place de char*
- vector à la place des tableaux de base du langage

Non je peux pas , c'est dans le cadre d'un TP et le code dans le main je peux pas le changer

Déjà le nom de l'erreur c'est "Erreur de segmentation", Core dump ça doit vouloir dire, mais je ne suis pas sur, qu'un fichier core (fichier d'erreur) a été écrit.

Il se trouve où ce fichier ?

[David Fleury]

Coca25 parlait des "cases" pour prenoms (ton tableau) pas les chaines allouées par strdup.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
Personne::Personne(char* leNom , const char** lesPrenoms , int lage) {
nom = strdup(leNom) ;
 
int i=0;
while(lesPrenoms[i] != 0) { i++;}
 
int j=0;
prenoms = new char*[i+1]; // quelque chose comme ça à ajouter
while (j != i) { 
prenoms[j] = strdup(lesPrenoms[j]) ;;
j++;
}
prenoms[j]=0;
 
age = lage ;
 
}

Sinon, si tu ne veux pas utiliser vector,
tu pourras toujours corriger les bugs ( pas de destructeur, constructeur de copie et affectation manquants)

Si le main ne peut pas changer, change ta classe

[pouic06]

oui j'avais pensé à mettre ce

Code :

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

 prenom = new char*[i+1];

mais le compilateur me dit

Code :

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

1
2
3
4
 
personne.cc: In constructor ‘Personne::Personne(const char*, const char**, int)’:
personne.cc:13: error: ‘prenom’ was not declared in this scope
make: *** [personne.o] Erreur 1