Discussion :

[hunter99]

slt j'ai ecrit 2 classe voiture et personne pour utiliser la classe voiture dans la classe personne, mais lors de l'execution ça n'a pas marché

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
20
21
22
23
24
25
26
27
   class voiture
{
    int numero;
    char* modele;
 
public:
    voiture();
    voiture(int,char*);
    voiture (const voiture&);
    ~voiture();
    voiture& operator=(const voiture&);
 
    char* get_name();
 
};
class personne
{
    char * nom;
 
    voiture voiture_perso;
    public:
 
    personne();
    personne (char*,voiture);
 
 
};

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
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
//.cpp
voiture::voiture()
{
    modele=new char(10);
 
}
voiture::voiture(int n,char* name):numero(n)
{
    modele=new char[strlen(name)+1];
    strcpy(modele,name);
 
}
 
voiture::voiture(const  voiture &orgnl)
{
    numero=orgnl.numero;
    strcpy(modele,orgnl.modele);
 
}
 
voiture::~voiture()
{
    delete  modele;
}
char * voiture::get_name()
{
    return modele;
}
personne::personne()
{
    nom=new char[10];
}
personne::personne(char* nm,voiture car):voiture_perso(car)
{
    cout<<"passe\n";
 
 
    nom=new char[strlen(nm)+1];
 
    strcpy(nom,nm);
}

Code :

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

1
2
3
4
5
6
7
int main()
{
    voiture clio(2,"clio1");
    personne p1("toto",clio); // ça plante ici avt le lancement du constructeur
 
    return 0;
}

Si quelqu'un peut m'aider je lui en serais reconnaissant .
Merci.
ps:je suis obligé d'utiliser le char * (pas de string).

[coyotte507]

C'est sûr que sans le code des fonctions on va pouvoir t'aider

[hunter99]

voila c'est reglé .

[Médinoc]

Le constructeur de copie est incorrect: Il n'alloue pas de mémoire pour le strcpy().
De plus, je pense qu'une personne ne devrait pas absorber ainsi la voiture, mais seulement garder un pointeur (ou une référence, si on considère que toute personne possède une et une seule voiture, mais j'en doute) vers elle.

[coyotte507]

tiens voilà

Code :

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

1
2
3
4
5
6
voiture::voiture(const  voiture &orgnl)
{
    numero=orgnl.numero;
    modele = new char[strlen(orgnl.modele)+1];
    strcpy(modele,orgnl.modele);
}

[yan]

Bonjour,
tu devrait utiliser la std::string

[Médinoc]

Mongaulois :

ps:je suis obligé d'utiliser le char * (pas de string).

Un exercice de toute évidence, mais ça ne t'autorise pas à ne pas lire son post...

[yan]

Un exercice de toute évidence, mais ça ne t'autorise pas à ne pas lire son post...

Je ne l'avais pas vu.
Ce n'est pas pour ça que je n'ai pas lu son poste.

[Médinoc]

Je sais, j'ai volontairement exagéré

[hunter99]

Dire que lorsque j'ai testé la classe voiture tte seule ça a fonctionné !
comment est ce que c'est possible ?

[coyotte507]

parceque tu n'as pas testé:

Code :

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

1
2
Voiture v1(2, "clio");
Voiture v2(v1);

Ca t'aurait fait buggé aussi.

[yan]

Je sais, j'ai volontairement exagéré

ok...

Dire que lorsque j'ai testé la classe voiture tte seule ça a fonctionné !
comment est ce que c'est possible ?

Par chance
comme tu n'initialisais pas ton pointeur, ça valeur devait correspondre a une zone mémoire non protégé. Donc pas de plantage mais grosse erreur memoire. Un redémarrage, quelques lignes de code ou un café plus tard et plus rien ne marchera. C'est assez aléatoire.
tu ne compile jamais en debug? généralement (mais pas obligatoire, enfin je ne sait pas trop) ce mode initialise les variables à 0 et tu aurai donc eu une erreur de pointeur NULL.

[hunter99]

et je compile comment en debug sous code::blocks ?

[yan]

et je compile comment en debug sous code::blocks ?

normalement, quand tu créé un projet avec, il créé deux compilation. debug et release.
Mais en y réfléchissant, je ne sait pas si GCC fait la mise a 0 des variables... Normalement visual oui

[EDIT]
j'ai essayer, GCC ne le fait apparemment pas.

[hunter99]

slt quel la difference entre delete et delete [] ?J'ai chercher sur le net mais j'ai trouvé bcp d'explications qui m'ont embrouillé.
Merci d'avance.

[yan]

slt quel la difference entre delete et delete [] ?J'ai chercher sur le net mais j'ai trouvé bcp d'explications qui m'ont embrouillé.
Merci d'avance.

delete détruit l'objet alloué par new ___
delete [] détruit l'ensemble alloué par new ___ []

Pour l'histoire des initialisations a zero, n'écoute pas ce que j'ai dit tout a l'heure . Normalement certain compilot le fait... Par exemple,avec GCC sous ubuntu 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
16
17
18
19
20
21
#include <iostream>
 
using namespace std;
 
class aze
{
    public :
    aze(){};
    int * pointeur;
};
 
 
int main()
{
    aze a;
    aze * b = new aze;
 
    cout << "a : " << a.pointeur<<endl;
    cout << "b : " << b->pointeur<<endl;
    return 0;
}

initialise a 0 le pointeur de aze alloué et non l'autre.. Mais c'est peut être un coup de chance. Si quelqu'un sait, ca m'intéresse, ça m'évitera de dire des conneries....

[hunter99]

Meric c'est simple et clair

[yan]

petit lien sur la faq :
http://cpp.developpez.com/faq/cpp/?page=pointeurs