Discussion :

[dingua]

Salut à tous!
En fait c'est une erreur dont je cherches la solution il y a environ une semaine.
En fait je crée un pointeur vers une instance de la classe "Chemin".

Code :

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

    Chemin * travail = depart;

depart est un attribut de la classe itineraire et qui est pointeur vers une instance de la classe "Chemin".
voici l'implémentation de la fameuse classe "Chemin":

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
    #include "Case.h"
    class Case;
    class Chemin
    {
    friend class Itineraire;
    private:
    Chemin(int ligne, int colonne):m_case(ligne,colonne){}
          Chemin(Case const & c):m_case(c){}
    Chemin(Chemin const &);
    Chemin(void):m_case(0,0){}
    Case m_case;
            Chemin* m_next;
    };

et dès que j'appelle la case "m_case" toutes les valeurs de l'instance sur laquelle on pointe changent!!
donc après cette ligne de code:

Code :

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

    travail->m_case;

tout change!
enfin voici l'implémentation de la classe case :

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
    class Case
    {
       friend class Itineraire;
    private:
 
       int ligne;
       int colonne;
 
    public:
 
 
       int get_Ligne() const;
       int get_Col()const ;
       void set_Ligne(const int);
       void set_Colonne(const int);
       Case(void):ligne(0),colonne(0){};
       Case(int,int);
       ~Case(void);
    };

Veuillez me donnez quelques explications ...

[Arzar]

Bonjour,
Si tu veux qu'on résolve rapidement le problème, il faudrait un bout de code *minimale* et *compilable* qui reproduit le problème, avec quelques printf avant/après des attributs en question, parce que pour le moment c'est assez difficile de se faire une idée de ce que tu veux dire par "un appel à la case m_case" qui ferait "changer les valeurs de l'instance sur laquelle on pointe".
Merci.

[dingua]

Voici la méthode qui accède à "m_case"

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
void Itineraire::afficher() const
{
 
 
    Chemin * travail = depart;
 
   while(travail) 
   {
 
 
 
	  cout<< "("<<travail->m_case.get_Col() ;
	   cout	   <<","<<travail->m_case.get_Ligne();
          cout<<")"<<std::endl;
 
		  travail=travail->m_next;
 
 
   }
}

et- "Itineraire.h" sera

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
#include "Grille.h"
#include "Case.h"
#include "Chemin.h"
#include <iostream>
using namespace std;
class Case;
class Itineraire
{
 
private:
 
	Chemin *depart;
	Chemin *arrivee;
 
 
 
 
 
public:
	Itineraire(void);
	Itineraire(Chemin* depart);
	~Itineraire(void);
	void add(Case const&);
	void afficher() const;
};

[3DArchi]

Salut,
Ton erreur me fait penser à un problème de pointeur. As-tu vérifier que la création de ta liste se passe bien ? Comment est initialisé m_next ?

[dingua]

En fait lorsque j'initialise m_next à NULL il m'affiche une erreur"identificateur non déclaré"(mais parfois il exécute sans rien afficher et dès qu'on fait une autre erreur il revient à cette erreur d'"identificateur non déclaré")

[Mohamed_DEV]

Je suis le seul à penser que c'est anormal de donner le nom case à une classe puisque c'est un mot réservé?.

[Goten]

Oui, car le mot clef réservé est case et non Case... (la casse compte, y'a que le parser du forum qui a pas l'air d'en tenir compte).
Même si je te l'accorde ça peut prêter à confusion..

[yan]

une structure qui change toute seule comme cela, a ma connaissance cela ne peut être qu'une erreur mémoire et justement, comme tu ne donne pas de valeur par défaut à m_next de ta classe Chemin, ce pointeur peut valoir n'importe quoi .
Et quand tu fait ta boucle sur les chemins, tu tombe sur cette valeur non initialisé de pointeur et lors de tes appels, tu produits une erreur mémoire et rend l'état de ton programme indéterminé.

Pour ton problème de compilation sur la mise à 0 de m_next, il faudrait voir ton code, pour te dire ce qui ne va pas.

[dingua]

une structure qui change toute seule comme cela, a ma connaissance cela ne peut être qu'une erreur mémoire et justement, comme tu ne donne pas de valeur par défaut à m_next de ta classe Chemin, ce pointeur peut valoir n'importe quoi .
Et quand tu fait ta boucle sur les chemins, tu tombe sur cette valeur non initialisé de pointeur et lors de tes appels, tu produits une erreur mémoire et rend l'état de ton programme indéterminé.

Pour ton problème de compilation sur la mise à 0 de m_next, il faudrait voir ton code, pour te dire ce qui ne va pas.

En fait comme je l'ai dit le compilateur ne m'a pas laissé l'occasion d'initialiser m_next à NULL et en faite dès que j'entre dans la boucle "while(travail){...}"
et après la ligne ci-dessous tout change (donc on n'a pas encore accédé à m_next!)
cout<< "("<<travail->m_case.get_Col() ;
et à propos du code je pense que j'en ai donné suffisamment ,presque toutes les classes qui ont un lien avec l'erreur produite sont en dessus.

[3DArchi]

Bonjour,

et à propos du code je pense que j'en ai donné suffisamment ,presque toutes les classes qui ont un lien avec l'erreur produite sont en dessus.

Non, il manque toute la construction et l'initialisation de ta liste. Comme dit dans mon premier message et comme dit par Yan, je penche pour un problème de pointeur mal initialisé.

[yan]

le compilateur ne m'a pas laissé l'occasion d'initialiser m_next à NULL

Pour être puriste c'est 0 et non NULL en c++
Sinon, ce n'est pas normale , l'initialisation d'un pointeur à 0 (ou NULL) ne doit pas poser de problème.
Il faudrait que tu montre un peu plus de code. Dans ce que tu as montré, à par la non initialisation de m_next, il ne semble pas qu'il y est d'autre problème.

Si tu peut utiliser boost, Qt ou autre (y as surement des lib moins lourde, peut être loki), ces lib fournissent des pointeurs intelligent bien plus agréable et moins case gueule à utiliser :
http://loic-joly.developpez.com/tuto...mart-pointers/
http://matthieu-brucher.developpez.c...ost/smartptrs/

[dingua]

J'ai utilisé 0 à la place de "NULL" ;alors le compilateur l'a accepté mais on se trouve dans le même cas; je peux vous donnez un autre point qui peut aider,c'est que dès que je construis une Case dans la méthdoe Itineraire::afficher() toutes les valeurs des instances pointés par "départ"change.
Tout ça se déroule sans que l'adresse du pointeur change!

[yan]

Surement que soit depart soit le pointeur sur Itineraire (si tu utilise un pointeur) pointe sur n'importe quoi .

[dingua]

Surement que soit depart soit le pointeur sur Itineraire (si tu utilise un pointeur) pointe sur n'importe quoi .

je ne suis pas du tout convaincu par ce que tu dis yan,car en "déboguant" je découvre qu'avant l'appel de "m_case" tout va bien mais juste après toutes les valeurs changent sans que les adresses des pointeurs le font!

[yan]

je ne suis pas du tout convaincu par ce que tu dis yan,car en "déboguant" je découvre qu'avant l'appel de "m_case" tout va bien mais juste après toutes les valeurs changent sans que les adresses des pointeurs le font!

Ce genre de bug j'en ai fait et j'en ai corrigé pas mal
Mais rien ne t'empêche de vérifier la valeur de "depart" et, si tu utilise un pointeur, celui de l'instance Itineraire.

Peut tu montrer comment tu initialise tes pointeurs?

[dingua]

voici le constructeur de l'instance Itineraire que j'utilise:

Code :

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

1
2
3
4
Itineraire::Itineraire(Chemin * d):depart(d),arrivee(NULL)
{
 
}

et la déclaration du pointeur vers l'instance itineraire est:

Code :

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

Itineraire* itineraire=new Itineraire( &Chemin(a));

[yan]

Code :

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

Itineraire( &Chemin(a));

en voila un jolie bug. Tu passe en paramètre un pointeur sur une variable temporaire. Une fois qu'elle est détruite ton pointeur est invalide

[dingua]

Merci, c'était vraiment ça!