Discussion :

[azmodai]

Bonjour,

Je souhaite "délocaliser" une partie de mon travail dans une fonction.
Cependant je suppose que je me heurte à un problème d'adresse ...

Voici 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
 
int main(void) {
        ........
 
        IplImage *source_hsv;
        IplImage *source_rgb;
 
        source_hsv = cvCloneImage(source);
        source_rgb = cvCreateImage(cvGetSize(source), source->depth, 1);
        result = cvCreateImage(cvGetSize(source_rgb), source_rgb->depth, 1);
 
        cvConvertImage(source, source_rgb, 0);
        cvCvtColor(source, source_hsv, CV_BGR2HSV);
 
        cvCanny(source_rgb, result, 20.0, 30.0, 3);
 
        ....

Dans mon main le bout de code marche parfaitement (ici l'image s'affiche correctement).

mais lorsque je mets tout ça dans une fonction de cette manière ci dessous
l'image n'est plus correcte, comme si l'opération ne c'était pas bien déroulée ..

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
26
27
28
29
 
 
void Detection(IplImage *source, IplImage *result) {
 
        IplImage *source_hsv;
        IplImage *source_rgb;
 
        source_hsv = cvCloneImage(source);
        source_rgb = cvCreateImage(cvGetSize(source), source->depth, 1);
        result = cvCreateImage(cvGetSize(source_rgb), source_rgb->depth, 1);
 
        cvConvertImage(source, source_rgb, 0);
        cvCvtColor(source, source_hsv, CV_BGR2HSV);
 
        cvCanny(source_rgb, result, 20.0, 30.0, 3);
}
 
int main(void) {
 
       ....
 
       Detection(source, result);
 
       .....
 
       // ici result ne semble pas correcte !
 
       return 0;
}

Je suppose qu'il y a un problème lors du passage d'adresse, en effet
une fois les adresses passées à la fonction Detection elles sont à nouveaux passées à d'autres fonctions et c'est surment là que ça cloche (?), l'adresse correcte n'est pas donnée ou autre (?).

Qu'en pensez-vous ?

merci,

PS: J'utilise GCC,

[jeroman]

Salut

Tu crées deux pointeurs

Code :

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

1
2
IplImage *source_hsv;
IplImage *source_rgb;

qui sont capitaux pour tes fonctions de traitement d'image. Je suppose que ces pointeurs sont essentiels également pour la suite du programme.

Dans le second code, le problème est que tu crées deux pointeurs locaux à la fonction "Detection"... qui sont perdus une fois la fonction terminée.
Il faut déclarer ces deux pointeurs non pas dans la fonction "Detection", mais dans la fonction "main". Et lorsque tu appelles "Detection", il faut lui faire passer l'adresse de ces deux pointeurs afin qu'elle puisse modifier leur valeur "à distance".

[azmodai]

Bonjour jeroman,

Non en fait les deux pointeurs source_* ne sont pas nécessaires pour la suite du code. En réalité ma fonction Detection n'est pas totalement terminée (les pointeurs source_* seront utilisés plus tard dans la fonction).
En fait je souhaite juste récupérer result, mais cela ne fonctionne pas.

[diogene]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
void Detection(IplImage *source, IplImage *result) {
...
        result = cvCreateImage(cvGetSize(source_rgb), source_rgb->depth, 1);
...
 
        cvCanny(source_rgb, result, 20.0, 30.0, 3);
}
int main(void) {
       ....
       Detection(source, result);
       .....
       // ici result ne semble pas correcte ! 
       return 0;
}

Dans la fonction, le seul fait d'avoir un paramètre, ici result, modifié avant toute utilisation doit être suspect : La valeur passée par ce paramètre est ignorée donc ce paramètre est inutile
Dans le main() : Detection(source, result). Quelle que soit la fonction Detection(), source et result ne peuvent être modifiés par la fonction.

En conclusion, puisque la fonction doit modifier result (du main) , il faut passer son adresse ou utiliser la valeur de retour :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void Detection(IplImage *source, IplImage **result) {
...
        *result = cvCreateImage(cvGetSize(source_rgb), source_rgb->depth, 1);
...
 
        cvCanny(source_rgb, *result, 20.0, 30.0, 3);
}
int main(void) {
       ....
       Detection(source, &result);
       ..... 
       return 0;
}

ou

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
IplImage *Detection(IplImage *source) {
...
       IplImage * result = cvCreateImage(cvGetSize(source_rgb), source_rgb->depth, 1);
...
 
        cvCanny(source_rgb, result, 20.0, 30.0, 3);
       return result;
}
int main(void) {
       ....
       result = Detection(source);
       .....
       return 0;
}

[azmodai]

Merci ça a reglé le problème!