Discussion :

[adn013]

Bonjour,

J'ai codé sous Eclipse une JFrame qui contient juste une JComboBox et un JPanel qui instancie une classe DrawImage.

Je voudrais que, en fonction de l'item sélectionné dans la JComboBox,
le JPanel affiche instantanément une image différente.
J'ai bien utilisé la méthode itemStateChanged()
pour catcher le changement d'item et j'ai utilisé plusieurs moyens pour rafraichir la JFrame et le JPanel : repaint(), revalidate(), etc...
mais rien ne marche et le JPanel contient toujours la même image.

Mon programme contient 3 classes :
MaClasse.java : contient juste le main() qui instancie la JFrame avec le setvisible(true).
MaFenetre.java : code ci-dessous :

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
 
public class MaFenetre extends JFrame implements ItemListener
{
	private JComboBox combo;
	private JPanel panel;
	private String imagePath = "img/image1.jpg";
 
 
	public MaFenetre()
	{
		setSize(400,400);
		getContentPane().setLayout(new FlowLayout(FlowLayout.LEFT));
 
		panel = new DrawImage(imagePath,30,30);
		panel.setPreferredSize(new Dimension(300,300));
 
		combo = new JComboBox();
		combo.addItem("image1");
		combo.addItem("image2");
		combo.addItemListener(this);
 
		getContentPane().add(combo);
		getContentPane().add(panel);
	}
 
	public void itemStateChanged(ItemEvent evt) 
	{
		if(combo.getSelectedItem().equals("image1"))
		{			
			this.imagePath = "img/image1.jpg";
			panel = new DrawImage(imagePath,30,30);
			panel.revalidate();
			repaint();
			getContentPane().validate();				
		}
		else if(combo.getSelectedItem().equals("image2"))
		{		
			getContentPane().validate();	
			this.imagePath = "img/image2.jpg";	
			panel = new DrawImage(imagePath,30,30);
			panel.revalidate();
			repaint();		
			getContentPane().validate();							
		}	
		System.out.println(imagePath);
	}
}

DrawImage.java : code ci-dessous :

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
 
public class DrawImage extends JPanel
{
	private String path;
	private int width;
	private int height;
 
	public DrawImage(String path, int width, int height) 
	{
		this.path = path;
		this.width = width;
		this.height = height;
	}
 
	protected void paintComponent(Graphics g)
	{
		super.paintComponent(g);
		g.drawImage(Toolkit.getDefaultToolkit().getImage(this.path), this.width, this.height, this);
	}
}

Je vous remercie pour votre aide car je bloque depuis un moment.

a+

[xavlours]

Bonjour,

Dans la méthode itemStateChanged, tu crées bien un nouveau panel, mais tu ne l'ajoutes pas à ta JFrame ! Il faut faire ceci :

Code :

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

1
2
3
4
5
6
7
if(combo.getSelectedItem().equals("image1"))
		{			
			this.imagePath = "img/image1.jpg";
                        getContentPane().remove(panel); // on enlève l'ancien JPanel
			panel = new DrawImage(imagePath,30,30); // on crée le nouveau
			getContentPane().add(panel); // on l'ajoute à la JFrame
		}

Petite remarque : dans la classe DrawImage, tu lis le fichier à chaque actualisation de l'affichage, c'est mal. Il vaut mieux lire le fichier dans le constructeur une bonne fois pour toutes, puis stocker l'image en tant qu'attribut.

[adn013]

Bonjour,

merci beaucoup pour ta réponse.

En effet, j'ai oublié (comme un con) de rajouter le JPanel à la JFrame.
J'ai donc testé ton bout de code mais ça ne marchais toujours pas.
Puis j'ai finalement trouvé la solution :
il fallait repréciser la taille du nouveau JPanel et terminer par un panel.revalidate();

Voici la correction :

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
 
public void itemStateChanged(ItemEvent evt) 
{
	if(combo.getSelectedItem().equals("image1"))
	{		
		this.imagePath = "img/image1.jpg";
	}
	else if(combo.getSelectedItem().equals("image2"))
	{			
		this.imagePath = "img/image2.jpg";					
	}
 
	getContentPane().remove(panel);
	panel = new DrawImage(imagePath,30,30)
	panel.setPreferredSize(new Dimension(300,300));		
	getContentPane().add(panel);
	panel.revalidate();
}

Ta remarque à la fin était interessante mais je ne comprend pas comment l'implémenter.
Peux-tu m'écrire un morceau de code pour que je comprenne mieux.

Merci beaucoup.

[xavlours]

Simplement ceci :

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
public class DrawImage extends JPanel
{
	private Image image;
	private int width;
	private int height;
 
	public DrawImage(String path, int width, int height) 
	{
		// image = Toolkit.getDefaultToolkit().getImage(path);
                image = ImageIO.read(new URL(path)); // c'est mieux, il paraît.
		this.width = width;
		this.height = height;
	}
 
	protected void paintComponent(Graphics g)
	{
		super.paintComponent(g);
		g.drawImage(image, this.width, this.height, this);
	}
}

Là, quel que soit le nombre de réaffichages, le fichier ne sera lu qu'une seule fois. De plus, en cas d'IOException, tu n'en as qu'une seule (ce qui est quand même mieux) au lieu d'une par réaffichage.

[adn013]

Ok merci c cool.

a+