Discussion :

[miketidy]

Bonjour,

J'ai un petit soucis et je suis nul en c++. Voilà j'ai compilé via un makefile mes sources mais quand j'exécute mon exe il me met Memery fault alors j'ai été voir dans le fichier coré et voici ce qu'il me met :

Code :

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

1
2
3
Program terminated with signal 11, Segmentation fault.
#0  0x08050ee0 in CRequeteListe::Supprimer (this=0x8154600, pCr=0xfeff748c) at /devl/sap1/unix/src/pssess.pc:108
108         for( ; pCrl->Suivant; pCrl=pCrl->Suivant)

L'erreur vient de cette fonction :

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
/* Suppression d'une requete dans la chaine */
CRequeteListe *CRequeteListe::Supprimer( CRequeteSql *pCr)
{
    CRequeteListe *pCrl= this;
    CRequeteListe *pSuivant;
    if( pCrl->Element==pCr)
        {
        pSuivant = Suivant ;
        Suivant = NULL; 
        delete pCrl;
        return pSuivant;
        }   
    for( ; pCrl->Suivant; pCrl=pCrl->Suivant)
        if( pCrl->Suivant->Element == pCr)
            {
            pSuivant = pCrl->Suivant->Suivant; 
            pCrl->Suivant->Suivant = NULL; 
            delete pCrl->Suivant;
            pCrl->Suivant = pSuivant;
            }   
    return this;
}

Quelqu'un aurait une idée pourquoi çà tombe en erreur

Merci d'avance

[oodini]

Ça me paraît louche, l'instruction

Code :

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

delete pCrl;

Tu supprimes l'instance dans laquelle tu te trouves ?

[miketidy]

Je ne sais pas trop, le programme n'a pas été fait par moi. Je recompile juste l'exe parce q'il y a eu migration de la version d'Oracle.

Je ne sais pa si c'est çà qui plante mais l'erreur indiqué par le fichier core se trouve sur la boucle.

Je suis un peu perdu avec tout ces pointeur...

[3DArchi]

A ton avis, que ce passe-t-il lorsque l'élément recherché est le dernier de ta liste chaînée?

[miketidy]

heu bonne question

Je ne sais pas trop, j'arrive souvent à comprendre une fonction mais là c'est un peu flou, je ne vois pas trop, je suppose qu'il y a un soucis si c'est le dernier element de la liste qui recherché, au niveau de la boucle surment, quand il arrive au dernier element il plante car li ne sait pas quelle est l'element suivant.

Je suis sur la bonne voie ou pas du tout, je comprend pas grand chose à cette fonction...

[3DArchi]

Effectivement, c'est la bonne voie.
Dans la boucle for, l'incrément est fait avant le test (sauf, bien sûr en entrée de boucle).
Donc, si on décortique:

Code :

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

1
2
3
4
5
6
7
8
   for( ; pCrl->Suivant; pCrl=pCrl->Suivant)
        if( pCrl->Suivant->Element == pCr)
            {
            pSuivant = pCrl->Suivant->Suivant; 
            pCrl->Suivant->Suivant = NULL; 
            delete pCrl->Suivant;
            pCrl->Suivant = pSuivant;
            }

Supposons que le dernier élément est celui recherché.
Donc pCrl pointe sur l'avant dernier élément.

Code :

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

if( pCrl->Suivant->Element == pCrl)

est vérifié donc on rentre:

Code :

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

pSuivant = pCrl->Suivant->Suivant;

Là tu récupères le suivant du suivant, donc le suivant du dernier, donc il y a de forte chance que ce soit NULL.
Les deux lignes suivantes déchaînent l'élément et le suppriment.

Code :

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

pCrl->Suivant = pSuivant;

l'élément courant (pCrl) est renchaîné avec NULL.
Retour dans ta boucle:

Code :

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

pCrl=pCrl->Suivant

. Donc pCrl vaut NULL.
Test:

Code :

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

pCrl->Suivant

Déréférencement de NULL, et là boum

Tu auras aussi un problème si deux éléments identiques se suivent. Seul le premier sera déchaîné.

En fait, la boucle est bancale car lorsqu'elle trouve un élément à déchaîner, elle avance d'un pas ce qui fait que l'élément suivant celui qui est déchaîné n'est pas pris en compte.

Autre chose me parait étrange: si l'élément recherché est le premier, on le supprime et on retourne l'élément suivant. Si par contre l'élément n'est pas le premier, alors on parcours toute la liste pour essayer d'enlever toutes les occurrences de l'éléments recherchés. Quelque chose cloche. Soit la boucle est faite pour enlever le premier élément correspondant à celui fourni en argument, soit pour enlever toutes ses occurrences.