C++

[C++] Pb de boucle for récalcitrante

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [C++] Pb de boucle for récalcitrante

skeye

Bon, là je crois que j'ai besoin d'un oeil extérieur.
Je suis en train de coder un bout de prog de traitement d'image, en l'occurence un filtre médian.
Le problème est que quelle que soit l'image sur laquelle je travaille, le code ci-dessous s'obstine à sortir un peu vite de la boucle:

Code :

CImage *
CImage::mediane()
{
int * tab=(int *) malloc (this->getSize() * sizeof(int));
int tmp[9];
cout<<"longueur="<<this->getLength()<<" hauteur="<<this->getHeight()<<endl;
for(int i = 1; i < this->getLength() - 1; i++)
{
cout<<"i="<<i<<endl;
for(int j = 1; j < this->getHeight() - 1; j++)
{
tmp[0] = this->getVal(i-1, j-1);
tmp[1] = this->getVal(i-1, j);
tmp[2] = this->getVal(i-1, j+1);
tmp[3] = this->getVal(i, j-1);
tmp[4] = this->getVal(i, j);
tmp[5] = this->getVal(i, j+1);
tmp[6] = this->getVal(i+1, j-1);
tmp[7] = this->getVal(i+1, j);
tmp[8] = this->getVal(i+1, j+1);
//tri du tableau
for(int k=0; k<8; k++)
for(l=k+1; l<9; l++)
if(tmp[k]>tmp[l])
{
int t=tmp[k];
tmp[k]=tmp[l];
tmp[l]=t;
}
//On met la mediane dans le tableau temporaire
tab[i + j * this->getLength()] = tmp[4];
}
}
cout<<"Ole!"<<endl;
//On met a jour l'image
for(int i = 1; i < this->getLength() - 1; i++)
for(int j = 1; j < this->getHeight() - 1; j++)
this->setVal(i,j,tab[i+j*this->getLength()]);
delete tab;
this->writePGM("mediane.pgm" );
return this;
}

Voici l'affichage que j'obtiens à l'exécution:

longueur=2470 largeur=3503

i=1
i=2
i=3
i=4
i=5
i=6
i=7
Ole!

Pour toutes les images i ne dépasse pas 7...et j'avoue que je ne comprends rien! :??:

PS: cette appli n'est là que pour quelques tests, commenter la rigueur/propreté du code n'est donc pas nécéssaire, merci...

Publicité

chrisbk

Ben rien vu de louche non plus (?) tu peux ptet essayer pas a pas au debug ?
et je dois dire que les ma...... non rien

skeye

chrisbk a écrit :

Ben rien vu de louche non plus (?) tu peux ptet essayer pas a pas au debug ?
et je dois dire que les ma...... non rien

...lloc ne doivent pas être castés? :whistle:
Aucune idée de ce que je peux utiliser comme débugger là, je code sous windows98, compile avec djgpp...ché pas si gdb est inclus, v regarder!

Taz

bisounours-codeur

assure toi que this->getLength() ne varie pas

et puis ton tri du ableau l'es tres pas tres tres beau, mais bon, y a que 9 elements

et pourquoi un malloc, t'aime pas les new, pourtant tu fais un joli delete apres? y a un truc qui est pas clair pour toi je crois. C -> malloc/free, C++ new/delete new[]/delete[]. tu fais du C avec classes j'ai bien l'impression

chrisbk

skeye a écrit :

...lloc ne doivent pas être castés? :whistle:

Nan
new/delete
malloc/free

(pis que tu fais du malloc en C++ ?skan meme plus pratique le new )

skeye

++Taz a écrit :

mauvais réflexes...dsl le ferai plus! :ange:

this->getLength() ne fait que renvoyer la valeur d'une variable que je ne modifie que lors de la lecture de mon image...

Citation :

cette appli n'est là que pour quelques tests, commenter la rigueur/propreté du code n'est donc pas nécéssaire, merci...

:ange:

Deaddy

affiche this->getSize()
assure toi que tab n'est pas null

aussi affiche this->getLength() dans ta boucle for i (pour controle)

skeye

chrisbk a écrit :

Nan
new/delete
malloc/free

(pis que tu fais du malloc en C++ ?skan meme plus pratique le new )

rahhhhhhhh v aller me flageller avec des orties... :cry:
J'ai jamais utilisé les new que sur des classes que j'avais définies moi-même...c'est un truc de ce genre que je dois mettre à la place avant qu'un autre ne me fasse la remarque? :

Code :

int * tab = new int[this->getSize()];

skeye

Plus la peine de répondre, j'ai trouvé...
Comme un gros boulet que je suis, j'ai oublié le int
dans mon

Code :

for(int l=0; l<9; l++)

Et l est aussi le nom de la variable membre designant la longueur...
Merci quand même à tous!

Message édité par skeye le 28-04-2003 à 16:41:27

chrisbk

skeye a écrit :

Plus la peine de répondre, j'ai trouvé...
Comme un gros boulet que je suis, j'ai oublié le

Code :

dans mon for(l=0; l<9; l++)
Et l est aussi le nom de la variable membre designant la longueur...
Merci quand même à tous!

jeune homme, vous aurez appris de cette experience l'interet de differencier variable membre de variable locale par un quelconque moyen syntaxique

Publicité

Taz

bisounours-codeur

vive g++ et son -Wshadow

skeye

chrisbk a écrit :

jeune homme, vous aurez appris de cette experience l'interet de differencier variable membre de variable locale par un quelconque moyen syntaxique

tootafé...Je cours de ce pas rajouter "m_" devant tous mes noms de variables membres! :ange:

Kristoph

chrisbk a *crit :

jeune homme, vous aurez appris de cette experience l'interet de differencier variable membre de variable locale par un quelconque moyen syntaxique

Je prefere la methode de Python : acceder a toutes les donnes membres en passant pas (*this)

Mais bon, je peux comprendre que ca en rebute certains aussi

skeye

Kristoph a écrit :

Je prefere la methode de Python : acceder a toutes les donnes membres en passant pas (*this)

Mais bon, je peux comprendre que ca en rebute certains aussi

Bah vi mais C++ il comprend même si on le met pas le this...et c'est ca qui m'a fait perdre un peu de temps cet aprem'!

skeye

skeye a écrit :

Code :

int * tab = new int[this->getSize()];

C'est juste ca non?
Parce-que j'aimerais savoir ce qui pourrait causer une segfault lors du delete d'une telle variable...

chrisbk

skeye a écrit :

C'est juste ca non?
Parce-que j'aimerais savoir ce qui pourrait causer une segfault lors du delete d'une telle variable...

en quoi ca pourrait etre faux ?

skeye

chrisbk a écrit :

en quoi ca pourrait etre faux ?

J'en sais rien, je suis juste trop habitué au C et aux malloc (pas taper!), ca me fait bizarre le new pour un int*...
Bon, c'est déjà ca, si c'est correct, mais ca me dit pas pourquoi je me retrouve avec un segfault lorsque je delete mon tab...!
La fonction donne à peu près ca:

Code :

void
CImage::rotation(int angle)
{
int * tmp = new int[this->getSize()];
//Recopie temporaire du tableau
for(int i=0;i<this->getSize();i++)
tmp[i]=this->getVal(i);
switch(angle)
{
case 90: //val[i, j] = val[j, i]
{
//code utilisant tmp
break;
}
case 180: //val[i] = val[size - i];
{
//code utilisant tmp
break;
}
case 270: //val[i,j] = val[j, size - i]
{
//code utilisant tmp
break;
}
default:
return;
}
delete tmp;
}

Après un passage dans le case 90, segmentation fault.
Si je commente le delete tmp; tout se passe bien...
Comprends pas... :??:

chrisbk

attention jeune homme

allocation d'un tableau :
new [..]

liberation d'un tableau :
delete []tableau;

skeye

chrisbk a écrit :

attention jeune homme

allocation d'un tableau :
new [..]

liberation d'un tableau :
delete []tableau;

ahhhhhhhhhhhhhhhhhh!
Voilà p-e l'erreur (c'est bizarre que ca me donne ce résultat qu'ici, je l'ai fait au moins 10 fois...:lol

Taz

bisounours-codeur

on va pas quoter, mais ça mériterait

skeye

Bon, ca me fait quelques erreurs de corrigées (:jap, mais ca ne résoud pas le pb...les symptomes sont les mêmes!

skeye

++Taz a écrit :

on va pas quoter, mais ça mériterait

Hésite pas...je reconnais être un gros boulet qd je m'y mets... :sweat:

skeye

Bon, ya du nouveau...après 2/3 tests, un make clean et une recompil, le segfault intervient maintenant même si je commente le delete.
Par contre je comprends toujours pas la cause du bug!
Dans le case 90 juste avant le break j'ai mis un cout, qui s'affiche.
Il devrait donc sortir de la fonction sans problème...
Mais j'ai aussi miss un cout dans la fonction appelante, juste après l'appel...et celui-ci n'apparait pas!
Je commence à :pt1cable: , d'autant plus que c'est probablement une connerie, comme d'habitude avec moi...

[edit]
Ca y est, je suis fou:
J'ai viré tout ce qu'il pouvait se passer après la fonction à-part les cout => plante tjrs.
J'ai alors viré le delete => plante plus
J'ai tout remis sauf le delete => le cout de la fonction appelante apparait, mais ca crashe qd meme... :fou: :pt1cable: :??:

Message édité par skeye le 29-04-2003 à 16:07:45

Publicité

FORUM HardWare.fr

Programmation

C++

[C++] Pb de boucle for récalcitrante

Sujets relatifs
boucle while indentifiant variable	[Socket Linux] Qd le client ferme, le serveur boucle sur le message
[PHP] petit probleme avec boucle	[PHP] Afficher dans boucle les caract de 3 objets de même type ...
Boucle While - printf - scanf - fflush	[C] Problème de getc dans une boucle
Boucle + Zone de texte = Help me !!	[VBA] interuption de traitement dans une boucle ...
[JS] Cocher une checkbox générée à partir d'une boucle PHP (réparé)	[C] contrôle if dans une boucle ... ?
Plus de sujets relatifs à : [C++] Pb de boucle for récalcitrante

Page générée en 0.105 secondes