Discussion :

[dissen0d]

Bonjour,

Je me remets sur Java, parcque jusqu'à présent j'ai réussi à tout faire en autoit, mais j'atteint un peu les limites alors je me remets sur un langage un peu plus "noble" (sans vouloir manquer de respect à qui que ce soit..).

Le code suivant fonctionne parfaitement, mais dès que je veux mettre la 2ème classe (CheckIPAddress) dans une autre classe, sous Eclipse, ce dernier me dit que combo ne peut être résolu...

Une idée?

Merci d'avance

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
import javax.swing.JFrame;
import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JButton;
import javax.swing.JComboBox;
import javax.swing.JPanel;
import javax.swing.JLabel;
 
public class MainWindow extends JFrame{
 
	private static final long serialVersionUID = 1L;
	private JPanel container = new JPanel();
	public JLabel label = new JLabel("Choose or type you IP address");
	private static JComboBox<?> combo = new JComboBox<Object>();
	private JButton check = new JButton("Check");
 
	public MainWindow() {
		this.setBounds(100, 100, 500, 400);
		this.setTitle("Bobst Synchronisation Software");
	    this.setTitle("Bobst Synchronisation Software");
	    this.setSize(600, 300);
	    this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
	    this.setLocationRelativeTo(null);
	    container.setBackground(Color.white);
	    container.setLayout(new BorderLayout());
	    combo.setPreferredSize(new Dimension(130, 20));
	    combo.setEditable(true);
 
	    JPanel top = new JPanel();
	    top.add(label);
	    top.add(combo);
	    top.add(check);
	    container.add(top, BorderLayout.NORTH);
	    this.setContentPane(container);
	    this.setVisible(true);
	    check.addActionListener(new CheckIPAddress());
	}
 
	public class CheckIPAddress implements ActionListener{
 
	    public  void    actionPerformed(ActionEvent e)
	    {
			System.out.println(combo.getSelectedItem().toString());
	    }
	}
 
	public static void main(String[] args) {
		new MainWindow();
	}
}

[joel.drigo]

Salut,

Si la class CheckIPAddress est définie dans son propre fichier, les variables membres d'une autre classe.ne lui sont pas accessibles autrement que par une instance de cette autre classe, ou par la classe si la variable est static.

Dans le cas de ton source, la classe CheckIPAddress n'est pas dans un autre fichier : c'est une classe interne de la classe MainWindow, donc elle peut accèder à la variable directement.

Il n'y a aucune raison que la variable combo soit static. C'est même potentiellement problématique. Fonctionnellement, à priori, il ne devrait y avoir qu'une seule instance de MainWindow, mais techniquement, il est possible d'en avoir plusieurs, et là, avec combo static, l'application risque de ne plus fonctionner correctement (en l'occurence, seule la dernière fenêtre créée affichera la combo). De manière générale, les variables qui référencent des composants graphique Swing ne doivent jamais être static. Tu peux même de manière générale considérer qu'il faut éviter d'avoir des variables static : cela ne sert que dans des cas particuliers. Si tu es tenté d'en utiliser une (ou que Eclipse te suggère de rendre static une variable) et que tu ne sais pas pourquoi, alors il y a de fortes chances qu'elle ne devrait pas être static.

Alors comment faire, finalement, pour pouvoir manipuler combo dans CheckIPAddress lorsqu'il ne s'agit pas d'une classe interne. En passant la référence en paramètre du constructeur :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public class CheckIPAddress implements ActionListener{
 
     private JComboBox combo;
     public CheckIPAddress(JComboBox combo) {
             this.combo=combo;
     }
 
     public  void    actionPerformed(ActionEvent e) {
          System.out.println(combo.getSelectedItem().toString());
     }
 
}

Et à la construction :

Code :

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

check.addActionListener(new CheckIPAddress(combo));

[dissen0d]

Merci Beaucoup Joel.

Non seulement j'ai résolu mon problème, mais en plus, j'ai appris de bonnes pratiques.

Bon week-end.

David