Discussion :

[lemath31]

Bonjour,

Je réalise un petit "atelier maison" qui consiste à envoyer des instructions à des équipements réseau. J'assure la partie communication SSH avec un coup de Python qui sera appelé plus tard (mais là n'est pas le souci, c'est juste pour le contexte :p ).

Afin de rendre plus convivial cet environnement, j'ai souhaité développer une interface en Java. Je me suis appuyé sur Swing.
La situation est la suivante :

• J'ai un fichier contenant les information sur les équipements réseaux ~ adresse et mdp (que le lis)
• Je souhaite créer une ligne dans mon interface par équipement contenu dans le fichier
• chaque ligne doit être assortie d'une checkbox afin d'en permettre la selection (et donc d'un listener associé)

j'arrive à mettre ça en place "graphiquement" parlant mais pas "fonctionnellement". Je m'explique : Je réalise une boucle pour la création de tous ces boutons.

Code :

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

1
2
3
4
5
6
		    for(int i = 1; i<= MonFichier.getLastRowNum(); i++){
 
		    	MaCheckBox = new CheckBox(HostnameList.get(i).toString());
		    	MainPanel.add(MaCheckBox );
 
}

Ma classe CheckBox est on ne peut plus simpliste :

Code :

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

1
2
3
4
5
6
7
8
public class CheckBox extends JCheckBox {
	private String name;
	public CheckBox(String name) {
		this.name = name;
		this.setName(name);
 
	}
}

Mon souci : En procédant de la sorte, chaque CheckBox est "écrasée" à chaque occurrence de la boucle. du coup, je n'arrive pas à obtenir d'ID unique celles ci. Du coup impossible d'y associer un listener spécifique avec lequel travailler. N'arrivant pas à créer de nom dynamiquement pour mes CheckBox je me retrouve bien embêté !

j'ai essayé d'implémenter une Factory mais ça ne semble pas pour autant solver ce souci d'intégrité.

j'ai parcouru les différentes sections du forum sans pour autant trouver de cas similaire (ou alors j'ai mal cherché ce qui n'est pas exclu !).

Du coup, ma question : est ce que j'emploie la bonne méthode pour réaliser ce que je souhaite faire ou suis-je dans le faux ?

J'ajoute que je ne suis pas une bête de compet' non plus en Java et que si mes notions peuvent paraîtres sommaires, elles sont ce qu'elles sont !

Je suis ouvert à toute question spécifique sur le sujet !

Merci d'avance !

[joel.drigo]

Salut,

Mon souci : En procédant de la sorte, chaque CheckBox est "écrasée" à chaque occurrence de la boucle. du coup, je n'arrive pas à obtenir d'ID unique celles ci. Du coup impossible d'y associer un listener spécifique avec lequel travailler. N'arrivant pas à créer de nom dynamiquement pour mes CheckBox je me retrouve bien embêté !

Déjà il faudrait préciser ce que tu entends par "écrasée", parce que tu ne montres pas beaucoup de code, et ça peut dire pas mal de choses différentes.

Ensuite, attention, le name d'un composant n'est pas forcément une notion d'id, au sens "identifiant unique de composant". Encore moins un libellé comme ça semble être le cas vu ton code. Il vaut mieux d'ailleurs ne pas l'utiliser, car Swing peut s'en servir de lui même dans certains cas. La référence de l'objet constitue déjà un identifiant, et l'avantage c'est qu'on a accès directement aux méthodes, donc pas d'intermédiaire inutile. Il n'est par ailleurs pas indispensable de faire une classe, mais bon, pourquoi pas. Il n'est pas non plus indispensable d'avoir un écouteur pour une checkbox, à moins que l'interface puisse être modifiée en fonction de la sélection.

Un exemple rapide, avec le fichier suivant :

cb1;text 1
cb2;text 2
cb3;text 3

Sur chaque ligne donc, un id et un texte, séparé par un point virgule (c'est un exemple, on peut générer automatiquement des ids aussi on incrémentant un compteur, ou en utilisant une information particulière).

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
 
public class Exemple1 {
 
	public static void main(String[] args) throws IOException {
 
		Path path = Paths.get("file.txt");
 
 
		JFrame frame = new JFrame();
 
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
 
		Box checkboxPanel = new Box(BoxLayout.Y_AXIS); // j'utilise un layout en box verticale
 
		Map<String, JCheckBox> checkboxes = new HashMap<>(); // je stocke mes checkboxes par id
		for(String line : Files.readAllLines(path)) { // je lis le fichier ligne à ligne
			String[] data = line.split(";"); // je divise chaque ligne en deux 
			JCheckBox checkbox = new JCheckBox(data[1]); // je crée une checkbox avec pour texte la deuxième information
			checkboxes.put(data[0], checkbox); // je stocke ma checkbox avec son identifiant
			checkboxPanel.add(checkbox); // je place ma checkbox dans l'affichage
                        // un exemple d'écouteur pour réagir au clic
			checkbox.addActionListener(e->System.out.format("La checkbox %s a été cliquée et sa valeur est %b%n",data[0],checkbox.isSelected()));
		}
 
 
		frame.add(new JScrollPane(checkboxPanel)); // je mets mon panel de checkbox dans une fenêtre
 
                // je créé un bouton
		JButton button = new JButton("Afficher les boites sélectionnées"); 
                // j'affecte un écouteur au bouton qui va appeler la méthode afficher
		button.addActionListener(e-> afficher(checkboxes));
 
                // je place le bouton en bas de la fenêtre
		frame.add(button,BorderLayout.SOUTH);
 
		frame.setSize(300, 300);
		frame.setLocationRelativeTo(null);
		frame.setVisible(true);
 
 
	}
 
	private static void afficher(Map<String, JCheckBox> checkboxes) {
 
                 // je construis mon affichage		
		StringBuilder sb = new StringBuilder();
 
		sb.append("Les checkboxes sélectionnées sont :\n");
		checkboxes.entrySet().stream().filter(e-> e.getValue().isSelected())
		          .forEach(e-> sb.append("- ").append(e.getKey()).append(" / ").append(e.getValue().getText()).append("\n"));
 
		sb.append("\n");
		sb.append("Les checkboxes non sélectionnées sont :\n");
		checkboxes.entrySet().stream().filter(e-> !e.getValue().isSelected())
					.forEach(e-> sb.append("- ").append(e.getKey()).append(" / ").append(e.getValue().getText()).append("\n"));
 
		JOptionPane.showMessageDialog(null, sb);
 
	}
 
}

[tchize_]

A noter qu'il serait peut être plus pertinent d'utiliser un JTable pour faire un listing

En plus, ça tombe bien, la doc sur la JTable explique comment y mettre des checkbox :p

[lemath31]

Mhmm voila une approche à laquelle je n'avais pas songée...

Tu utilises une map, j'avais raisonné de la même manière mais ne m'en était pas sorti !

J'ai tendance à ne pas utiliser les bons termes technique ce qui a le don de me jouer des tours de temps à autres, il faut que je reconsidère ça !

Je vais essayer une solution proche de celle que tu proposes, je ferai un retour avec un code un peu plus béton si tout fonctionne comme je l'entends !

Merci beaucoup pour votre temps !

[lemath31]

j'ai essayé plusieurs autres solutions, utilsant map, Array et bien d'autres, finalement j'ai dû me retourner vers une version proche de celle proposée par Joël qui fonctionne très bien (extrait du code):

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
 
static Map<String, JCheckBox> checkboxes = new HashMap<>();
 
//Here - push is my Jmenu calling SelectOptions method
Push.addActionListener(e-> SelectOptions(checkboxes));
 
for(int i = 1; i<= PWDSheet.getLastRowNum(); i++){
 
                        // HostnameList, AddressList are ArrayLists
		    	CheckBox checkbox = new CheckBox(list1.get(i).toString()+ " " +list2.get(i).toString());
		    	checkboxes.put((list1.get(i).toString()), checkbox);  
		    	MainPanel.add(checkbox);
}
 
private  void SelectOptions(Map<String, JCheckBox> checkboxes) {
		this.checkboxes = checkboxes;
		//Building display
		StringBuilder sb = new StringBuilder();
                //New Windows (fileChooser)
		SelectionFrame selectionFrame;
		//for each selected boxes, return Key as String
		sb.append("Selected devices :\n");
		checkboxes.entrySet().stream().filter(e-> e.getValue().isSelected()).forEach(e-> sb.append("- ").append(e.getKey()).append("\n"));
		sb.append("\n");
		// Call new selection window
		selectionFrame = new SelectionFrame();
		JOptionPane.showMessageDialog(null, sb);
				}

Encore merci à Joël pour le tuyau !