Discussion :

[Crypt]

Bonjour,

Il y a quelque chose que je ne comprend pas.

Admettons que dans une application on ait:

message("LOADING"); ==> message affiche quelque chose à l'écran dans une JPanel, par exemple.

suivi directement d'une série de lectures/écritures de fichiers non formatés.


message() est en fait executé APRES la sequence de sérialisation.

Même si je fais quelque chose du genre:

Code :

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

1
2
3
4
5
6
 
try { 
 message("LOADING");
 Thread.sleep(1000);
} catch (InterruptedException ex) {ex.printStackTrace();}
// et ensuite la sérialisation

ça fait exactement la même chose. Je ne comprend pas, ça me semble même carrément illogique (évidemment je me trompe forcément quelque part.)

PS: Si par contre message() est remplacé par un affichage dans la console par un System.out.print alors ça s'execute dans le bon ordre (mais là ça ne m'interesse pas des masses.)


PPS: au lieu de message(String str) ça peut aussi bien être la gestion d'une JProgressBar, à vue de nez, ça donnera la même chose, sauf erreur de ma part.

[afrikha]

tu pourrais poster le code de la méthode message ? car vraisemblablement c'est delà que provient le "malfonctionnement" .

à vue de nez , je dirais que tu effectues la sérialisation dans l'EDT .

[®om]

Si tu nous montrais ta méthode "message(...)"

Si dans cette méthode tu as (comme tu devrais avoir) un Runnable à ajouter dans l'EDT (SwingUtilities.invokeLater(Runnable)), c'est normal, l'EDT est une liste fifo des actions à effectuer dans l'interface graphique...

Si tu n'as pas ceci dans ta méthode, cela m'étonne, car il ne me semble pas que ça soit automatique de transmettre à l'EDT pour un .setText()...

[Crypt]

tu pourrais poster le code de la méthode message ? car vraisemblablement c'est delà que provient le "malfonctionnement" .

à vue de nez , je dirais que tu effectues la sérialisation dans l'EDT .

ça ne peut pas venir de là, j'ai fais le même test avec d'autres methodes ayant d'une manière ou d'une autre un impact sur le GUI et sans aucun rapport avec cette méthode "message"
==> même résultat.

un exemple:

Code :

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

1
2
3
4
5
6
7
8
9
10
try {
// teleportBol détermine l'affichage ou non d'un certain JPanel
             monde.jeuPan.teleportBol=true;
//même avec un repaint, que dalle
             monde.jeuPan.repaint();
            Thread.sleep(1000);
        } catch (InterruptedException ex) {
            ex.printStackTrace();
        }
//suivi de la sérialisation

et finalement le contenu du try est affiché après la sérialisation.

[®om]

Montre-nous le "vrai" code de ta méthode (complet)

[Crypt]

Code :

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

1
2
3
4
5
6
public void message(String me,boolean l,String couleur){
        if(jeuPan!=null){
            jeuPan.messagesPanel.remplissageMessages(me,l,couleur);
            afficherSeparateurRound=true;
        }
    }

jeuPan est une instance de PanneauJeu:
public class PanneauJeu extends JPanel implements KeyListener
cette dernière a bien sûr:

Code :

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

1
2
3
4
5
public void paint(Graphics g){
        super.paint(g);
..........etc
 
}

messagePanel est une instance de MessagesPanel=>
pour vous faire une idée c'est la zone de texte en bas sur cette image=



bon là je met le code entier de MessagesPanel si vous voulez mais je doute que ça vienne de là=

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
package mainPack;
import javax.swing.*;
import java.awt.event.*;
import java.awt.*;
import javax.swing.text.*;
 
class MessagesPanel extends JPanel implements FocusListener,MouseListener{
 
 
    private JScrollPane scroll;
    public JScrollPane getScroll(){return scroll;}
 
    private String messages="";
 
    JTextPane messagesList = new JTextPane();
    DefaultStyledDocument doc = new DefaultStyledDocument();
    Style def = StyleContext.getDefaultStyleContext().getStyle(StyleContext.DEFAULT_STYLE);
 
    private PanneauJeu jeuPan;
 
    Font police;
 
    Style sImage;
    public MessagesPanel(PanneauJeu pj){
        police=pj.getPolice();
        jeuPan=pj;
        setLayout(null);
 
        messagesList.setDocument(doc);
        StyleConstants.setFontFamily(def, "SansSerif");
        Style regular = messagesList.addStyle("regular", def);
        Style s = messagesList.addStyle("red", regular);
        StyleConstants.setForeground(s, Color.RED);
        s = messagesList.addStyle("green", regular);
        StyleConstants.setForeground(s, Color.GREEN);
        s = messagesList.addStyle("blue", regular);
        StyleConstants.setForeground(s, new Color(0 ,191 ,255));
        s = messagesList.addStyle("white", regular);
        StyleConstants.setForeground(s, Color.WHITE);
 
        sImage = messagesList.addStyle("image", null);
 
 
        scroll = new JScrollPane(messagesList);
 
        scroll.setVerticalScrollBarPolicy(
                JScrollPane.VERTICAL_SCROLLBAR_ALWAYS);
 
        scroll.setBounds(0,0,640,165);
        add(scroll);
 
        messagesList.setEditable(false);
        messagesList.addFocusListener(this);
        messagesList.setBackground(Color.black);
        messagesList.setForeground(Color.green);
        messagesList.setFont(police);
 
        messagesList.addMouseListener(this);
    }
 
    public void paint(Graphics g){
        super.paint(g);
    }
    public void update(Graphics g){
        paint(g);
    }
 
 
    public void mouseEntered(MouseEvent e){
 
        jeuPan.resizeMessagePanel(true);
    }
    public void mouseExited(MouseEvent e){
        jeuPan.resizeMessagePanel(false);
    }
 
    public void mouseClicked(MouseEvent e){}
    public void mousePressed(MouseEvent e){}
    public void mouseReleased(MouseEvent e){}
 
 
 
 
 
    public void remplissageImage(String imStr, int pos){
        try {
            StyleConstants.setIcon(sImage, new ImageIcon(imStr));
            doc.insertString(doc.getLength()+pos,"\n", sImage);
            messagesList.scrollRectToVisible(messagesList.modelToView(messagesList.getDocument().getLength()));
        } catch (BadLocationException e) {}
    }
 
    public void remplissageImage_et_Message(String imStr,String mess,boolean langMess,String couleur){
        if(CryptRL.lang==langMess){
            try{
 
                StyleConstants.setIcon(sImage, new ImageIcon(imStr));
                doc.insertString(doc.getLength(),"tagadapouet", sImage);
 
                doc.insertString(doc.getLength(),mess+"\n", messagesList.getStyle(couleur));
 
                messagesList.scrollRectToVisible(messagesList.modelToView(messagesList.getDocument().getLength()));
 
            }catch(BadLocationException ble){}
        }
 
    }
 
 
    public void remplissageMessages(String mess,boolean langMess,String couleur){
        if(CryptRL.lang==langMess){
            try{
 
 
                doc.insertString(doc.getLength(),mess+"\n", messagesList.getStyle(couleur));
 
                messagesList.scrollRectToVisible(messagesList.modelToView(messagesList.getDocument().getLength()));
 
 
            }catch(BadLocationException ble){
            }
 
        }
 
    }
 
 
    public void focusGained(FocusEvent e){
        jeuPan.requestFocus();
    }
    public void focusLost(FocusEvent e){
    }
 
}

Si par exemple je demande que l'inventaire (le panneau à droite sur l'image) apparaisse juste avant la sérialisation ça sera pareil (et ça n'a aucun rapport avec message.)




une des méthode de sérialisation (si plus d'infos est nécessaire je les donnerai mais j'ai bon espoir que mon probleme soit en fait un probleme simple, un oubli dû à quelque chose que j'ignore concernant la sérialisation et les threads) =

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
 
public static void voyageHorizontal(Monde monde,int direction){
 
        monde.pluieImpossible=false;
        monde.mondePause=true;
 
        boolean random=false;
 
        monde.stockageCoord_RIVIERES_ROUTES(direction);
        monde.creaturesSuiventPersoDansVoyage(false,direction,false);
 
        Sauvegarde s=saveGame(monde);
 
 
        for(int x=monde.minBd;x<=monde.maxBd;x++){
            for(int y=monde.minBd;y<=monde.maxBd;y++){
                monde.trousPlafondGrid[x][y]=false;
            }
        }
 
        monde.oldInsertPt=new Point(monde.perso.getCooX(),monde.perso.getCooY());
        //changement coordonnées perso
        if(direction==1){//NORD
            monde.perso.indexTerrainY--;
            monde.perso.setCooY(monde.getTG()-11);
        }
        if(direction==2){//EST
            monde.perso.indexTerrainX++;
            monde.perso.setCooX(10);
        }
        if(direction==3){//SUD
            monde.perso.indexTerrainY++;
            monde.perso.setCooY(10);
        }
        if(direction==4){//OUEST
            monde.perso.indexTerrainX--;
            monde.perso.setCooX(monde.getTG()-11);
        }
 
 
 
        String mapStr=monde.perso.getNom()+"/"+"x"+monde.perso.indexTerrainX+"y"+monde.perso.indexTerrainY+"z"+monde.perso.indexTerrainZ;
        File fiMap=new File("./save/"+mapStr+".zip");
 
        //map=null;
        if(fiMap.exists()){
            MapFile map=null;
            unzip("./save/",mapStr);
            monde.message("Vous êtes déjà venu. Map trouvée : "+mapStr,false,"white");
            monde.message("You've already been there. Map Found :"+mapStr,true,"white");
            map=lecteurMap(monde,"./save/"+mapStr+".sav");
            monde.message("Map Found :"+map,true,"white");
 
            // mettre à jour monde avec les différents champs du fichier
            monde.grille=map.grille;
            monde.groundsGrid=map.groundsGrid;
 
            monde.mapGrid=map.mapGrid;
            monde.bloodGrid=map.bloodGrid;
            monde.memoTerrainGrid=map.memoTerrainGrid;
            monde.trousPlafondGrid=map.trousPlafondGrid;
            monde.nomMap=map.mapName;
            monde.nomMapE=map.mapNameE;
 
            monde.grille[monde.perso.getCooX()][monde.perso.getCooY()].add(monde.perso);
 
        } else{
            monde.message("Vous venez ici pour la premiere fois. Pas de map nommée "+mapStr,false,"white");
            monde.message("This is the first time you come here. There is no map file named :"+mapStr,true,"white");
 
            // soit c'est une carte prédéfinie //terrainx0y0z0.zip
            File fi=new File("./predefinedMaps/terrain"+"x"+monde.perso.indexTerrainX+"y"+monde.perso.indexTerrainY+"z"+monde.perso.indexTerrainZ+".zip");
            if(fi.exists()){
                monde.message("Chargement carte prédéfinie",false,"white");
                monde.message("Loading predefined map",true,"white");
 
                unzip("./predefinedMaps/","terrain"+"x"+monde.perso.indexTerrainX+"y"+monde.perso.indexTerrainY+"z"+monde.perso.indexTerrainZ);
 
                FileInputStream out2=null;
                ObjectInputStream flux2=null;
                boolean boolFlux=true;
 
                try{
                    out2=new FileInputStream("./predefinedMaps/terrain"+"x"+monde.perso.indexTerrainX+"y"+monde.perso.indexTerrainY+"z"+monde.perso.indexTerrainZ+".sav");
                    flux2=new ObjectInputStream(out2);
 
                    MapFile mpP=(MapFile)flux2.readObject();
                    monde.grille=mpP.grille;
                    monde.groundsGrid=mpP.groundsGrid;
                    monde.nomMap=mpP.mapName;
                    monde.nomMapE=mpP.mapNameE;
                    monde.grille[monde.perso.getCooX()][monde.perso.getCooY()].add(monde.perso);
                }
 
                catch(FileNotFoundException fnfe){
                    boolFlux=false;
                    monde.message("ERROR predefined map doesn't exists !",true,"white");
                    monde.message("ERREUR la carte prédéfinie est introuvable !",false,"white");
                }
 
                catch(ClassNotFoundException f){f.printStackTrace();} catch(IOException i){i.printStackTrace();} finally{
                    if(boolFlux){
                        try {
                            flux2.close();
                            out2.close();
                        } catch (IOException e1) {
                            //e1.printStackTrace();
                        }
                    }
                }
                videur("./predefinedMaps/");
            }
 
 
            // soit il faut faire un random
            else{
                monde.message("Random Generation",true,"white");
                monde.message("Génération aléatoire",false,"white");
                monde.creationTerrainAleatoire(false,false,/*typeEscalier*/ -1);
                monde.grille[monde.perso.getCooX()][monde.perso.getCooY()].add(monde.perso);
 
                random=true;
            }
 
 
            //////////////////////////////////////////////
            monde.initialisationMapGrid();
 
            for(int x=monde.minBd;x<=monde.maxBd;x++){
                for(int y=monde.minBd;y<=monde.maxBd;y++){
                    monde.bloodGrid[x][y]=0;
                }
            }
 
 
            for(int x=monde.minBd;x<=monde.maxBd;x++){
                for(int y=monde.minBd;y<=monde.maxBd;y++){
                    monde.memoTerrainGrid[x][y]=false;
                }
            }
 
        }
 
        ///////// SUPPRESSION DES OBSTACLES SOUS LE PERSO ////////////
        //////////////On casse le mur sur lequel le personnage pourrait éventuellement arriver
        for(int i=0;i<monde.grille[monde.perso.getCooX()][monde.perso.getCooY()].size();i++){
            try{
                Entite objet = (Entite)monde.grille[monde.perso.getCooX()][monde.perso.getCooY()].get(i);
                if(objet.isObstacleId()) {
                    monde.grille[monde.perso.getCooX()][monde.perso.getCooY()].remove(objet);
                }
            } catch(NoSuchElementException exc){};
        }
 
 
 
       MapsGates dg=new MapsGates("./save/"+monde.perso.getNom()+"/dg.dat");
       dg.work(monde);
 
        monde.recuperationObjetsTombes();
        CryptRL.stockImage.creationImagesCat(monde);
        monde.miseAJourResumeStatsPanel();
        monde.undeads();//marche mal
        monde.jeuPan.mapPanel.mapGrid=monde.mapGrid;
        monde.jeuPan.mapPanel.repaint();
        monde.miseAJourMapVisiteesNoms();
        monde.miseAJourNat_Cult_Types();
 
        monde.mondePause=false;
 
 
        monde.assureCoords_Apres_Voyage();
}
 
 
 
 
 
 
public static Sauvegarde saveGame(Monde monde){
 
 
        monde.mondePause=true;
 
        monde.analyzeDynamicGates();
 
 
        Sauvegarde s=null;
 
        String nomSauv=monde.perso.getNom()+"/"+monde.perso.getNom();
 
 
 
        FileInputStream out=null;
        ObjectInputStream flux=null;
        boolean fileExist=true;
        try{
            out=new FileInputStream("./save/"+nomSauv+".sav");
            flux=new ObjectInputStream(out);
            s=(Sauvegarde)flux.readObject();
        } catch(FileNotFoundException fnf){fileExist=false;
        monde.message("ERREUR Pas de sauvegarde nommée "+nomSauv+".sav",false,"white");
        monde.message("ERROR There is no save file named :"+nomSauv+".sav",true,"white");
        } catch(ClassNotFoundException f){f.printStackTrace();} catch(IOException i){i.printStackTrace();} finally{
 
            try {
                flux.close();
                out.close();
            } catch (IOException e1) {
                //e1.printStackTrace();
            }
 
        }
        // chercher la map correspondant au terrain courant
        String mapStr=monde.perso.getNom()+"/"+"x"+monde.perso.indexTerrainX+"y"+monde.perso.indexTerrainY+"z"+monde.perso.indexTerrainZ;
        File fiMap=new File("./save/"+mapStr+".zip");
        MapFile map=new MapFile();
        //map=null;
        if(fiMap.exists()){
            unzip("./save/",mapStr);
            monde.message("Map trouvée : "+mapStr,false,"white");
            monde.message("Map Found :"+mapStr,true,"white");
            map=lecteurMap(monde,"./save/"+mapStr+".sav");
            monde.message("Map Found :"+map,true,"white");
        } else{
            monde.message("Pas de map nommée "+mapStr,false,"white");
            monde.message("There is no map file named :"+mapStr,true,"white");
 
 
 
        }
 
        //Mise à jour de la map
        map.grille=monde.grille;
        map.groundsGrid=monde.groundsGrid;
 
 
        map.mapName=monde.nomMap;
        map.mapNameE=monde.nomMapE;
 
 
        map.mapGrid=monde.mapGrid;
        map.bloodGrid=monde.bloodGrid;
        map.memoTerrainGrid=monde.memoTerrainGrid;
        map.entitesTombeesXYZ=monde.entitesTombeesXYZ;
        map.trousPlafondGrid=monde.trousPlafondGrid;
        map.grille[monde.perso.getCooX()][monde.perso.getCooY()].remove(monde.perso);
 
        monde.message("MapFile mis à jour "+map,false,"white");
        monde.message("MapFile Updated :"+map,true,"white");
 
        // Mise à jour de s
        s.pj=monde.perso;
        s.nuit=monde.getNuit();
        s.tour=monde.getTour();
 
        monde.message("SaveFile mis à jour "+s,false,"white");
        monde.message("SaveFile Updated :"+s,true,"white");
 
 
        enregistreur(map, "./save/"+monde.perso.getNom()+"/"+"x"+monde.perso.indexTerrainX+"y"+monde.perso.indexTerrainY+"z"+monde.perso.indexTerrainZ+".sav");
        zip("./save/",monde.perso.getNom()+"/"+"x"+monde.perso.indexTerrainX+"y"+monde.perso.indexTerrainY+"z"+monde.perso.indexTerrainZ);
        videur("./save/"+monde.perso.getNom());
        enregistreur(s, "./save/"+nomSauv+".sav");
 
        monde.message("SaveFile & MapFile sauvés ",false,"white");
        monde.message("SaveFile & MapFile saved ",true,"white");
 
        monde.grille[monde.perso.getCooX()][monde.perso.getCooY()].add(monde.perso);
 
 
        monde.message("****Game Saved****",true,"white");
        monde.message("****Partie sauvee****",false,"white");
        monde.mondePause=false;
 
        return s;
    }
 
 
public static void videur(String dos){
        File repertoire=new File(dos);
        direct = ".";
        File f1 = new File(direct);
        int k;
        String[] ls;
        FilenameFilter filter = new FiltreFichier("sav");
        File[] list = repertoire.listFiles(filter);
        for(int i=0;i<list.length;i++){
            list[i].delete();
        }
    }
 
 
 
public static void enregistreur(Object ob, String nomSave){
        FileOutputStream out_Save=null;
        ObjectOutputStream flux_Save=null;
        try{
            out_Save=new FileOutputStream(nomSave);
            flux_Save=new ObjectOutputStream(out_Save);
            flux_Save.writeObject(ob);
        } catch(IOException i){i.printStackTrace();} finally{
            try {
                flux_Save.flush();
                flux_Save.close();
                out_Save.close();
            } catch (IOException e1) {e1.printStackTrace();}
        }
    }

[Crypt]

Si tu nous montrais ta méthode "message(...)"
Si dans cette méthode tu as (comme tu devrais avoir) un Runnable à ajouter dans l'EDT (SwingUtilities.invokeLater(Runnable)), c'est normal, l'EDT est une liste fifo des actions à effectuer dans l'interface graphique...

Pas de Runnable.
Est-ce que tu pourrais m'expliquer un peu plus en détail ?


Il y a assez peu de threads dans mon application: les sons, un de rafraichissement forçé du terrain (ce qui permet quelques animations) et les threads "cachés" de la GUI auquels je n'ai pas touché.

[®om]

Pas de Runnable.
Est-ce que tu pourrais m'expliquer un peu plus en détail ?


Il y a assez peu de threads dans mon application: les sons, un de rafraichissement forçé du terrain (ce qui permet quelques animations) et les threads "cachés" de la GUI auquels je n'ai pas touché.

À mon avis c'est parce que tu ne fais pas tes modifications graphiques dans l'EDT (pour tes 2 actions qui se font pas dans le bonne ordre, devant chacune affiche le resultat de SwingUtilities.isEventDispatchThread(), si ça affiche false y'a de grandes chances que ça vienne de là)...

Pour plus d'infos:
http://gfx.developpez.com/tutoriel/j...ing-threading/

[Crypt]

À mon avis c'est parce que tu ne fais pas tes modifications graphiques dans l'EDT (pour tes 2 actions qui se font pas dans le bonne ordre, devant chacune affiche le resultat de SwingUtilities.isEventDispatchThread(), si ça affiche false y'a de grandes chances que ça vienne de là)...

Pour plus d'infos:
http://gfx.developpez.com/tutoriel/j...ing-threading/

voilà une piste interessante ! Merci, je vais étudier ça sérieusementn.

[Crypt]

au délà de la question de l'ordre d'execution, est-ce que cela signifie qu'il est bon de placer toute instruction SWING dans :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
new Thread(new Runnable() {
        public void run() {
            SwingUtilities.invokeLater(new Runnable() {
                public void run() {
 
                   //instruction swing
 
 
                }
            });
        }
    }).start();

?


hormis pour le contenu d'une méthode paint() ou paintComponent() je suppose ( ? )



Pour info =
System.out.println("EDT : "+SwingUtilities.isEventDispatchThread());
placé devant l'execution de message et devant celle de la sérialisation affiche "true" systématiquement. A ma grande surprise.


Par contre par la suite j'ai mis le contenu de chaque methode message() dans le bloc juste au dessus.
J'ai fait la même chose avec le contenu de la methode de serialisation.
==> l'execution se fait dans le bon ordre.



J'en ai profité pour faire un test : depuis longtemps les effets sonores dans mon appli ont tendance à faire de vilains hoquets. Il semblerait (faut je test un peu plus) qu'en plaçant le démarrage des sons dans le bloc magique sus-nommé le problème disparaisse !!! Si ça se confirme alors c'est la fête.

[sinok]

Pour le paint et le paintComponent forcément vu que c'est Swing qui va directement les gérer (et comme toujours éviter les opréations trop lourdes à l'intérieur.

Sinon pour le reste: http://download.java.net/jdk6/docs/a...html#threading

ET pour le SwingUtilities, il faut nécessairement l'appeler quand tu veux faire des opérations sur Swing depuis l'extérieur de l'EDT, le reste est à ta discrétion

[broumbroum]

au délà de la question de l'ordre d'execution, est-ce que cela signifie qu'il est bon de placer toute instruction SWING dans :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
new Thread(new Runnable() {
        public void run() {
            SwingUtilities.invokeLater(new Runnable() {
                public void run() {
 
                   //instruction swing
 
 
                }
            });
        }
    }).start();

(...)

c'est un bon moyen de lancer les processus sans devoir attendre leur complétion, mais dans ce cas tu n'as plus le contrôle de la dite complétion, ce qui amène à des problèmes de synchronisation typiques.
dans le code très court que tu fournis, je pense que l'instanciantion du Thread se fait déjà avec la classe swingutilities. Si tu veux observer le cours de ton processus (Thread) alors ajoute une commande de notification d'avancement, du style n++ puis invite le processus à rafraichir une barre de progression, je te recommanderais JProgressBar, à l'aide d'un listener éventuellement.
C'est une des méthodes que j'utilise actuellement.

[Crypt]

je te recommanderais JProgressBar, à l'aide d'un listener éventuellement.

oui au final c'est là que je voulais en venir.


Par contre je ne vois pas bien comment je suis censé utiliser une JPB par exemple pour donner l'information de la progression de la méthode suivante =

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
public static void voyageHorizontal(Monde monde,int direction){
 
        monde.mondePause=true;
 
        monde.pluieImpossible=false;
        boolean random=false;
        monde.stockageCoord_RIVIERES_ROUTES(direction);
        monde.creaturesSuiventPersoDansVoyage(false,direction,false);
 
        Sauvegarde s=saveGame(monde);
 
        ///Reinitialisation trousPlafondGrid à 0
        for(int x=monde.minBd;x<=monde.maxBd;x++){
            for(int y=monde.minBd;y<=monde.maxBd;y++){
                monde.trousPlafondGrid[x][y]=false;
            }
        }
 
        monde.oldInsertPt=new Point(monde.perso.getCooX(),monde.perso.getCooY());
        //changement coordonnées perso
        if(direction==1){//NORD
            monde.perso.indexTerrainY--;
            monde.perso.setCooY(monde.getTG()-11);
        }
        if(direction==2){//EST
            monde.perso.indexTerrainX++;
            monde.perso.setCooX(10);
        }
        if(direction==3){//SUD
            monde.perso.indexTerrainY++;
            monde.perso.setCooY(10);
        }
        if(direction==4){//OUEST
            monde.perso.indexTerrainX--;
            monde.perso.setCooX(monde.getTG()-11);
        }
 
 
        String mapStr=monde.perso.getNom()+"/"+"x"+monde.perso.indexTerrainX+"y"+monde.perso.indexTerrainY+"z"+monde.perso.indexTerrainZ;
        File fiMap=new File("./save/"+mapStr+".zip");
 
        //map=null;
        if(fiMap.exists()){
            MapFile map=null;
            unzip("./save/",mapStr);
            monde.message("Vous êtes déjà venu. Map trouvée : "+mapStr,false,"white");
            monde.message("You've already been there. Map Found :"+mapStr,true,"white");
            map=lecteurMap(monde,"./save/"+mapStr+".sav");
            monde.message("Map Found :"+map,true,"white");
 
            // mettre à jour monde avec les différents champs du fichier
            monde.grille=map.grille;
            monde.groundsGrid=map.groundsGrid;
 
            monde.mapGrid=map.mapGrid;
            monde.bloodGrid=map.bloodGrid;
            monde.memoTerrainGrid=map.memoTerrainGrid;
            monde.trousPlafondGrid=map.trousPlafondGrid;
            //monde.entitesTombeesXYZ=map.entitesTombeesXYZ;// en commentaire sinon problemes
 
            monde.nomMap=map.mapName;
            monde.nomMapE=map.mapNameE;
 
            monde.grille[monde.perso.getCooX()][monde.perso.getCooY()].add(monde.perso);
 
        } else{
            monde.message("Vous venez ici pour la premiere fois. Pas de map nommée "+mapStr,false,"white");
            monde.message("This is the first time you come here. There is no map file named :"+mapStr,true,"white");
 
            // soit c'est une carte prédéfinie //terrainx0y0z0.zip
            File fi=new File("./predefinedMaps/terrain"+"x"+monde.perso.indexTerrainX+"y"+monde.perso.indexTerrainY+"z"+monde.perso.indexTerrainZ+".zip");
            if(fi.exists()){
                monde.message("Chargement carte prédéfinie",false,"white");
                monde.message("Loading predefined map",true,"white");
 
                unzip("./predefinedMaps/","terrain"+"x"+monde.perso.indexTerrainX+"y"+monde.perso.indexTerrainY+"z"+monde.perso.indexTerrainZ);
 
                FileInputStream out2=null;
                ObjectInputStream flux2=null;
                boolean boolFlux=true;
 
                try{
                    out2=new FileInputStream("./predefinedMaps/terrain"+"x"+monde.perso.indexTerrainX+"y"+monde.perso.indexTerrainY+"z"+monde.perso.indexTerrainZ+".sav");
                    flux2=new ObjectInputStream(out2);
 
                    MapFile mpP=(MapFile)flux2.readObject();
                    monde.grille=mpP.grille;
                    monde.groundsGrid=mpP.groundsGrid;
                    monde.nomMap=mpP.mapName;
                    monde.nomMapE=mpP.mapNameE;
                    monde.grille[monde.perso.getCooX()][monde.perso.getCooY()].add(monde.perso);
                }
 
                catch(FileNotFoundException fnfe){
                    boolFlux=false;
                    monde.message("ERROR predefined map doesn't exists !",true,"white");
                    monde.message("ERREUR la carte prédéfinie est introuvable !",false,"white");
                }
 
                catch(ClassNotFoundException f){f.printStackTrace();} catch(IOException i){i.printStackTrace();} finally{
                    if(boolFlux){
                        try {
                            flux2.close();
                            out2.close();
                        } catch (IOException e1) {
                            //e1.printStackTrace();
                        }
                    }
                }
                videur("./predefinedMaps/");
            }
 
 
            // soit il faut faire un random
            else{
                monde.message("Random Generation",true,"white");
                monde.message("Génération aléatoire",false,"white");
                monde.creationTerrainAleatoire(false,false,/*typeEscalier*/ -1);
                monde.grille[monde.perso.getCooX()][monde.perso.getCooY()].add(monde.perso);
 
                random=true;
            }
 
 
            //////////////////////////////////////////////
            monde.initialisationMapGrid();
 
 
            for(int x=monde.minBd;x<=monde.maxBd;x++){
                for(int y=monde.minBd;y<=monde.maxBd;y++){
                    monde.bloodGrid[x][y]=0;
                    monde.memoTerrainGrid[x][y]=false;
                }
            }
 
 
        }
 
 
        ///////// SUPPRESSION DES OBSTACLES SOUS LE PERSO ////////////
        //////////////On casse le mur sur lequel le personnage pourrait éventuellement arriver
        for(int i=0;i<monde.grille[monde.perso.getCooX()][monde.perso.getCooY()].size();i++){
            try{
                Entite objet = (Entite)monde.grille[monde.perso.getCooX()][monde.perso.getCooY()].get(i);
                if(objet.isObstacleId()) {
                    monde.grille[monde.perso.getCooX()][monde.perso.getCooY()].remove(objet);
                }
            } catch(NoSuchElementException exc){};
        }
 
 
 
        MapsGates dg=new MapsGates("./save/"+monde.perso.getNom()+"/dg.dat");
        dg.work(monde);
 
 
 
        monde.recuperationObjetsTombes();
        CryptRL.stockImage.creationImagesCat(monde);
        monde.miseAJourResumeStatsPanel();
        monde.undeads();//marche mal
        monde.jeuPan.mapPanel.mapGrid=monde.mapGrid;
        monde.miseAJourMapVisiteesNoms();
        monde.miseAJourNat_Cult_Types();
        monde.assureCoords_Apres_Voyage();
 
 
        monde.jeuPan.mapPanel.repaint();
        monde.mondePause=false;
 
}

[Crypt]

Par exemple si je fais ça :

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
 public static void voyageHorizontal(Monde mo,int dir){
        final Monde monde=mo;
        final int direction=dir;
 
        monde.mondePause=true;
 
        // jpBol => pour l'affichage de la JPB. A ce stade elle est bien visible
        monde.jeuPan.jpBol=true;
        monde.jeuPan.jBar.setValue(0);
 
        /// pour que l'affichage de la JPB se fasse bien avant.
        new Thread(new Runnable() {
            public void run() {
                SwingUtilities.invokeLater(new Runnable() {
                    public void run() {
 
                        /////////////////////////////////////////////////////////////
                        // LA SEQUENCE DE SERIALISATION                        
                        ........(du code)
 
                        // Un changement de valeur de la JPB
                        monde.jeuPan.jBar.setValue(10);
 
                        // SUITE DE LA SEQUENCE DE SERIALISATION
                        ........(du code)
 
                        // Un autre changement de valeur de la JPB
                        monde.jeuPan.jBar.setValue(20);
 
                        // SUITE DE LA SEQUENCE DE SERIALISATION
                        .........ETC
 
 
 
                        /// FIN
                        monde.mondePause=false;
                        monde.jeuPan.jpBol=false;
                        /////////////////////////////////////////////////////////////
 
                    }
                });
            }
        }).start();
 
    }

Les changements de valeur de la JPB
ex: monde.jeuPan.jBar.setValue(10);
ne se font qu'après (je suppose fortement) et donc au final on ne les voit pas du tout.

Pour info la JPB est déclaré dans une JPanel (je poste uniquement les tronçons concernés, la classe compléte fait 1700 lignes) =>

Code :

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

1
2
JProgressBar jBar = new JProgressBar();
boolean jpBol=false;

Dans le constructeur de la JPanel :

Code :

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

1
2
3
4
5
6
7
8
9
jBar.setBounds(172,10,100,20);
jBar.setString("Progression");
jBar.setStringPainted(true);
jBar.addChangeListener(this);
jBar.setMinimum(0);
jBar.setMaximum(99);
jBar.setValue(0);
add(jBar);
jBar.setVisible(false);

dans la méthode paint() de la JPanel:

Code :

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

jBar.setVisible(jBol);

et on a aussi la méthode suivante:

Code :

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

1
2
3
4
public void stateChanged(ChangeEvent e) 
{
    jBar.setString ((int)(jBar.getPercentComplete()*100) + "%");
}

Comment faire pour que les actualisation de la JPB se fasse pendant la sérialisation ?

[®om]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
    new Thread(new Runnable() {
        public void run() {
            SwingUtilities.invokeLater(new Runnable() {
                public void run() {
 
                   //instruction swing
 
 
                }
            });
        }
    }).start();

C'est totalement inutile de créer un nouveau Thread...


Soit tu es dans l'EDT, et tu fais directement:

Code :

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

// instructions swing

Soit tu n'es pas dans l'EDT, et tu fais:

Code :

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

1
2
3
4
5
SwingUtilities.invokeLater(new Runnable() {
    public void run() {
        // instructions swing
    }
}

Le cas où il faut combiner les 2, c'est quand tu dois effectuer un traitement long dans un thread séparé, et qu'à la fin du traitement, tu mets à jour l'interface:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
new Thread() {
    @Override public void run() {
        // quoi qu'il arrive, ici on n'est pas dans l'EDT
        final String s = traitementLong(); //dure longtemps
        SwingUtilities.invokeLater(new Runnable() {
            public void run() {
                // instructions Swing, par exemple:
                monTextField.setText(s);
            }
        }
        System.out.println("test");
    }
}.start();

à noter que l'ordre entre l'affichage de "test" et l'attribution de la String s à monTextField n'est pas garanti (ie ça dépend des exécutions)...

[Crypt]

Dois-je en déduire que le fait que l'affichage se fait dans le bon ordre (je ne parle pas de la JPB mais simplement d'une instruction swing juste avant la sérialisation) est, dans le code suivant, uniquement dû à l'usage d'un Thread et non pas à l'usage de SwingUtilities.invokeLater ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
new Thread(new Runnable() {
        public void run() {
            SwingUtilities.invokeLater(new Runnable() {
               public void run() {
 
                    //sérialisation......etc
 
 
                }
            });
        }
   }).start();

Je dis ça parcequ'en mettant la sérialisation uniquement dans

Code :

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

1
2
3
4
5
6
7
SwingUtilities.invokeLater(new Runnable() {
     public void run() {
 
 
 
     }
});

l'ordre d'exécution n'est pas le bon.




Je crois que je ne capte plus rien là...

Ce qui m'interesse ce n'est pas ce qui se passe après la sérialisation (de toute façon c'est un jeu en tour par tour) mais ce qui se passe juste avant.



PS: SwingUtilities.isEventDispatchThread() renvoyait déjà true juste avant la serialisation , même avant ces modifs.



Bon, une question: il y a t'il un interêt à mettre autre chose qu'une instruction Swing dans un

Code :

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

1
2
3
4
5
6
SwingUtilities.invokeLater(new Runnable() {
               public void run() {
 
 
                }
           });

??

[Crypt]

Le cas où il faut combiner les 2, c'est quand tu dois effectuer un traitement long dans un thread séparé, et qu'à la fin du traitement, tu mets à jour l'interface:
Code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
new Thread() {
    @Override public void run() {
        // quoi qu'il arrive, ici on n'est pas dans l'EDT
        final String s = traitementLong(); //dure longtemps
        SwingUtilities.invokeLater(new Runnable() {
            public void run() {
                // instructions Swing, par exemple:
                monTextField.setText(s);
            }
        }
        System.out.println("test");
    }
}.start();

Je crois pouvoir dire que c'est le cas ici, tout le bloc de sérialisation est relativement long à l'exécution (1 ou 2 secondes).
La seule différence c'est que je met le tout dans le SwingUtilities.invokeLater, pas seulement les instructions Swing.

Est-ce que c'est risqué de mettre autre chose que du Swing dans invokeLater ?

[sinok]

Oui, c'est comme si tu faisait tes traitements directement dans une actionListener.

Traitements non relatifs à Swing sont à faire en dehors de l'EDT. enfin le mieux est de se mettre au swingworker: https://swingworker.dev.java.net/

[Crypt]

Bon, ok, je vois.

Merci pour toutes ces infos.