Discussion :

[ZeFMG]

Bonjour à tous,

j'ai un petit bout de code à l'intérieur d'un programme :

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
 
h=figure;
        taille_ecran=get( 0, 'ScreenSize');
        set(h,'Position',[taille_ecran(3)/2, taille_ecran(4)/2, 300, 120]);
        set(h,'Menubar','none');
        set(h,'Name','Dans quelle feuille coller les données ?');
        set(h,'NumberTitle','off');
        set(h,'BusyAction','cancel');
        set(h,'Tag','fenetre_choixfeuille');
        hpop = uicontrol('Style', 'popup',...
               'String', 'pneu1|pneu2|pneu3|pneu4|pneu5|pneu6|pneu7|pneu8|pneu9|pneu10',...
               'Position', [20 70 260 30]);
        val = 0;
        h_button = uicontrol('Style', 'pushbutton', 'String', 'Ok',...
            'Position', [20 20 260 30], 'Callback', 'val = get(hpop,''Value''); close(h);');
        uiwait(h);

Ce code fonctionne parfaitement quand je l'execute tout seul, mais quand je l'utilise dans mon programme, lorsque je clique sur le bouton, Matlab me renvoie :

Code :

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

1
2
3
4
5
??? Error using ==> waitfor
Undefined function or variable 'hpop'.
 
??? Error using ==> waitfor
Error while evaluating uicontrol Callback.

Quelqu'un aurait-il une idée du problème ?

Merci d'avance,
ZeFMG

[Jerome Briot]

C'est tout à fait normale, vu la manière dont tu as écris le Callback.

Les variables hpop et h ne sont visibles qu'à l'intérieur du Callback, elle ne sont donc pas transmise par le programme principale.

Tu devrais jeter un oeil à ceci : Développement efficace des interfaces graphiques

[ZeFMG]

Merci, c'était une erreur bien bête...
Voilà le code corrigé, ça aidera peut-être quelqu'un d'autre :

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
 
h=figure;
        taille_ecran=get( 0, 'ScreenSize');
        set(h,'Position',[taille_ecran(3)/2, taille_ecran(4)/2, 300, 120]);
        set(h,'Menubar','none');
        set(h,'Name','Dans quelle feuille coller les données ?');
        set(h,'NumberTitle','off');
        set(h,'BusyAction','cancel');
        set(h,'Tag','fenetre_choixfeuille');
        hpop = uicontrol('Style', 'popup',...
               'String', 'pneu1|pneu2|pneu3|pneu4|pneu5|pneu6|pneu7|pneu8|pneu9|pneu10',...
               'Position', [20 70 260 30]);
        val = 0;
        h_button = uicontrol('Style', 'pushbutton', 'String', 'Ok',...
            'Position', [20 20 260 30], 'Callback', 'val = get(findobj(''Style'',''popup''),''Value''); setappdata(0,''val'',val); close(findobj(''Tag'',''fenetre_choixfeuille''));');
        uiwait(h);
        val=getappdata(0,'val');

[Jerome Briot]

Une petite remarque au passage.

Mettre un Callback dans une chaine de caractère (comme tu l'as fait) n'est pas très optimisé sous MATLAB.

En effet, MATLAB est obligé d'interpréter à chaque fois le Callback avant de l'exécuter... si tu copies le Callback dans une fonction (sous-fonction ou fonction imbriquée), MATLAB n'iterprétera qu'une seule fois la fonction qui sera précompilé en mémoire pour une utilisation ultérieur.

Sur ton exemple cela ne fait pas beaucoup de différence, mais c'est quand même une bonne habitude à prendre