Coordination de création d'image avec la paint(Graphics g) par un un Thread

Bonjour;
Je veux faire contrôler la création de mes images avec la methode paint(Graphis g) d'une classe par un Thread d'une autre classe, j'ai essayé d'utiliser un objet de ma classe qui crée les images dans ma class thread du coups ça il y a rien qui s'affiche dans ma fenetre JPanel et la je me démande quoi pour que le thread puisse synchroniser la création de mes images, voici mon code

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

public class ThControl extends SwingWorker<Object, String> { private boolean actif = true; private static final int DELAI = 100; creeImage c= new creeImage (); //Graphics g = null; @Override protected Object doInBackground() throws Exception { while(actif) { Thread.sleep(DELAI); c.paint(g) /****Je me demande quoi mettre ici pour que je puis coordonner le deplacement de mes images entre par exemple lorque jmon compteur atteint 5         deux imagesou trois quittent le point depart1, lorqu'elles arrivent à la position de l'image2 elles disparassent et provoque le depart de l'image 2 ****/ } return null; }   }

---

Voici la classe qui crée les images.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

public class creeImage extends JPanel {   public void paint(Graphics g) { super.paint(g); // On ajoute à la position le delta x et y de la vitesse depart1.translate(-vitesse.x, -vitesse.y); depart2.translate(-vitesse.x, vitesse.y);     try { element1= ImageIO.read(new File("image1.png")); element2 = ImageIO.read(new File("image2.png"));   } catch (Exception ex) { ex.printStackTrace(); }   //g.drawImage(avion, position2.x, position2.y, this); if(depart1.x >= depart1.x ) g.drawImage(element1, depart1.x, depart1.y, this); ifdepart2.x <= 15) g.drawImage(element1,ifdepart2.x, ifdepart2.y, this);   }

---

Merci d'avance.

Salut,

Tout d'abord, le JPanel que tu crées par creeImage c= new creeImage (); n'est affiché nulle part : on ne risque pas de voir quoique ce soit.

Deuxièmement, on ne doit pas invoquer des méthodes Swing dans doInBackground : cette méthode est là pour traiter le processus dans un thread différent de l'EDT (Event Dispatch Thread). C'est la méthode process() qui est exécutée dans l'EDT et sert à faire les mises à jour de l'affichage. Et la méthode doInBackground() génère les ordres qui seront traités par process() avec la méthode publish().

Troisièmement, on ne devrait pas appeler dans ton cas la méthode paint(Graphics) mais repaint(). Pour tout ce qui se dessiner à l'écran, paint() est censé être appelée par Swing, dans l'EDT, avec l'instance de Graphics qu'il gère lui.

Quatrièmement, ne lis pas de fichier dans la méthode paint() sinon tu vas ralentir inutilement l'EDT (surtout dans le cadre d'une animation, ça risque de produire des saccades) .

Cinquièmement, évite de faire des modifications d'états dans le paint() : cette méthode peut être appelée à ton insu (quand Swing estime qu'il est nécessaire de redessiner), donc provoquer ces changements d'états lorsque tu ne veux pas (dans ton cas faire par exemple depart1.translate(-vitesse.x, -vitesse.y) intempestivement).

Enfin, pour faire ce que tu fais, à moins d'avoir des calculs de durée variable et relativement longs dans doInBackground (là tu ne fais que répéter une tâche à intervalle régulier), il vaut mieux utiliser un javax.swing.Timer, ce sera plus simple.

Merci, pour tes conseils je suis encore débutant en programmation, il y a des conceptes qui m’échappe encore.
En fait pour l'affichage peut être je me suis mal exprimé, j'ai une autre méthode main qui contient l'appel des classes pour l'affichage, je n'ai pas le droit de fournir tout mon code ici, si tu acceptais les messages privées j'allais te l'envoyer peut être ça pourra être plus claire.
Si je retire l'instanciation

Code:

---

creeImage c= new creeImage()

---

dans la methode doInBackground() tout s'affiche correctement, c'est pourquoi je m'en doutais qu'on ne peux pas faire ça dans la methode doInBackground() et je n'ai pas le droit de modifier le code qui m'a été fournie actuellement je cherche un moyen pour implémenter le controle de l'affichage, de mouvement, le nombre de tours de mes élements dans la méthode doInBackground(), mais là je ne sais pas par quelle je pourrais le faire, sans utiliser la class CreeImage et ses méhodes.

Merci d'avance.

Citation:

---

Envoyé par Khalfe

je retire l'instanciation

Code:

---

creeImage c= new creeImage()

---

dans la methode doInBackground()

---

Cette instruction n'est pas dans la méthode doInBackground() mais en membre de la classe.

Citation:

---

Envoyé par Khalfe

tout s'affiche correctement, c'est pourquoi je m'en doutais qu'on ne peux pas faire ça dans la methode doInBackground() et je n'ai pas le droit de modifier le code qui m'a été fournie actuellement

---

C'est-à-dire ? Quel code tu ne peux pas modifier ? Si tu ne peux pas modifier le code que tu montres, tu peux tout de suite arrêter de bosser sur ce projet, parce qu'il n'y a pas moyen de faire autrement.

Citation:

---

Envoyé par Khalfe

je cherche un moyen pour implémenter le controle de l'affichage, de mouvement, le nombre de tours de mes élements dans la méthode doInBackground(), mais là je ne sais pas par quelle je pourrais le faire, sans utiliser la class CreeImage et ses méhodes.

---

Je n'ai pas dit que tu ne pouvais pas utiliser cette classe et ces méthodes, j'ai dit qu'il ne fallait pas le faire dans la méthode doInBackground().

Je te l'ai dit, le plus simple est d'utiliser un Timer Swing, parce que la partie à programmer s'exécute dans le thread de Swing, donc pas de souci pour appeler des méthodes qui ne doivent être exécutées que dans le thread Swing.

Si tu persistes dans l'usage de SwingWorker, parce qu'on te demande absolument d'utiliser ça, alors il faut procéder comme suit : dans doInBackground, avec la méthode publish() tu généres des ordres que la méthode process convertira en ordre Swing. Tu peux simple produire des Runnable par exemple.

Voici un exemple :

Code:

---

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

import java.awt.Color; import java.awt.Dimension; import java.awt.Graphics; import java.awt.Rectangle; import java.awt.event.WindowAdapter; import java.awt.event.WindowEvent; import java.util.Arrays; import java.util.Collection; import java.util.List;   import javax.swing.JFrame; import javax.swing.JPanel; import javax.swing.SwingWorker;   public class ExempleAnimationSwingWorker {   private static final int BORDER = 50; private static final int WIDTH = 300; private static final int MOBILE_WIDTH = 50;   private static final long DELAY = 33; private static final int SPEED = 5;   public static void main(String[] args) {   JFrame frame = new JFrame("Démo"); frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);   Panel panel = new Panel(ExempleAnimationSwingWorker.WIDTH+ExempleAnimationSwingWorker.BORDER*2, new Mobile(ExempleAnimationSwingWorker.MOBILE_WIDTH, ExempleAnimationSwingWorker.BORDER, ExempleAnimationSwingWorker.BORDER, Dir.EAST, Color.RED, ExempleAnimationSwingWorker.SPEED), new Mobile(ExempleAnimationSwingWorker.MOBILE_WIDTH, ExempleAnimationSwingWorker.BORDER+ExempleAnimationSwingWorker.WIDTH-ExempleAnimationSwingWorker.MOBILE_WIDTH, ExempleAnimationSwingWorker.BORDER, Dir.SOUTH, Color.GREEN, ExempleAnimationSwingWorker.SPEED), new Mobile(ExempleAnimationSwingWorker.MOBILE_WIDTH, ExempleAnimationSwingWorker.BORDER+ExempleAnimationSwingWorker.WIDTH-ExempleAnimationSwingWorker.MOBILE_WIDTH, ExempleAnimationSwingWorker.BORDER+ExempleAnimationSwingWorker.WIDTH-ExempleAnimationSwingWorker.MOBILE_WIDTH, Dir.WEST, Color.YELLOW, ExempleAnimationSwingWorker.SPEED), new Mobile(ExempleAnimationSwingWorker.MOBILE_WIDTH, ExempleAnimationSwingWorker.BORDER, ExempleAnimationSwingWorker.BORDER+ExempleAnimationSwingWorker.WIDTH-ExempleAnimationSwingWorker.MOBILE_WIDTH, Dir.NORTH, Color.CYAN, ExempleAnimationSwingWorker.SPEED) );   frame.add(panel);   frame.pack(); frame.setLocationRelativeTo(null); frame.setResizable(false); frame.setVisible(true);   Animation animation = new Animation(ExempleAnimationSwingWorker.DELAY, panel, new Rectangle(ExempleAnimationSwingWorker.BORDER,ExempleAnimationSwingWorker.BORDER,ExempleAnimationSwingWorker.WIDTH,ExempleAnimationSwingWorker.WIDTH));   frame.addWindowListener(new WindowAdapter() { @Override public void windowClosed(WindowEvent e) { animation.cancel(true); } });     animation.execute();       }   public enum Dir { EAST(1,0), SOUTH(0,1), WEST(-1,0), NORTH(0,-1);   private final int dx; private final int dy; private Dir(int dx, int dy) { this.dx=dx; this.dy=dy; }   public int getDx() { return dx; }   public int getDy() { return dy; }   public Dir next() { int ordinal = this.ordinal()+1; if ( ordinal>=values().length ) { ordinal=0; } return values()[ordinal]; }   }   public static class Mobile {   private Color color; private int width; private int x; private int y; private Dir dir; private int speed;   public Mobile(int width, int x, int y, Dir dir, Color color, int speed) { this.color=color; this.width=width; this.x=x; this.y=y; this.dir=dir; this.speed=speed; }   public void draw(Graphics g) { g.setColor(color); g.fillRect(x, y, width, width); }   public void move(Rectangle bounds) { x+=dir.getDx()*speed; y+=dir.getDy()*speed; switch(dir) { case EAST: int maxwidth = (int) bounds.getMaxX(); if ( x+width>maxwidth ) { x=maxwidth-width; dir = dir.next(); } break; case WEST: int minwidth = (int) bounds.getMinX(); if ( x<minwidth ) { x=minwidth; dir = dir.next(); } break; case SOUTH: int maxheight = (int) bounds.getMaxY(); if ( y+width>maxheight ) { y=maxheight-width; dir = dir.next(); } break; case NORTH: int minheight = (int) bounds.getMinY(); if ( y<minheight ) { y=minheight; dir = dir.next(); } break; } }     }   public static class Panel extends JPanel {   private List<Mobile> mobiles;   public Panel(int width, Mobile...mobiles) { super(true); setBackground(Color.BLACK); setPreferredSize(new Dimension(width,width)); this.mobiles = Arrays.asList(mobiles); }   @Override protected void paintComponent(Graphics g) { super.paintComponent(g); for(Mobile mobile : mobiles) { mobile.draw(g); } }   public Collection<? extends Mobile> getMobiles() { return mobiles; }   }   public static class Animation extends SwingWorker<Object, Runnable> {   private final Panel panel; private final Rectangle bounds; private final long delai;   public Animation(long delai, Panel panel, Rectangle bounds) { this.panel=panel; this.bounds=bounds; this.delai=delai; }   @Override protected Object doInBackground() throws Exception { while(!isCancelled()) { for(Mobile mobile:panel.getMobiles()) { publish(()->mobile.move(bounds)); } Thread.sleep(delai); } return null; }   @Override protected void process(List<Runnable> runnables) { if ( !runnables.isEmpty() ) { for(Runnable runnable : runnables) { runnable.run(); } panel.repaint(); } }   }   }

---