Discussion :

[hyuga33]

Bonjour,
Bon j'ai un petit problème... Je bosse sur un code en ce moment que je n'ai pas commencé mais que je dois finir.
Mais je suis coincé sur une erreur qui m'a l'air stupide... pourtant je n'ai pas trouvé la solution donc j'en appelle à la communauté!!
Je considére un objet qui ne m'appartient pas mais que je dois utiliser. Je veux le placer dans une liste de pointeurs. Mais ça ne marche pas...
Voilà grosso modo ce que je fais.
Peu importe la structure de cette classe car je veux juste "garder"(j'ai pas trouver le mot adéquat) un élément Objet1:

Code :

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

1
2
3
4
5
class Objet1{
 
...
 
}

Ensuite, dans une autre classe j'ai besoin de ma liste de pointeur sur les éléments:

Code :

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

1
2
3
4
5
6
7
8
class Objet2{
 
    list <Objet1*> *obj;
    Objet2(list<Objet1*> * obj= NULL);
    ~Objet2();
    void Ajout(Objet1 * toto) {obj->push_back(toto);};
 
}

Mais quand j'appelle la fonction Ajout, j'ai une erreur de segmentation et je bloque dessus. J'ai essayé pas mal de truc mais toujours rien.
Dés demain matin, je m'y remets .
En attendant je vous remercie de votre aide.
A bientôt.

[ram-0000]

C'est un peu normal que cela se plante:

Le membre obj de la classe Objet2 est un pointeur sur une liste d'Objet1 mais derrière ce pointeur, il n'y a rien. C'est un pointeur fou ou NULL.

donc quand tu appelles obj->Ajout(...), tu déréference un pointeur invalide et cela part en cacahuètes.

Essaye ceci:

Code :

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

1
2
3
4
5
6
7
8
class Objet2{
 
    list <Objet1*> obj;
    Objet2(list<Objet1*> * obj= NULL); // je ne sais pas trop quoi faire de ce paramètre
    ~Objet2();
    void Ajout(Objet1 * toto) {obj.push_back(toto);};
 
}

[hyuga33]

J'avais essayé comme ça mais sans la déclaration du pointeur null dans le constructeur.
Je vais réessayer, peut-être avais-je mal fait tout ça.
Par contre, je ne vois plus trop l'intêret d'initialiser un pointeur null, non!?
Etant donnée que par la suite je travaille sur obj et non sur un pointeur vers lui, a quel moment j'initialise ma list?
A la création de mon Objet2, je veux juste une liste vide. Par la suite j'y ajouterai des éléments.
J'essaie tout ça!!
Merci pour ton aide

[hyuga33]

Bon, j'ai essayé dans mon code et tout marche!!!
Par contre je trouve ça toujours bizarre du coup l'initialisation d'un pointeur...
Merci beaucoup pour ton aide ram-0000.

[ram-0000]

Par contre je trouve ça toujours bizarre du coup l'initialisation d'un pointeur...

Que trouves tu bizarre ?

[hyuga33]

Dans le constructeur, on initialise un pointeur sur l'attribut.
Je ne comprends pas en quoi ça initialise cet attribut, surtout qu'après on utilise plus de pointeurs pour y accéder.

[ram-0000]

Dans le constructeur, on initialise un pointeur sur l'attribut.

Non non.

Dans le constructeur, tu définis un paramètre d'appel qui se nomme obj, qui est un pointeur et dont la valeur par défaut est NULL (si tu utilises un constructeur sans paramètre).

Le nom de ce paramètre apporte la confusion avec l'attribut obj (je ne sais pas si le mot attribut est très C++) qui est du type list<> qui est un membre de la classe. Ces 2 objets (le paramètres et le membre de classe) sont totalement différents et le compilateur fait très bien la différence même s'ils portent le même nom de manière maladroite. De toute façon, si le compilateur a le moindre doute, il va générer un message d'insultes.

D'où l'intérêt d'avoir une politique de nommage qui permet d'éviter ces confusions.

[hyuga33]

D'accord!
Je me suis fait avoir par le nom de l'argument. Je comprends mieux maintenant pourquoi je m'en sortais pas. Pour moi, l'argument et le membre de la classe devait être identique.
Merci beaucoup pour ton aide et tes explications qui m'éviteront de retomber dans ce genre de piège.