Problème de comparaison.

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Problème de comparaison.

Evil_Ifrit

gooby pls

Bien le bonjour.

Je suis en train de dévelloper une petite application de traitement d'image,
dans laquelle j'ai écrit cette ligne de code, qui crashe à l'éxécution
(le programme compile sans aucune erreur ni warning) :

Code :

if(getPixel(source,i,j) == 255) {
/*traitement*/
}

Ce que je ne comprend pas, c'est que plus haut dans le code,
j'ai cette ligne qui fonctionne parfaitement:

Code :

if(getPixel(source,i,j) == 0) {
/*traitement*/
}

J'ai aussi testé une comparaison avec 254 et 256, bilan : ligne fonctionnelle.

PS : La fonction getpixel renvoie un unsigned char.

Quelqu'un a une explication ? Merci d'avance.

Message édité par Evil_Ifrit le 01-02-2007 à 14:16:53

---------------
Soon.

Publicité

Elmoricq

Modérateur

Mince !

Message édité par Elmoricq le 01-02-2007 à 14:20:18

franceso

ce qui est important, c'est pas la valeur (255 ou 0) à laquelle tu compares le résultat, mais plutôt les valeurs de i et j. Il faut t'assurer que i et j ne dépassent pas les tailles max allouées pour les données de ton image (variable 'source', je suppose).

Je ne peux pas t'en dire plus sans avoir un peu de contexte. En particulier, d'où viennent i et j, et surtout comment a été alloué 'source' (et accessoirement, le code de getPixel serait peut-être utile)

---------------
TriScale innov

olivthill

Les variable i et j sont peut-être hors des limites du bitmap, ce qui peut arriver sur l'on inverse la colonne avec la ligne, ou si l'on croit que le bitmap courvre tous l'écran au lieu de ne couvrir qu'une portion de l'écran

Evil_Ifrit

gooby pls

En fait j'ai créé ces deux fonctions aux codes quasi-identiques (mis à part la fameuse comparaison),
la preuve en code (désolé pour la longueur) :

Code :

void Dilatation(tImage *source, tImage *res) {
int i, j, val;
tHistogramme histo;
res->dimX = source->dimX;
res->dimY = source->dimY;
res->buffer = (unsigned char*)malloc((res->dimX)*(res->dimY)*sizeof(char));
for ( i=1 ; i < res->dimX ; i++ ) {
for( j=1 ; j < res->dimY ; j++ ) {
if(getPixel(source,i,j) == 0) {
setPixel(res, i-1, j, 0);
setPixel(res, i, j-1, 0);
setPixel(res, i, j, 0);
setPixel(res, i, j+1, 0);
setPixel(res, i+1, j, 0);
}
else
setPixel(res, i, j,getPixel(source,i,j));
}
}
DessineImageGris(IMAGE_TRAITEE, res);
ConstruitHistogramme(res, &histo);
DessineHistogramme(HISTOGRAMME_IMAGE_TRAITEE, &histo);
free(res->buffer);
}
void Erosion(tImage *source, tImage *res) {
int i, j, val;
tHistogramme histo;
res->dimX = source->dimX;
res->dimY = source->dimY;
res->buffer = (unsigned char*)malloc((res->dimX)*(res->dimY)*sizeof(char));
for ( i=1 ; i < res->dimX ; i++ ) {
for( j=1 ; j < res->dimY ; j++ ) {
if(getPixel(source,i,j) == 255) {
setPixel(res, i-1, j, 255);
setPixel(res, i, j-1, 255);
setPixel(res, i, j, 255);
setPixel(res, i, j+1, 255);
setPixel(res, i+1, j, 255);
}
else
setPixel(res, i, j,getPixel(source,i,j));
}
}
DessineImageGris(IMAGE_TRAITEE, res);
ConstruitHistogramme(res, &histo);
DessineHistogramme(HISTOGRAMME_IMAGE_TRAITEE, &histo);
free(res->buffer);
}

Sur la même image, la fonction dilatation marche, et érosion crashe.
Je suis sûr à 100% que l'erreur ne vient pas des fonctions appelées (DessineImageGris & Cie),
Donc je ne pense pas non plus que l'erreur vienne des variables i et j passant hors limite.

D'autres idées ? :]

PS : Au cas où, le code de getpixel :

Code :

unsigned char getPixel(tImage *img, int x, int y) {
return (*(img->buffer + (y*img->dimX)+x));

Message cité 1 fois
Message édité par Evil_Ifrit le 01-02-2007 à 14:32:13

---------------
Soon.

franceso

Code :

for ( i=1 ; i < res->dimX ; i++ ) {
for( j=1 ; j < res->dimY ; j++ ) {

Je pense qu'il faut plutôt mettre

Code :

for ( i=1 ; i < res->dimX-1 ; i++ ) {
for( j=1 ; j < res->dimY-1 ; j++ ) {

Comme ça les indices que tu utilises dans tes traitements (i-1,i+1,j-1,j+1) restent toujours dans les limites allouées : [0;dimX] et [0;dimY]

---------------
TriScale innov

Elmoricq

Modérateur

A priori rien à voir avec ton bug, mais après les malloc() (dont le cast est inutile d'ailleurs), tu devrais vérifier que l'allocation s'est bien effectuée.
Pour le cas où, histoire de ne pas taper dans un res->buffer qui contiendrait NULL.

Evil_Ifrit

gooby pls

Aw yeah, la solution de francesco marche :]
Comme quoi les erreurs ne sont pas tout le temps là où on les attend.
Erreur toute nulle en plus ^^

Merci beaucoup à tous (et spécialement à francesco).

Message cité 1 fois
Message édité par Evil_Ifrit le 01-02-2007 à 14:44:50

franceso

Evil_Ifrit a écrit :

Aw yeah, ça marche :]
Comme quoi les erreurs ne sont pas tout le temps là où on les attend.
Erreur toute nulle en plus ^^

Les erreurs ne sont jamais où on les attend

C'est pas une erreur si nulle que ça : c'est sûrement le type d'erreur le plus rencontré en C. Tu peux aussi apprendre à utiliser un debugger, ce qui te permettrait de localiser beaucoup plus rapidement et précisément l'origine des problèmes.

---------------
TriScale innov

Emmanuel Delahaye

C is a sharp tool

Evil_Ifrit a écrit :

En fait j'ai créé ces deux fonctions aux codes quasi-identiques (mis à part la fameuse comparaison),

Code :

void Dilatation(tImage *source, tImage *res) {
int i, j, val;
tHistogramme histo;
res->dimX = source->dimX;
res->dimY = source->dimY;

T'as rien dilaté du tout...

Code :

res->dimX = source->dimX + 2;
res->dimY = source->dimY + 2;

ou un truc comme ça...

Le tableau est donc trop petit. Comportement indéfini.

Toujours vérifier ses indices. assert() peut aider...

Message cité 1 fois
Message édité par Emmanuel Delahaye le 10-02-2007 à 13:24:47

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/

Publicité

Evil_Ifrit

gooby pls

Emmanuel Delahaye a écrit :

T'as rien dilaté du tout...

Code :

res->dimX = source->dimX + 2;
res->dimY = source->dimY + 2;

ou un truc comme ça...

Le tableau est donc trop petit. Comportement indéfini.

Toujours vérifier ses indices. assert() peut aider...

Je tenais à garder la taille de l'image de base,
la dilatation ne devait s'appliquer que sur les formes contenues dans l'image.

De plus, je n'applique pas l'algorithme de dilatation sur les pixels situés 'au bord' de l'image.
La tableau n'est donc pas trop petit, et le comportement défini.

Cordialement.

FORUM HardWare.fr

Programmation

Problème de comparaison.

Sujets relatifs
[résolu]Probléme de comparaison de checkbox	Probleme de comparaison #2 [resolu]
[resolu]Probleme de comparaison	problème de comparaison.. apparemment
probleme comparaison de variable	Problème de jointure ORACLE avec comparaison de champs null
[c++] probleme de comparaison de type d'objets derivés	Probléme de Comparaison de Dates...
problème de comparaison de date (mysql).	[XML] comparaison de syntaxe, probleme d'occurrence
Plus de sujets relatifs à : Problème de comparaison.

Page générée en 0.076 secondes