Problème avec ActionPerformed

**gribou1020** · 26/04/2017, 07h06

Bonjour à tous,

Je débute en java et je fais des essais sur les composants... et j'ai un problème au niveau de la méthode ActionPerformed.
Le système met une erreur au niveau de b1 et b2..(dans la méthode).

Une petite aide ??
Merci

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
public class ExFlowLayout extends JFrame implements ActionListener {
 
 
 
 
	public ExFlowLayout()
	  {
		SimpleDateFormat dateFormat = new SimpleDateFormat("dd-MM-YYYY");
		System.out.println(dateFormat.format(new Date()));
 
 
		this.setTitle("Interface Graphique");
 
		setLayout(null);
		setSize(600,400);
		setResizable(false);
 
		JButton b1=new JButton("TEST 1");
		b1.setSize(125,25);
		b1.setLocation(175,10);
		b1.setBackground(Color.gray);
		b1.addActionListener(this);
		add(b1);
 
 
		JButton b2=new JButton("TEST 2");		
		b2.setSize(125,25);
		b2.setLocation(175,100);
		b2.addActionListener(this);
		add(b2);
 
 
		JLabel l1=new JLabel("Francis Sluiters");
		l1.setSize(125,25);
		l1.setLocation(0,0);
		add(l1);
 
 
		JTextField tf=new JTextField("Bonjour...nous sommes le "+dateFormat.format(new Date()));
		tf.setLocation(0,50);
		tf.setSize(250,30);
		tf.setBackground(Color.orange);
		tf.setBorder(null);
	//	tf.setCursor(getCursor());
		add(tf);
 
 
		String[] liste={"orange","pommes","poires"};
		JList jl=new JList(liste);
		JScrollPane jsp=new JScrollPane(jl,ScrollPaneConstants.VERTICAL_SCROLLBAR_ALWAYS,ScrollPaneConstants.HORIZONTAL_SCROLLBAR_AS_NEEDED);
		jsp.setLocation(300,150);
		jsp.setSize(150,150);
		add(jsp);
 
 
		JPanel p=new JPanel();	
		p.setBounds(150,150,120,80);
		p.setBackground(Color.BLUE);	
		add(p);
 
	 }
 
	public void actionPerformed(ActionEvent ev)
	  {
		 Object  source=ev.getSource();	
 
		if(source==b1)		
		     System.out.println(".....");
		else			
	    if(source==b2)	
			 System.out.println("....");		
	   }
 
 
	public static void main(String[] args)
	 {
	    ExFlowLayout f=new ExFlowLayout();
	    f.setLocationRelativeTo(null);
	    f.setVisible(true);
 
	 }
 
}

**Népomucène** · 26/04/2017, 10h57

Bonjour
ça coince à cet endroit :

Code :

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

1
2
3
4
5
6
7
8
9
10
	public void actionPerformed(ActionEvent ev)
	  {
		 Object  source=ev.getSource();	
 
		if(source==b1)		
		     System.out.println(".....");
		else			
	    if(source==b2)	
			 System.out.println("....");		
	   }

En effet, b1 et b2 ont été déclaré dans le constructeur ExFlowLayout() et donc ils n'existent que dans le constructeur.
Du coup, ils sont inconnus dans actionPerformed.

Solution simple :
1) ajouter la déclaration des champs dans la classe (en dehors du constructeur donc) :

Code :

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

1
2
3
 
    private JButton b1;
    private JButton b2;

2) modifier le code du constructeur en supprimant la déclaration des boutons:

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
 
		......
 
		b1=new JButton("TEST 1");
		b1.setSize(125,25);
		b1.setLocation(175,10);
		b1.setBackground(Color.gray);
		b1.addActionListener(this);
		add(b1);
 
 
		b2=new JButton("TEST 2");		
		b2.setSize(125,25);
		b2.setLocation(175,100);
		b2.addActionListener(this);
		add(b2);
 
		......

au final, tu dois avoir le code suivant :

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
public class ExFlowLayout extends JFrame implements ActionListener {
 
    private JButton b1;
    private JButton b2;
 
    public ExFlowLayout() {
        SimpleDateFormat dateFormat = new SimpleDateFormat("dd-MM-YYYY");
        System.out.println(dateFormat.format(new Date()));
 
 
        this.setTitle("Interface Graphique");
 
        setLayout(null);
        setSize(600, 400);
        setResizable(false);
 
        b1 = new JButton("TEST 1");
        b1.setSize(125, 25);
        b1.setLocation(175, 10);
        b1.setBackground(Color.gray);
        b1.addActionListener(this);
        add(b1);
 
 
        b2 = new JButton("TEST 2");
        b2.setSize(125, 25);
        b2.setLocation(175, 100);
        b2.addActionListener(this);
        add(b2);
 
 
        JLabel l1 = new JLabel("Francis Sluiters");
        l1.setSize(125, 25);
        l1.setLocation(0, 0);
        add(l1);
 
 
        JTextField tf = new JTextField("Bonjour...nous sommes le " + dateFormat.format(new Date()));
        tf.setLocation(0, 50);
        tf.setSize(250, 30);
        tf.setBackground(Color.orange);
        tf.setBorder(null);
        //	tf.setCursor(getCursor());
        add(tf);
 
 
        String[] liste = {"orange", "pommes", "poires"};
        JList jl = new JList(liste);
        JScrollPane jsp = new JScrollPane(jl, ScrollPaneConstants.VERTICAL_SCROLLBAR_ALWAYS, ScrollPaneConstants.HORIZONTAL_SCROLLBAR_AS_NEEDED);
        jsp.setLocation(300, 150);
        jsp.setSize(150, 150);
        add(jsp);
 
 
        JPanel p = new JPanel();
        p.setBounds(150, 150, 120, 80);
        p.setBackground(Color.BLUE);
        add(p);
 
    }
 
    public void actionPerformed(ActionEvent ev) {
        Object source = ev.getSource();
 
        if (source == b1)
            System.out.println(".....");
        else if (source == b2)
            System.out.println("....");
    }
 
    public static void main(String[] args) {
        ExFlowLayout f = new ExFlowLayout();
        f.setLocationRelativeTo(null);
        f.setVisible(true);
 
    }
}

**wax78** · 26/04/2017, 10h58

Tes variable b1 et b2 n'existent qu'en local dans le constructeur donc elle ne sont pas visible dans actionPerfomed().

Crée 2 variable dans la classe ExFlowLayout pour ces 2 boutons ça ira mieux je pense.

**joel.drigo** · 26/04/2017, 11h39

Salut,

Il y pleins d'autres solutions qui permettent de ne pas avoir à utiliser les instances de JButton, ce qui peut permettre de ne pas être dépendant des composants de UI (un bouton pourra être remplacé par un menu, ou son action déclenchée par un timer, etc...)

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
public class ExempleButton extends JFrame {
 
	public static final String COMMAND_TRUC = "TRUC";
	public static final String COMMAND_MACHIN = "MACHIN";
 
	public ExempleButton() {
 
		JPanel panel = new JPanel(new GridBagLayout());
		add(panel);
 
		GridBagConstraints gbc = new GridBagConstraints();
		gbc.gridx=0;
		gbc.insets=new Insets(2, 2, 2, 2);
 
		// Utiliser un action listener propre au bouton (qui peut d'ailleurs être une classe externe, qui aurait difficilement accès aux attributs
		JButton button1 = new JButton("Exemple 1");
		button1.addActionListener(new ActionListener() {
			@Override
			public void actionPerformed(ActionEvent e) {
				affiche("Exemple 1");
			}
		});
		panel.add(button1,gbc); 
 
		// Utiliser un action listener propre au bouton, par expression lambda
		JButton button2 = new JButton("Exemple 2");
		button2.addActionListener(e-> affiche("Exemple 2")); 
		panel.add(button2,gbc);
 
		// Utiliser la commande du bouton, ce qui permet de rendre plus indépendant l'UI des fonctionnalités propres de l'application (avec différents controleurs, les mêmes commandes peuvent faire des choses différentes)
		ActionListener commandActionListener = new ActionListener() {
			@Override
			public void actionPerformed(ActionEvent e) {
				switch( e.getActionCommand() ) {
				case ExempleButton.COMMAND_TRUC:
					affiche("Exemple 3");
					break;
				case ExempleButton.COMMAND_MACHIN:
					affiche("Exemple 4");
					break;
				}
			}
		};
		JButton button3 = new JButton("Exemple 3");
		button3.addActionListener(commandActionListener);
		button3.setActionCommand(ExempleButton.COMMAND_TRUC);
		panel.add(button3,gbc);
		JButton button4 = new JButton("Exemple 4");
		button4.addActionListener(commandActionListener);
		button4.setActionCommand(ExempleButton.COMMAND_MACHIN);
		panel.add(button4,gbc);
 
		// Utiliser des actions, voire un ActionManager, ce qui permet de gérer les actions à part, de pouvoir les réutiliser dans différents composants (dans des menus par exemple)
		List<Action> actions = Arrays.asList(
				new Action("Exemple 5", "Exemple 5"),
				new Action("Exemple 6", "Exemple 6")
				);
 
		actions.forEach(action-> panel.add(new JButton(action),gbc));
 
		pack();
		setLocationRelativeTo(null);
 
	}
 
	private void affiche(String message) {
		JOptionPane.showMessageDialog(this, message);
	}
 
	public class Action extends AbstractAction {
 
		private final String message;
 
		public Action(String text, String message) {
			super(text);
			this.message=message;
		}
 
		@Override
		public void actionPerformed(ActionEvent e) {
			affiche(message);
		}
 
	} 
 
	public static void main(String[] args) {
		new ExempleButton().setVisible(true);
	}
 
}

Problème avec ActionPerformed

AWT/Swing Java

Vue hybride

Discussions similaires

Partager

Partager