Probleme pointeur : allocation/desalocation

**SpongeBob** · 12/07/2007, 16h45

Bonjour tout le monde,
Je ne voulais vraiment pas poser de questions comme ca ici, mais comme je ne comprend vraiment pas mon erreur, vous etes en quelques sorte ma derniere chance. Voici mon code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
....
    if(nbSeg[iCam+1]>0)
      tabBin[iCam]=new LineBin[nbSeg[iCam+1]];
....
  if(tabBin[iCam]!=NULL)
 {
        cout<<"****_"<<tabBin[iCam]<<endl;
	delete(tabBin[iCam]);
	tabBin[iCam]=NULL;
  }

Les 3 petits points sont d'autres parties du code... qui ne touche pas au pointeur tabBin[iCam].
Voila donc c'est assez simple, le message d'erreur est le suivant :

*** glibc detected *** free(): invalid pointer: 0x080a464c ***

Program received signal SIGABRT, Aborted.
[Switching to Thread -1226942784 (LWP 23401)]
0xffffe410 in __kernel_vsyscall ()
(gdb) where
#0 0xffffe410 in __kernel_vsyscall ()
#1 0xb7a679a1 in raise () from /lib/tls/i686/cmov/libc.so.6
#2 0xb7a692b9 in abort () from /lib/tls/i686/cmov/libc.so.6
#3 0xb7a9b87a in __fsetlocking () from /lib/tls/i686/cmov/libc.so.6
#4 0xb7aa1fd4 in malloc_usable_size () from /lib/tls/i686/cmov/libc.so.6
#5 0xb7aa234a in free () from /lib/tls/i686/cmov/libc.so.6
#6 0xb7c438b1 in operator delete () from /usr/lib/libstdc++.so.6
#7 0x080509b8 in main (argc=2, argv=0xbfa31d54) at visual8.cpp:544

Donc, moi, je ne comprends pas en quoi le pointeur est invalide et en quoi il y a une erreur...
Merci d'avance a tout le monde.

p.s : desole pour les accents, je suis en ce moment sur un clavier americain.

**Mat007** · 12/07/2007, 16h56

Salut,

Il faut utiliser l'opérateur 'delete []' pour libérer un tableau, cf. la FAQ à ce sujet.

MAT.

**SpongeBob** · 12/07/2007, 17h08

Honte sur moi

Merci
Mais dans ce cas, pourquoi je n'ai aucun probleme avec :

Code :

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

1
2
3
4
5
6
 
...
   tabLine[iCam]=new Line*[nbSeg[iCam]];
 
....
   delete(tabLine[iCam]);

Comprends pas, merci

**roulious** · 12/07/2007, 17h13

Envoyé par SpongeBob

Honte sur moi

Merci
Mais dans ce cas, pourquoi je n'ai aucun probleme avec :

Code :

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

1
2
3
4
5
6
 
...
   tabLine[iCam]=new Line*[nbSeg[iCam]];
 
....
   delete(tabLine[iCam]);

delete appliqué sur un tableau à la place de delete[] est un comportement indéfini : le compilo peut faire n'importe quoi. Dans la pratique, j'ai observé qu'il supprimait uniquement le premier élément du tableau.

**Mat007** · 12/07/2007, 17h15

J'imagine que c'est une illustration du principe : 'ce n'est pas parce qu'il n'y a pas l'air d'y avoir de problème qu'il n'y en a pas'

MAT.

**SpongeBob** · 12/07/2007, 17h18

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
for(unsigned int i=0;i<nbSeg[iCam];i++)
{	
     if(*(tabLine[iCam]+i)!=NULL)
      {
		delete(*(tabLine[iCam]+i));
		*(tabLine[iCam]+i)=NULL;
	}
}
 if(tabLine[iCam]!=NULL)
 {
	delete(tabLine[iCam]);
	tabLine[iCam]=NULL;
 }

En l'occurence, je m'assure de bien supprimer tous les pointeurs que je cree. Donc comment savoir si je dois ici utiliser le delete [] () ou delete (). Cela reste un tableau alloue dynamiquement. D'apres la FAQ
ca ne devrait pas passer...
Comprend pas...
Merci encore.

**SpongeBob** · 12/07/2007, 17h19

Envoyé par Mat007

J'imagine que c'est une illustration du principe : 'ce n'est pas parce qu'il n'y a pas l'air d'y avoir de problème qu'il n'y en a pas'

MAT.

Oui surement, mais donc, je suis ce que me dit la FAQ et j'utilise delete [] () ou je laisse l'autre... comment puis-je verifier et comprendre la reelle difference entre ces deux zozio !

**r0d** · 12/07/2007, 17h23

Envoyé par SpongeBob

Donc comment savoir si je dois ici utiliser le delete [] () ou delete ().

Utiliser un vector (comme c'est expliqué dans la faq que tu as cité), ainsi tu n'auras plus la question à te poser et tu feras du C++. Parce que ton code ressemble plus à du C qu'à du C++.

**Mat007** · 12/07/2007, 17h31

Le problème c'est que "passer" n'est pas un critère sufisant pour que ça soit correct.

Quelques trucs pour la lisiblité déjà :
. on utilise 0 plutôt que NULL en C++ en général
. pas besoin de tester si un pointeur vaut 0 avant de le 'delete' car 'delete 0;' est valide
. pas de parenthèses à 'delete', ni à 'delete []' non plus

Après j'avoue que j'ai un peu de mal à savoir ce qu'est ton tabLine exactement... Un tableau de Line ?
Tu ne veux pas utiliser des vecteurs plutôt ? Ca évite justement d'avoir à jongler avec tout ça...

MAT.

**SpongeBob** · 12/07/2007, 17h42

Bien je voulais utiliser des Vector, mais... on m'a dit que la strucure etait plus lourde et qu'il valait mieu pour mon utilisation temps reel et la rapidite des calculs et des differentes operation que j'ai a effectuer avec mes lignes que je gere tout ca moi meme...
Apres je suis pret a prendre tous les conseils. Mais la je commence a me dire que j'aurais du faire un travail plus profond de preproduction... troisieme fois que je change de type de structure... ca commence a faire beaucoup...

**SpongeBob** · 12/07/2007, 17h45

Envoyé par r0d

Utiliser un vector (comme c'est expliqué dans la faq que tu as cité), ainsi tu n'auras plus la question à te poser et tu feras du C++. Parce que ton code ressemble plus à du C qu'à du C++.

Merci pour cette reponse seche, mais ca ne repond pas a ma question. J'aime bien comprendre ce que je fais. Et meme si dans la FAQ il ecrit qu'en C++ on utilise plus des vector que de l'allocation "moulinette" comme j'ai fait, ca ne me dit pas pourquoi ni comment.
Neanmoins, merci

**Radikal333** · 12/07/2007, 17h51

Comment as-tu déclaré tabBin?
Sinon effectivement écris plutôt delete [] tabBin; directement.

**r0d** · 12/07/2007, 18h06

Envoyé par SpongeBob

Merci pour cette reponse seche, mais ca ne repond pas a ma question. J'aime bien comprendre ce que je fais. Et meme si dans la FAQ il ecrit qu'en C++ on utilise plus des vector que de l'allocation "moulinette" comme j'ai fait, ca ne me dit pas pourquoi ni comment.
Neanmoins, merci

A ce moment là, commence par analyser le code asm produit par ton compilateur

Je suis taquin ce soir

Plus sérieusement, le sujet a été maintes fois discuté sur ce forum: le vecteur de la STL n'est pas plus lent qu'un tableau de type C.

Le code suivant compile et ne plante pas:

Code :

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

1
2
3
int* monTab;
monTab = new int[100];
delete(monTab);

Cependant, il génère une fuite mémoire des plus odieuses. Et, étant donné que ceci est interdit par la norme, personne ne peut savoir ce qu'il va réellement se passer. Et à mon avis, ça va dépendre du compilateur et du système.

**SpongeBob** · 12/07/2007, 18h10

Ah, cette reponse me convient parfaitement ! Merci beaucoup r0d.
En revanche, elle sous entend que je vais me taper un nouveau changement de structure...ce qui ne m'enchante guere

Mais ce sont les joies de la programation, on decouvre tous les jours quelques choses !
Probleme RESOLU !

**Laurent Gomila** · 12/07/2007, 18h18

Probleme RESOLU !

Alors

**r0d** · 12/07/2007, 18h19

Envoyé par SpongeBob

En revanche, elle sous entend que je vais me taper un nouveau changement de structure...ce qui ne m'enchante guere

Arf. C'est la raison pour laquelle on dit souvent que le stylo et le cahier sont les meilleurs amis du développeur

Envoyé par SpongeBob

Mais ce sont les joies de la programation, on decouvre tous les jours quelques choses !

Héé oui... surtout avec le C++. Parfois ça en devient même agaçant

**SpongeBob** · 12/07/2007, 18h26

Envoyé par Laurent Gomila

Alors

Oui c'est fait... la remarque etait-elle utile ? Moi qui croyais que vous veniez m'aider a passer de ma structure bancal au Vector

D'ailleurs ou puis-je trouver les informations (discussion) sur la rapidite entre l'utilisation de Vector et le mode manuel ?

**r0d** · 12/07/2007, 18h44

http://www.developpez.net/forums/sho...d.php?t=195318 Regardes, en particulier, le dernier post de la première page.

Un plus récent:
http://www.developpez.net/forums/sho...d.php?t=369831

Et puis, il y a une fonctionnalité "recherche" sur ce site...