Discussion :

[Stany]

Salut,
J'ai fait un programme visant a créer des lacets pour une recherche mathématique. le lacet est cree a partir de ligne chainee, j'ai un probleme d'allocation car dans l'un de mes constructeur Ligne(Lacet*,Ligne* pP,Ligne* pS,int). Il me donne une adresse de memoir deja allouer par cette meme fonction cette addresse est en locurence l'element precedent de la nouvelle ligne. En gros c'est ca :

Ligne* tmp;
tmp = new Ligne(&Lacet,NULL,NULL,1); // cree le premier element donne une adresse 0x35750
tmp = new Ligne(&Lacet,tmp ,NULL,1);// deusieme element, me donne une adresse 0x35750

ca fait le deusieme fois que je rencontre ce probleme.
je n'utilise pas de librairy de debugage particulaire car la derniere que j'ai utilisier generai des erreurs

+++++++++++++++++++++++++++++++++
Ligne(&pLacet,NULL,NULL,xT);
Ligne::Ligne(0x351f8, 0, 0, -2)
donne : 0x35750
Ligne::Ligne(0x351f8, 0x35750, 0, 2)
donne : 0x34ea8
Ligne::Ligne(0x351f8, 0x34ea8 ,0, -3)
donne : 0x34ea8
Ligne::Ligne(0x351f8, 0x34ea8, 0, -5)
donne : 0x35750
++++++++++++++++++++++++++++++++

je ne sais pas si ca vien de passage d'argument par valeu plustot que par reference ou qq chose d'autre.

Merci,

[Jack_serious]

Tu pourrais nous montrer le code du constructeur de Ligne ?

C'est assez etrange je trouve. Sans plus de code je vois pas...

[Stany]

mon constructeur de Ligne est

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
Ligne::Ligne(Lacet* pL,Ligne* pP,Ligne* pS,int L)
{
pOwner = pL;
pPrecedent = pP;
       if(pPrecedent!=NULL)
              pPrecedent->pSuivant = this;
pSuivant = pS;
       if(pSuivant !=NULL)
              pSuivant->pPrecedent= this;
Longueur = L;
// d'autre variable sont initaliser ici
}

Cette class support une fenetre windows.
le probleme est que ce constructeur marche bien dans certaine fonctions.
je veux bien mettre mon code source mais je suis tres mauvai a mettre des commantaire sur mon code. Enfin ce pb se passe que dans une certaine fonction. je mettre mon code se soir si ca vous dit.
Merci

[Médinoc]

Déjà un conseil: Avant d'écrire dans pPrecedent->pSuivant et Suivant->pPrecedent, vérifie qu'ils ne pointent pas déjà sur quelque chose.
Et décide d'une politique à adopter si c'est le cas:

• Gueuler systématiquement,
• OSEF,
• Faire une exception si pPrecedent et pSuivant pointent l'un sur l'autre pour s'insérer entre les deux, gueuler dans tous les autres cas,
• etc.

[Stany]

Salut, Médinoc
Vraiment dsl mais j'ai du mal a te comprendre sur
* Gueuler systématiquement,
* OSEF, // surtout ici
* Faire une exception si pPrecedent et pSuivant pointent l'un sur l'autre pour s'insérer entre les deux, gueuler dans tous les autres cas,
* etc.

et je ne vois pas pourquoi je testerai si il pointe sur qq chose car c'est le constructeur donc 'est peut-etre possible qu'il pointe sur qq chose mais la plus part du temps c'est des adresses de memoire invalide, enfin je pense, comme quand des fois la variable prend un valeur bizard a la decaration car elle occupe un espace de memoire qui n'a pas ete initiliser a une valeur.

Enfin je poste mon code Bonne chance et merci a ceux qui le regarderons.

Donc la fonction ou le constructeur plante est dans la fichier GridFunctions.cpp a la ligne 613. Prenez le temps de ragrder le prog et dit moi ce que vous pense du coding, je sais que il y a plien de chose a ameliorer.
je code sous Dev-C++ .

[Médinoc]

Je parle de tester les pointeurs des objets qu'on passe au constructeur.
En clair, je conseillais de vérifier s'ils n'étaient pas déjà attachés à d'autres objets...

Les différents choix possibles sont simples:

• Soit on lance une exception si les objets sont déjà attachés à quelque chose...
• Soit on s'en fout et on écrase,
• Soit on s'en fout mais on détache proprement l'autre objet,
• Soit on lance une exception seulement si on ne peut pas s'insérer entre les deux...