Bouton colorant le background

Bonjour à tous,

je suis novice en JAVA, et je me suis interessé recemment sur les interfaces graphiques.
J'essaie de construire une frame avec deux boutons, qui permettent de changer la couleur du background.
Au final, la fenêtre s'affiche avec les boutons mais ne changent pas la couleur lors du clique.
Après avoir lu plusieurs tutos, je ne vois toujours pas d'où vient le problème. J'aurai donc besoin de vos lumières pour m'éclaircir sur ce sujet si possible.
Je vous remercie d'avance.

Voici mon code:

Citation:

---

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;

public class Fenetre extends JFrame implements ActionListener{

public static void main(String [] args){
Fenetre MaFenetre= new Fenetre();
MaFenetre.setVisible(true);
}

private JButton buttonVert;
private JButton buttonRouge;
private JLabel label;


// Construction de la fenetre avec les deux bouttons
public Fenetre(){
super("Ma fenetre Couleur");
Container pane=this.getContentPane();
pane.setBackground(Color.WHITE);

this.label= new JLabel("Je change de couleur");
pane.add(this.label,BorderLayout.NORTH);

this.buttonVert=new JButton("Vert");
this.buttonRouge=new JButton("Rouge");

pane.add(this.buttonVert,BorderLayout.WEST);
buttonVert.addActionListener(this);
buttonVert.setActionCommand("action1");

pane.add(this.buttonRouge,BorderLayout.EAST);
buttonRouge.addActionListener(this);
buttonRouge.setActionCommand("action2");

this.setSize(400,300);
this.setDefaultCloseOperation (JFrame.EXIT_ON_CLOSE);

}

public void actionPerformed(ActionEvent evt){
if("action1".equals(evt.getActionCommand()))
this.setBackground(Color.GREEN);
else
this.setBackground(Color.RED);


}

---

Cordialement,
kenchu.

Je suis pas expert du tout en IHM JAVA mais il me semble que quand tu fais

Code:

---

1 2 3

Container pane=this.getContentPane(); pane.setBackground(Color.WHITE);

---

tu colore en blanc le panel qui recouvre ta JFrame.
Tu devrait peut être essayer de changer la couleur de ce panel plutot

Code:

---

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

public void actionPerformed(ActionEvent evt){ if("action1".equals(evt.getActionCommand())) { Container pane=this.getContentPane(); pane.setBackground(Color.GREEN); } else ....     }

---

En espérant t'aider

Merci guigui5931 d'avoir pris le temps de lire mon post.

J'ai essayé plusieurs façons

1- Sans le else : Ca marche, bien evidemment le deuxième bouton ne change pas la couleur

Citation:

---

public void actionPerformed(ActionEvent evt){
if("action1".equals(evt.getActionCommand()))
{
Container pane=this.getContentPane();
pane.setBackground(Color.GREEN);
}

---

2- Avec le else : ca ne marche pas, il y a une erreur " panel1 cannot be resolved ".

Citation:

---

public void actionPerformed(ActionEvent evt){
if("action1".equals(evt.getActionCommand()))
{Container panel1=this.getContentPane();
panel1.setBackground(Color.GREEN);
}
else{
panel1.setBackground(Color.RED);
}

---

3- Avec le else et en utilisant un autre panel : ca marche très bien.

Citation:

---

public void actionPerformed(ActionEvent evt){
if("action1".equals(evt.getActionCommand()))
{Container panel1=this.getContentPane();
panel1.setBackground(Color.GREEN);
}
else{
Container panel2=this.getContentPane();
panel2.setBackground(Color.RED);
}

---

Par contre, je ne comprends pas pourquoi il faut utiliser un autre container/panel que celui d'origine pour changer la couleur du background.
Au total il y a 3 containers/panels " pane, panel1, panel2" .

En fait il n'y a pas 3 panel différent il n'y en as qu'un. Je m'explique (ou du moins j'essaye). La JFrame A un objet JPanel que tu récupère en faisant this.getContentPane();
Quand tu fait panel1=this.getContentPane(); tu donne à panel1 une référence vers le panel de ta JFrame. Donc tes pane, panel1, panel2 sont uniquement des références vers le panel de le JFrame.
Pour ce qui est de ton point 2 il ne trouve pas panel1 car il n'est pas déclaré dans le même bloc il devrait être écris comme ça

Code:

---

1 2 3 4 5 6 7 8 9 10

public void actionPerformed(ActionEvent evt){ Container panel1=this.getContentPane(); if("action1".equals(evt.getActionCommand())) { panel1.setBackground(Color.GREEN); } else{ panel1.setBackground(Color.RED); }

---

J'espère avoir été clair.

Tout à fait, ca marche très bien. J'ai du faire une mauvaise frappe, car j'ai eu une exception dans le main.

Merci beaucoup pour ton aide guigui5931.

Dans ce cas n'oublie pas pas de cliquer sur le petit bouton :resolu: en bas de la discussion.