Problème avec l'instruction Thread.sleep

[I]Bonjour,
Dans le cadre d'un petit programme, j'a réalisé une application qui est censée afficher alternativement une image puis une autre image. Pour cela j'ai utilisé l'instruction Thread.sleep() pour que le programme affiche une image, attende 1 seconde, affiche une autre image, attende 1 seconde...

Lorsque j'exécute ce programme à part, tout se passe bien, mais lorsque j'intègre le code du programme dans un autre programme (qui est en fait le menu du logiciel), le Thread.sleep ne marche pas, et au lieu d'afficher alternativement les deux images, la fenêtre reste blanche au début puis affiche seulement la dernière image.

Voici le code source du programme qui fonctionne correctement quand il est lancé à part :

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

public class Fenetre extends JFrame { public Fenetre() { new JFrame(); this.setTitle("Série d'images"); this.setSize(500, 500); this.setLocationRelativeTo(null); this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); this.setVisible(true); this.setContentPane(new Panneau2()); try { Thread.sleep(1000); } catch (InterruptedException e) { e.printStackTrace(); } this.validate(); this.setContentPane(new Panneau()); try { Thread.sleep(1000); } catch (InterruptedException e) { e.printStackTrace(); } this.validate(); this.setContentPane(new Panneau2());   } }

---

Les classes Panneau et Panneau2 servent seulement à afficher l'une ou l'autre des deux images.

Et voici comment j'intègre le code précédent au code du menu (je ne mets que la partie du code qui correspond car le code entier est trop long) :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

public static class Test extends JPanel implements ActionListener{   public void actionPerformed (ActionEvent e){   fenetre.dispose(); new Fenetre(); }       }

---

Je ne sais pas à quel niveau se situe le problème, mais il est possible qu'il y ait des erreurs liées au fait que je débute en java, donc merci de me signaler si quelque chose ne va pas.
Je vous remercie de votre aide.

Tu à essayé de l'intégrer dans un thread à part? Parce que là le thread de ton jpanel aussi prend le sleep...

Et au passage, ta fenêtre étend jframe, donc ne fais pas new jframe() dans ton constructeur mais super()

avec des Thread.sleep, tu bloque le thread courant. Avec ton intégration, le thread courant, c'est le thread graphique => tu bloque l'interface graphique.

Tu ne devrais pas faire de Thread.sleep, en encore moins dans un constructeur. On attends d'un constructeur qu'il construise et préparer l'objet, pas qu'il fasse tout y compris le café.

Dans ton cas, si cette fenêtre a pour but de faire défiler des images, je te recommanderais:

d'utiliser javax.swing.Timer pour gérer le défilement
de démarrer ce timer soit dans le constructeur (démarrage automatique), soit dans une méthode à part, que tu appellerais après avoir construit la fenêtre (à préférer, ça évite d'exposer this avant la fin du constructeur)

cafeinoman : merci pour l'indication concernant le JFrame.
tchize : comment est-ce que je peux utiliser le timer dans ce cas, parce que je dois faire afficher deux images alternativement, mais pour l'instant j'ai seulement trouvé la méthode scheduleAtFixedRate() qui permet d'exécuter une action de manière périodique, donc si tu connais une instruction qui puisse m'être utile, je suis preneur.
Merci.

ben oui, a intervale régulier, tu change l'image, tout simplement :)

Si j'ai bien compris la suggestion de Tchize :

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

public class Fenetre extends JFrame { public Fenetre() { super(); init();   this.setContentPane(new Panneau2()); try { Thread.sleep(1000); } catch (InterruptedException e) { e.printStackTrace(); } this.validate(); this.setContentPane(new Panneau()); try { Thread.sleep(1000); } catch (InterruptedException e) { e.printStackTrace(); } this.validate(); this.setContentPane(new Panneau2());   }   private void init () { this.setTitle("Série d'images"); this.setSize(500, 500); this.setLocationRelativeTo(null); this.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE); this.setVisible(true); int pan=0; ActionListener listener = new ActionListener (   @Override public void actionPerformed(ActionEvent evt) { switch(pan++% 2){ case (0):this.setContentPane ( new Panneau ( ) ); break; case (1):this.setContentPane ( new Panneau2 ( ) ); break; } } Timer timer = new Timer (1000, listener); timer.start (); }   }

---

Bon courage.

salut,
en réponse à cafeinoman, j'ai un problème avec la méthode private void init() : il me semble que tu t'es trompé après le new ActionListener : tu rajoutes une parenthèse, mais du coup lorsque je l'enlève, j'ai beau tout essayer (doubles parenthèses, accolades...), eclipse me signale des erreurs sur les accolades... je ne vois pas où est le problème :

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

public class Fenetre extends JFrame { public Fenetre() { super(); init();   this.setContentPane(new Panneau2()); try { Thread.sleep(1000); } catch (InterruptedException e) { e.printStackTrace(); } this.validate(); this.setContentPane(new Panneau()); try { Thread.sleep(1000); } catch (InterruptedException e) { e.printStackTrace(); } this.validate(); this.setContentPane(new Panneau2());   }   private void init () { this.setTitle("Série d'images"); this.setSize(500, 500); this.setLocationRelativeTo(null); this.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE); this.setVisible(true); int pan=0; ActionListener listener = new ActionListener (   @Override public void actionPerformed(ActionEvent evt) { switch(pan++% 2){ case (0):this.setContentPane ( new Panneau ( ) ); break; case (1):this.setContentPane ( new Panneau2 ( ) ); break; } } Timer timer = new Timer (1000, listener); timer.start (); }   }

---

sinon cela me semble une bonne idée mais est-ce que le problème sera résolu malgré qu'il y ait encore le Thread.sleep() ? Parce que d'après mes essais c'est bien cette instruction qui posait problème lorsque le programme lagguait.

Oui, désolé, je poste depuis mon téléphone... il faut après avoir fermé l'accolade du ActionListener fermer la parenthèse et mettre le ; , comme ceci :

Code:

---

1 2 3

timer.start (); });

---

Je corrige parce qu'il faudrait pas poster du code depuis un téléphone, c'est moche :)

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

ActionListener listener = new ActionListener (){ // <- accolade de début de classe anonyme   @Override public void actionPerformed(ActionEvent evt) { switch(pan++% 2){ case (0):this.setContentPane ( new Panneau ( ) ); break; case (1):this.setContentPane ( new Panneau2 ( ) ); break; } } // fin de méthode }; // fin de classe et fin dinstruction Timer timer = new Timer (1000, listener); timer.start ();

---

Effectivement :oops:

Sauf que du coup il me met une erreur sur le .start() : "method .start() is undefined for the type Timer", et il me met aussi une erreur sur le new Timer(1000,listener) : "The constructor Timer(int, ActionListener) is undefined" .

Code:

---

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

private void init () { this.setTitle("Série d'images"); this.setSize(500, 500); this.setLocationRelativeTo(null); this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); this.setVisible(true); final int pan=0; ActionListener listener = new ActionListener (){ // <- accolade de début de classe anonyme   @Override public void actionPerformed(ActionEvent evt) { switch(pan++% 2){ case (0):this.setContentPane ( new Panneau ( ) ); break; case (1):this.setContentPane ( new Panneau2 ( ) ); break; } } // fin de méthode }; // fin de classe et fin dinstruction Timer timer = new Timer (1000, listener); timer.start (); }

---

une idée ?

Oui : vérifie tes imports, c'est javax.swing.Timer, pas java.util.Timer. Ou alors la javadoc est fausse :aie:

Effectivement il s'agissait de cela.
Mais il me reste encore un problème (je commence à en avoir assez :mur: ) : il me dit que la méthode setContentPane () est undefined pour le type new ActionListener. Je ne sais pas si c'est encore un problème d'import, mais j'ai tout essayé et ça ne veut toujours pas m'enlever cette erreur.

Code:

---

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

private void init () { this.setTitle("Série d'images"); this.setSize(500, 500); this.setLocationRelativeTo(null); this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); this.setVisible(true); ActionListener listener = new ActionListener (){ // <- accolade de début de classe anonyme   @Override public void actionPerformed(ActionEvent evt) { int pan =0; switch(pan++% 2){ case (0):this.setContentPane(new Panneau()); break; case (1):this.setContentPane(new Panneau2 ()); break; } } // fin de méthode }; // fin de classe et fin dinstruction Timer timer = new Timer (1000, listener); timer.start (); }

---

retire ce this.


il référence la classe en cours, mais n'est nécessaire qu'en cas d'ambiguité. Et là, non seulement tu n'a pas d'ambiguité (Fenetre et la seule classe courante à avoir une méthode setContentPane), mais this représente à ce niveau là ActionListener et non pas Fenetre, donc c'est une erreur.

J'ai bien retiré this, mais du coup, qu'est-ce que je dois mettre à la place ? J'ai essayé de créer le JFrame dans la méthode init(), mais lorsque je lance le programme, ça m'affiche seulement une image, celle de Panneau2().
Voici le code :

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

public class Fenetre extends JFrame { public Fenetre() { init();   this.setContentPane(new Panneau2()); try { Thread.sleep(1000); } catch (InterruptedException e) { e.printStackTrace(); } this.validate(); this.setContentPane(new Panneau()); try { Thread.sleep(1000); } catch (InterruptedException e) { e.printStackTrace(); } this.validate(); this.setContentPane(new Panneau2());   }   private void init () { final JFrame fen = new JFrame(); this.setTitle("Série d'images"); this.setSize(500, 500); this.setLocationRelativeTo(null); this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); this.setVisible(true); ActionListener listener = new ActionListener (){ // <- accolade de début de classe anonyme   @Override public void actionPerformed(ActionEvent evt) { int pan = 0; switch(pan++% 2){ case (0):fen.setContentPane(new Panneau()); break; case (1):fen.setContentPane(new Panneau2 ()); break; } } // fin de méthode }; // fin de classe et fin dinstruction Timer timer = new Timer (1000, listener); timer.start (); } }

---

J'ajoute aussi que la méthode init() ne semble pas fonctionner : lorsque j'enlève cette méthode, le résultat est le même. Je commence à surchauffer :D

Salut,

Quand @Tchize_ t'a dit de retirer le this (sous entendu celui dans l'action listener, pour lequel tu invoques setContentPane), il ne t'a pas dit de le remplacer par la référence d'une autre fenêtre. L'enlever, c'est juste l'enlever. Cette méthode (setContentPane) est une méthode de JFrame, pas de ActionListener, donc l'invoquer sur this était indiqué comme une erreur (méthode inexistante sur ActionListener) : le fait d'enlever "this." permettait de l'invoquer sur l'instance de classe englobante, soit la fenêtre Fenetre. Là, ce que tu as fait, c'est créer une nouvelle instance de JFrame, référencée par la variable fen, mais qui n'est jamais affichée.
Si tu veux explicitement indiquer le this, tu peux écrire :

Code:

---

1 2 3 4 5 6 7 8

@Override public void actionPerformed(ActionEvent evt) { int pan = 0; switch(pan++% 2){ case 0:Fenetre.this.setContentPane(new Panneau()); break; case 1:Fenetre.this.setContentPane(new Panneau2 ()); break; } } // fin de méthode

---

Ensuite, la variable pan est créée à chaque invocation de actionPerformed, elle risque pas de changer de valeur, donc ça affichera toujours la même image.

Il faudrait créer une variable d'instance :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

private int pan = 0; @Override public void actionPerformed(ActionEvent evt) { switch(pan++% 2){ case 0:Fenetre.this.setContentPane(new Panneau()); break; case 1:Fenetre.this.setContentPane(new Panneau2 ()); break; }   } // fin de méthode

---

Si tu n'as que 2 images, tu peux utiliser aussi juste un booléen et faire un not à chaque appel (ou pan=1-pan).

D'autres part, enlèves également tout le code dans le constructeur de la fenêtre qui se trouve après l'appel de init : c'est le timer qui est chargé de faire le changement et en plus les sleep() bloque l'ui pendant la constructeur de la fenêtre, comme déjà dit par @Tchize_ (et donc tu ne verras que l'état final de l'éxécution de ce code, pas les étapes intermédiaires, en plus de bloquer l'UI.

Bonjour,
J'ai suivi vos indications et le programme marche cette fois-ci. Je tenais donc à vous remercier pour vos différentes aides et conseils. Cependant, il me reste deux petites questions : lorsque la fenêtre s'ouvre, il y a un très court temps de "latence" pendant lequel l'écran est gris : savez-vous d'où cela peut-il venir ? Et deuxième question : si j'ai besoin d'afficher successivement 5 images, comment dois-je modifier la boucle, et surtout y a-t-il un moyen plus rapide que de créer 5 classes différentes "Panneau1.java", "Panneau2.java", etc... pour afficher une image ?

Pour indication, je mets le code de la classe fenêtre désormais fonctionnelle :D :

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

public class Fenetre extends JFrame {   public Fenetre() { super(); init(); }   private void init () { this.setTitle("Série d'images"); this.setSize(500, 500); this.setLocationRelativeTo(null); this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); this.setVisible(true);   ActionListener listener = new ActionListener (){ // <- accolade de début de classe anonyme private int pan = 0; @Override public void actionPerformed(ActionEvent evt) { switch(pan++% 2){ case (0):Fenetre.this.setContentPane(new Panneau()); Fenetre.this.validate(); break; case (1):Fenetre.this.setContentPane(new Panneau2 ()); Fenetre.this.validate(); break; } } }; Timer timer = new Timer (1000, listener); timer.start (); } }

---

Et le code de la classe "Panneau" qui sert à afficher l'une des images :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

public class Panneau extends JPanel { public void paintComponent(Graphics g){ try { BufferedImage img = ImageIO.read(new File("images.jpg")); g.drawImage(img, 0, 0, this.getWidth(), this.getHeight(), this); } catch (IOException e) { e.printStackTrace();   } } }

---

Le délai initial d'une action dans un timer est le délai entre chaque action, ici donc 1 seconde, d'où une latence d'une seconde.

Soit tu initialises ta frame avec l'image qui correspond à l'état inverse de la première action (le pannel 2 donc)... soit tu mets à un délai initial de 0 par timer.setInitialDelay(0);, (tu peux avoir un tout petit délai normalement non visible, sauf si ta machine rame, puisque tu affiches ta frame avant de lancer le timer.)
En plus, j'éviterais de charger l'image dans la méthode paintComponent : fait le avant d'ouvrir la frame.

C'est à dire ? Qu'est-ce que vous entendez par avant d'ouvrir la JFrame ?

Par exemple, dans ta classe panneau, tu modifie la méthode pour qu'elle prenne une image en paramètre plutôt que de charger l'image. E
nsuite tu créé une classe ChargeurImage que tu instancie avant ta JFrame. A l'intérieur, tu mets X champs static File, un par image, ou tu fait une Enum, et tu charge les images à l'instanciation.
Pour finir, tu modifie fenêtre : tu instancie Panneau et tu le declare comme contentPane avant le ActionListener, et dans le switch, tu remplace par panneau.setImage(ChargeurImage.image1).paint();
Ca t'évite de faire de l'IO dans tes composants graphique, ainsi qu'un certain nombre d'instanciation inutiles d'un même objet.

Par "avant d'ouvrir la fenêtre", je voulais dire "avant d'afficher la fenêtre" : tout ce qui peut prendre du temps et que tu fais entre le moment où la fenêtre est visible et le moment où la première image du premier panneau s'affiche, contribue au délai dont tu parlais. En plus, effectivement, le fait de sortir le chargement de l'image d'une méthode exécutée dans le thread swing, evite de faire des accès disques, relativement long, et en plus répétés inutilement à chaque affichage.

Tu peux faire tout simplement, par exemple (voir les commentaires dans le code) :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

public class Panneau extends JPanel {   private final Image image;   public Panneau(Image image) { this.image=image; }   public void paintComponent(Graphics g){ if ( IMAGE!=null ) g.drawImage(IMAGE, 0, 0, this.getWidth(), this.getHeight(), this); }   }

---

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

public class Fenetre extends JFrame {   public Fenetre() { super(); init(); }   private Image readImage(String imagePath) { try { return ImageIO.read(new File(imagePath)); // return ImageIO.read(getClass().getResource(imagePath)); // tu pourrais faire comme ça, en plus, si tu exportes ton appli en jar, tu n'auras pas de problèmes d'emplacement de fichiers et tu pourras les intégrer dans le jar } catch (IOException e) { e.printStackTrace(); } return null; }   private void init () {   final JPanel panneau1=new Panneau(readImage("image1.jpg")); final JPanel panneau2=new Panneau(readImage("image2.jpg")); ActionListener listener = new ActionListener (){ // <- accolade de début de classe anonyme private int pan = 0; @Override public void actionPerformed(ActionEvent evt) { switch(pan++% 2){ case (0):Fenetre.this.setContentPane(panneau1); Fenetre.this.validate(); break; case (1):Fenetre.this.setContentPane(panneau2); Fenetre.this.validate(); break; } } }; Timer timer = new Timer (1000, listener); timer.setInitialDelay(0); // on a fait un max de choses avant d'ouvrir/afficher la fenêtre   this.setTitle("Série d'images"); this.setSize(500, 500); this.setLocationRelativeTo(null); this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);   // enfin ça aussi, on peut considérer que ça fait partie des max de choses à faire avant d'ouvrir/afficher la fenêtre   this.setContentPane(panneau); // afficher par défaut l'image correspondant à 1%2==0 (donc la fenêtre n'apparaitra jamais "grise") // si pas setInitialDelay(0) this.setContentPane(panneau2); // afficher par défaut l'image correspondant ) à 1%2==1   this.setVisible(true); timer.start ();   } }

---

Citation:

---

Envoyé par Cafeinoman

Par exemple, dans ta classe panneau, tu modifie la méthode pour qu'elle prenne une image en paramètre plutôt que de charger l'image.

---

Je suppose que tu voulais dire "modifier le constructeur", parce que là, on risque de penser que tu parles de modifier la méthode paintComponent...

Citation:

---

Envoyé par Cafeinoman

...

et dans le switch, tu remplace par panneau.setImage(ChargeurImage.image1).paint();

---

La le .paint(), soit il a rien à faire là, soit il manque des détails dans tes explications sur le type retourné par setImage(...), soit tu tapes encore depuis ton téléphone et le réveil est difficile ;)

Citation:

---

Envoyé par Cafeinoman

.Ca t'évite de faire de l'IO dans tes composants graphique, ainsi qu'un certain nombre d'instanciation inutiles d'un même objet.

---

Oui, pour résumé, c'était ça l'idée :ccool:

Citation:

---

Envoyé par joel.drigo

La le .paint(), soit il a rien à faire là, soit il manque des détails dans tes explications sur le type retourné par setImage(...), soit tu tapes encore depuis ton téléphone et le réveil est difficile ;)

---

Les deux mon commandant ! :aie: Après une heure de métro et avec un pc, l'idée c'était ça :

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 Panneau extends JPanel {   private Image image;   public Panneau() { }   public void afficher() { // c'est mieux que paint() comme non, ça évite les confusion... if ( image!=null ) { paintComponent(this.getGraphics()); } }   public void paintComponent(Graphics g){ g.drawImage(this.image, 0, 0, this.getWidth(), this.getHeight(), this); }   public Panneau setImage(Image image){ this.image = image; } }

---

Tout en externalisant le readImage vers une autre classe histoire de ne pas avoir du tout d'IO dans les classes swing... Mais ta méthode me semble meilleure (l'expérience peut-être :roll: )

Accessoirement, le plus simple pour afficher une image en java, ce n'est pas de surcharger paintComponent. C'est d'utiliser à la place de ton JPanel un JLabel et sa méthode setICon.

Code:

---

label.setIcon(new ImageIcon(Toolkit.getDefaultToolkit().getImage(location)));

---

Ce qui a l'avantage de gérer le caching des images et la taille du label, entre autres.

Citation:

---

Envoyé par tchize_

Accessoirement, le plus simple pour afficher une image en java, ce n'est pas de surcharger paintComponent. C'est d'utiliser à la place de ton JPanel un JLabel et sa méthode setICon.

---

Certes, au détail près, que l'image s'affichera dans sa taille d'origine.

Il n'a jamais été question de redimensionnement dans ce sujet et le code montré ne fait aucun redimensionnement ;)

Je ne parlais pas de redimensionnement mais d'étendre l'image à la taille "courante", comme dans le code posté par @DMIII :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

public class Panneau extends JPanel { public void paintComponent(Graphics g){ try { BufferedImage img = ImageIO.read(new File("images.jpg")); g.drawImage(img, 0, 0, this.getWidth(), this.getHeight(), this); } catch (IOException e) { e.printStackTrace(); } } }

---

Le jlabel fonctionne comme cela aussi, il ne dessine que sur la zone qui lui est attribuée, comme tous les composants, je ne vois pas où tu veux en venir du coup :weird:

Ah bon ? J'ai toujours constaté au contraire, en SWING, que l'affichage de l'image se faisait dans sa taille (celle de l'image dans le fichier), et non pas dans la taille du JLabel : si l'image fait 48 x 48 et que mon JLabel fait 1000x1000, j'ai une image de 48x48 affichée (au milieu, si j'ai un alignement CENTER) d'un rectangle gris de 1000x1000. Comment fais-tu pour que l'image s'affiche en 1000x1000 ?

Au temps pour moi, j'ai mal lu la javadoc de cette version de drawImage. Je me suis arrêté à la définition courte:

Citation:

---

Draws as much of the specified image as has already been scaled to fit inside the specified rectangle.

---

Et comme en dessous il y a une autre version de drawImage

Citation:

---

Draws as much of the specified area of the specified image as is currently available, scaling it on the fly to fit inside the specified area of the destination drawable surface.

---

J'ai cru qu'avec la première, il n'y avait pas de redimensionnement de l'image :aie:

Et effectivement, si tu veux du fitting de l'image au composant, il ne faut pas utiliser JLabel.

Merci pour votre aide, j'ai personnellement essayé avec la méthode de joel.drigo et cela fonctionne sans délai de latence. Du coup je me demande si il est possible que le contenu du actionPerformed ne s'exécute qu'un nombre fini de fois (par exemple 5 fois ou 10 fois) : j'ai essayé d'encadrer la méthode avec un "if" mais ça ne marche pas.
En fait, pour ne rien vous cacher, mon programme consiste à afficher des images de molécules (par exemple 5 images à la suite), puis de demander à l'utilisateur laquelle de ces molécules était par exemple un alcool. Du coup ça m'amène aussi à me poser une deuxième question : est-il possible d'afficher des images de manière aléatoire ? Je m'explique : si je possède un dossier contenant un grand nombre d'images nommées "image1.jpg" "image2.jpg" "image3.jpg", etc... existe-t-il un moyen de charger une image au hasard parmi toutes les images ?

Je poste le code des deux classes qu'il reste, étant donné que vos méthodes me permettent d'éliminer la classe Panneau2 et de ne conserver qu'une seule classe Panneau pour afficher le images sur la fenêtre.

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

public class Fenetre extends JFrame {   public Fenetre() { super(); init(); }   private Image readImage(String imagePath) { try { return ImageIO.read(new File(imagePath)); // return ImageIO.read(getClass().getResource(imagePath)); // tu pourrais faire comme ça, en plus, si tu exportes ton appli en jar, tu n'auras pas de problèmes d'emplacement de fichiers et tu pourras les intégrer dans le jar } catch (IOException e) { e.printStackTrace(); } return null; }   private void init () {   final JPanel panneau1=new Panneau(readImage("images.jpg")); final JPanel panneau2=new Panneau(readImage("zidane.jpg")); ActionListener listener = new ActionListener (){ // <- accolade de début de classe anonyme private int pan = 0; @Override public void actionPerformed(ActionEvent evt) { switch(pan++% 2){ case (0):Fenetre.this.setContentPane(panneau1); Fenetre.this.validate(); break; case (1):Fenetre.this.setContentPane(panneau2); Fenetre.this.validate(); break; } } }; Timer timer = new Timer (1000, listener); timer.setInitialDelay(0); // on a fait un max de choses avant d'ouvrir/afficher la fenêtre   this.setTitle("Série d'images"); this.setSize(500, 500); this.setLocationRelativeTo(null); this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);   // enfin ça aussi, on peut considérer que ça fait partie des max de choses à faire avant d'ouvrir/afficher la fenêtre   this.setContentPane(panneau1); // afficher par défaut l'image correspondant à 1%2==0 (donc la fenêtre n'apparaitra jamais "grise") // si pas setInitialDelay(0) this.setContentPane(panneau2); // afficher par défaut l'image correspondant ) à 1%2==1   this.setVisible(true); timer.start ();   } }

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

public class Panneau extends JPanel {   private final Image image;   public Panneau(Image image) { this.image=image; }   public void paintComponent(Graphics g){ if ( image!=null ) g.drawImage(image, 0, 0, this.getWidth(), this.getHeight(), this); }   }

---

La classe Random te permet de générer des nombres aléatoires, sous différentes contraintes. A toi de les utiliser correctement après.

Pour arrêter le timer, il suffit de faire

Code:

---

leTimerConcerné.stop();

---

Oui mais du coup, si je place le timer.stop() juste après le timer.start(), ça n'effectue qu'une seule fois le contenu de actionPerformed, alors qu'il faudrait que ça l'effectue un nombre précis de fois. Sinon en ce qui concerne le Random, je m'apprêtais à poster que je ne comprenais pas comment un nombre généré aléatoirement pouvait intervenir dans le chemin d'accès d'une image, mais je viens d'essayer et ça marche :yaisse3: : en effet, d'après ce que j'ai compris, il est possible d'écrire une ligne du type :

Code:

---

readImage("images/images/acides_carboxyliques/image"+nombreAleatoire+".png"));

---

c'est à dire pour que ça m'ouvre au hasard le fichier "image45.png" ou "image12.png".etc...

Mon seul problème reste donc le fait que le contenu du actionPerformed ne s'effectue qu'un nombre précis de fois.

Ce que tu peux faire, c'est charger un tableau d'images (si elles ne sont pas trop grosses non plus, parce qu'il ne s'agit pas de saturer la mémoire non plus), ensuite, dans l'action du timer, tu utilises un compteur que tu incrémentes, de 0 à 4 (pour chaque image), en allant donc à chaque fois chercher l'image correspondante dans le tableau, et pour arrêter le timer quand tu as atteint la fin du tableau (donc quand le compteur dépasse la taille du tableau), appeler this.stop(), ce qui arrêtera le timer,.

A partir, d'un moment, l'évolution de ton prog, va peut être justifier du regarder du côté des SwingWorker : on n'est pas dans le traitement long avec mise à jour de l'UI à la fin, mais c'est assez similaire finalement (à la place d'une barre de progression, tu as des images successives, et à la place d'un traitement long, tu as des IO), la (petite) difficulté peut être avec le SwingWorker, c'est le temps d'attente entre chaque mise à jour de l'UI.

Mes images sont en fait assez grandes... enfin ça dépend ce que l'on entend par "grandes" : certaines ont une résolution de 500 x 300 ou quelque chose comme ça, mais d'autres ont des résolutions du type 1650 x 500, donc je me pose aussi la question de savoir si je laisse la classe Panneau tel quel, avec des images qui seront du coup agrandies à la résolution de l'écran, ou alors je garde de plus petites images, mais le problème est que je ne sais pas comment centrer les images au milieu de la fenêtre.
Ah oui, j'ai un autre problème qui est apparu : lorsque je lance le programme, la première image s'affiche normalement, mais la seconde s'affiche par dessus la première, puis elle s'enlève, et à partir de là le reste se déroule normalement. Je ne sais pas à quoi c'est dû, j'ai fait quelques modifications mais il ne me semble pas qu'elles aient une répercussion sur ça. Voici le code :

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

public class Fenetre extends JFrame {   public Fenetre() { super(); init(); }   private Image readImage(String imagePath) { try { return ImageIO.read(new File(imagePath)); // return ImageIO.read(getClass().getResource(imagePath)); // tu pourrais faire comme ça, en plus, si tu exportes ton appli en jar, tu n'auras pas de problèmes d'emplacement de fichiers et tu pourras les intégrer dans le jar } catch (IOException e) { e.printStackTrace(); } return null; }   private void init () { int nombreAleatoire = 1 + (int)(Math.random() * ((12))); int nombreAleatoire2 = 1 + (int)(Math.random() * ((12)));   final JPanel panneau1=new Panneau(readImage("images/images/acides_carboxyliques/"+nombreAleatoire+".png")); final JPanel panneau2=new Panneau(readImage("images/images/alcanes/"+nombreAleatoire2+".png")); ActionListener listener = new ActionListener (){ // <- accolade de début de classe anonyme private int pan = 0; @Override public void actionPerformed(ActionEvent evt) { switch(pan++% 2){ case (0):Fenetre.this.setContentPane(panneau1); Fenetre.this.validate(); break; case (1):Fenetre.this.setContentPane(panneau2); Fenetre.this.validate(); break; } } }; Timer timer = new Timer (2000, listener); timer.setInitialDelay(0); // on a fait un max de choses avant d'ouvrir/afficher la fenêtre     this.setTitle("Série d'images"); this.setSize(500, 500); this.setLocationRelativeTo(null); this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);   // enfin ça aussi, on peut considérer que ça fait partie des max de choses à faire avant d'ouvrir/afficher la fenêtre   this.setContentPane(panneau1); // afficher par défaut l'image correspondant à 1%2==0 (donc la fenêtre n'apparaitra jamais "grise") // si pas setInitialDelay(0) this.setContentPane(panneau2); // afficher par défaut l'image correspondant ) à 1%2==1   this.setVisible(true); timer.start ();   }

---

Quelqu'un a une idée ?

Bonjour,
En fait j'ai réglé ce problème : il fallait rajouter super.paintComponent(g) dans ma classe Panneau, pour que les images se dessinent correctement.