Discussion :

[Invité]

Hello tout le monde,

voilà, j'ai un bête problème de pointeur dans mon code qui m'amène à une erreur de type
*** glibc detected *** ./percolation: double free or corruption (!prev): 0x09525138 ***

Voici les bribes utiles de mon 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
22
23
24
25
int *lat; //<- variable déclarée globalement
 
percolation::percolation(int seed, float proba, int N)
{
...
lat = new int[squareN]; //a l'initialisation de la classe, j'alloue la mémoire à ma variable
...
}
 
//A un moment, j'appelle la fonction suivante
int percolation::addToCluster(int i,int mainCluster,int * M,int k)
{
...
    lat[i] = lat[mainCluster]; //<- selon mes tests, c'est précisément à cause de cette ligne que l'erreur apparait.
 
    M[lat[i]]++;
 
    return k;
}
 
//naturellement, le destructeur de mon objet libère la mémoire
percolation::~percolation()
{
    delete [] lat;
}

Concernant la ligne

Code :

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

lat[i] = lat[mainCluster];

Si je la mets en commentaire, je n'ai pas l'erreur. Si je la remplace par quelque chose du style

Code :

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

 lat[i] = 15;

Je n'ai pas l'erreur.

J'appelle aussi la fonction addToCluster a un autre moment du code mais ca ne pose pas de bugg non plus...

Voila, ca fait un moment que je tourne en rond sur google, sans trouvé la solution à mon problème.

Si quelqu'un à une idée, je serais vraiment content.

Salutations.

[3DArchi]

Salut,
L'utilisation de std::vector permettrait déjà d'écarter le double delete. std::vector::at te permettrait d'avoir une exception si les indices ne sont pas bons et pourrait t'aider en recherche de panne.
Sinon, ben la même chose à la main : fait un assert sur tes indices là où tu adresses ton tableau pour voir si tu n'as pas un problème de dépassement d'indices.

[Davidbrcz]

Les variables globales, surtout des pointeurs, c'est le mal : conception à revoir ? Sans doute.

Sinon as tu un constructeur de copie et un opérateur d'affectation ?

Après +100 avec 3DArchi.

[poukill]

Les variables globales, surtout des pointeurs, c'est le mal : conception à revoir ? Sans doute.

+1.
Surtout que tu initialises ta mémoire dans ton constructeur. Or le rôle du constructeur c'est d'initialiser les variables membres. Je rejoins david : pourquoi une variable globale et pas une variable membre de ta classe ?
Idem pour le destructeur, tu détruis une variable qui ne t'appartient pas... Elle pourrait être partagée, attention !

Une fois ces problèmes de conception résolus, vérifie tes indices !

[bruno_pages]

chrisburrc : exécuter votre appli via valgrind et il vous dira où sont vos problèmes

[Invité]

Merci pour vos messages.

En fait, je me suis mal exprimé. J'ai dit que ma variable était déclarée globalement, mais en fait elle est un attribut de ma classe. Il me manque un peu de jargon... désolé.

Ensuite, selon les conseils de 3DArchi, j'ai remplacé tous mes pointeurs par des vecteurs. Très puissant, je connaissait pas. Merci.

Malgrès ça, mon problème persiste. Mais de manière très étrange. En fait ma variable lat prends la taille du carré d'une variable donnée :

Code :

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

1
2
squareN = N * N;
lat.assign(squareN,0);

Et j'ai obtenu mon erreur uniquement pour une valeur de N=5. J'ai essayé N=4, 10, 100, 1000 et pas d'erreur qui fasse... Donc je baisse les bras, j'ai pas besoin de N=5 et pas le temps non plus.

Sinon as tu un constructeur de copie et un opérateur d'affectation ?
Après +100 avec 3DArchi.

Ca veut dire quoi cette combine ?

chrisburrc : exécuter votre appli via valgrind et il vous dira où sont vos problèmes

Merci du conseil. pas le temps ce soir, mais j'essayerais à l'occase

[3DArchi]

Malgrès ça, mon problème persiste. Mais de manière très étrange. En fait ma variable lat prends la taille du carré d'une variable donnée :

Code :

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

1
2
squareN = N * N;
lat.assign(squareN,0);

Et j'ai obtenu mon erreur uniquement pour une valeur de N=5. J'ai essayé N=4, 10, 100, 1000 et pas d'erreur qui fasse... Donc je baisse les bras, j'ai pas besoin de N=5 et pas le temps non plus.

Il faut aussi remplacer les lat[index] par lat.at(index). Cette seconde version génère une exception si l'index est invalide.

Ca veut dire quoi cette combine ?

F.A.Q : Quand dois-je définir un constructeur par copie ?, Quand dois-je définir l'opérateur d'affectation ?, Qu'est-ce que la forme canonique orthodoxe de Coplien ?, Comment gérer proprement des allocations / désallocations de ressources ? Le RAII !.
Mais l'utilisation des std::vector le fait pour toi.