Discussion :

[pendoRa]

Bonjour,

je m'explique, dans le cadre de développements de petits projets techniques informatiques, j'ai commencé il y a quelques mois 2 applications Java.

Je souhaite optimiser 2 partie du code, pour un gain de clarté et éviter la redondance étant donné que l'application est de type procédurale.

Je commence avec les boutons, j'ai 21 JButtons, il sont tous déclarés tel quels :

Code :

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

1
2
// (final pour certains)
JButton bouton0 	= new JButton(active);  	bouton0.setText("0");

Ils sont tous codés a la suite les uns des autres, et on ma fortement conseillé d'optimiser ce bout de code. Hélas j'ai de multiples erreurs lorsque que je fais par exemple

Code :

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

JButton bouton0, bouton1, bouton(n..) = new JButton();

Bien entendu, cela ce passe aussi pour les 2 menus et 5 menuItem ...



Pour la deuxième partie je la mentionnerais plus tard.

Merci pour votre aide.

[herve91]

Passe par un tableau ou une collection :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
JButton[] buttons;
 
public void initButtons(String... texts) {
   buttons = new JButton[texts.length];
   int i = 0;
 
   for (String text : texts) {
      JButton button = new JButton(text);
      buttons[i++] = button;
   }
}

[pendoRa]

Merci de ta réponse,
j'ai procéde de la sorte :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
		JButton [] caract = new JButton[10];
        for(int i = 0; i < caract.length; i++){
            caract[i]= new JButton (""+i);
            caract[i].setName("bouton" + i);
            caract[i].addActionListener(new ActionListener() {
                public void actionPerformed(ActionEvent e) {
                    JButton b = (JButton)e.getSource();
                    System.out.println("bouton "+b.getName());
                }
            });
            panel.add(caract[i]);
         }
        setContentPane(panel);

Cependant ( deuxieme soucis d'optimisation ), les boutons sont placés via un GridBagLayout, comme ceci : ( exemple du bouton1)

Code :

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

1
2
3
4
5
6
7
8
9
// # 1
	     gbl.gridx = 0;
	     gbl.gridy = 3;
	     gbl.gridwidth = 1;
	     gbl.gridheight = 1;
	     gbl.fill = GridBagConstraints.HORIZONTAL;
	    	panel.add(bouton1, gbl); 
	    	bouton1.setBackground(Color.black);
	     	bouton1.setForeground(Color.white);

Donc, il y a les coordonées puis l'ajout sur le panel. Or, avec un tableau, il ne reconnait pas encore le bouton comme bouton1 étant donné que le process n'est pas encore enclenché. Je ne vois pas comment procéder.
J'ai essayé "b.getName()" a la place du nom du bouton ( bouton1 par exemple ), en vain, pourtant b.getName(); retourne le numéro respectif du bouton.


edit : j'avai essayé ceci :

Code :

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

1
2
3
4
5
6
7
8
    gbl.gridx = 0;
	     gbl.gridy = 3;
	     gbl.gridwidth = 1;
	     gbl.gridheight = 1;
	     gbl.fill = GridBagConstraints.HORIZONTAL;
	    	panel.add(caract[i], gbl); 
	    	caract[i].setBackground(Color.black);
	     	caract[i].setForeground(Color.white);

Merci

[herve91]

Code :

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

1
2
... // gbl = ...
panel.add(caract[i], gbl);

[pendoRa]

Merci, hélas les boutons se placent a la suite horizontalement, il ne sont pas placé via le gbl...

[pendoRa]

Voici mes 2 boucles pour les 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
19
20
21
22
23
24
// Boucle de création des boutons Delta, Graph, +, -, *, %, C, ON,x^, ,...
		final JButton[] tabButton = new JButton[12] ;
		String[] tabButtonLabels = {"C", ",", "%", "Graph", "Delta", "+", "-"
				, "/", "*", "x^", "ON", "="};
		for (int j=0 ; j<tabButton.length ; j++)
	     tabButton[j] = new JButton(tabButtonLabels[j]) ;
 
		// Boucle de création des boutons "chiffre"
		final JButton [] caractPdt = new JButton[10]; // On veut 10 boutons
        for(int i = 0; i < caractPdt.length; i++){
            caractPdt[i]= new JButton (""+i);
            caractPdt[i].setName("bouton" + i);
            // on place apres chaque bouton son chiffre (i)
                    // incrémenté 
            caractPdt[i].addActionListener(new ActionListener() {
                public void actionPerformed(ActionEvent e) {
                    JButton b = (JButton)e.getSource();
                    System.out.println("bouton "+b.getName());
 
                }
            });
          panel.add(caractPdt[i]);
         }
        setContentPane(panel);

Mon dernier soucis est de savoir comment supprimé ces paragraphe de code

Code :

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

1
2
3
4
5
                      gbl.gridx = 1;
		     gbl.gridy = 5;
		     gbl.gridwidth = 1;
		     gbl.gridheight = 1;
		     	panel.add(caractPdt[0], gbl);

sachant que chaque coordonnées est differentes pour un bon placement de chaque composant. On m'a conseillé une boucle, mais les valeurs de gridy et gridx sont differentes pour chaque composants. Ce qui signifie que j'ai une vingtaine de petit paragraphe tel que celui-ci...

Merci

Pour info, voici le cadrillage pour le placement de mes composants.
Pièce jointe 68937