Création dynamique de cases à cocher.

Version imprimable

Bonjour,

Sans doute cette question peut sembler étrange, mais si elle trouve une solution, cela me permettra d'alléger mon code de façon sensible.

J'ai défini un panneau JPanel. En fonction d'une liste d'articles qu'un utilisateur peut commander, je souhaite créer dans ce JPanel autant de cases à cocher qu'il y a d'articles (avec un maxi bien sûr). Pour cela j'avais pensé créer une méthode de ce genre :
Code:

1 2 3 4 5 6 7 8 9 10 private boolean CreerCheckBox(String nom, String texte, int x, int y) { javax.swing.JCheckBox ? = new javax.swing.JCheckBox(); ?.setName(nom); ?.setText(texte); ?.setBounds(x, 22, y, 22); ?.addActionListener...... panel.add(?); }
Évidemment, je me heurte au problème du nom de la checkbox (représenté ici par ?) . Y-a-t-il un moyen de créer dynamiquement son nom ?

Bien sûr je pourrais définir de façon statique n cases à cocher que je pourrais afficher ("setVisible(true)") selon qu'elles seraient active ou non, mais si je pouvais éviter j'aimerais beaucoup.

Merci de vos conseils avisés.
gégé

20/09/2010, 15h57
adiGuba

Salut,

Pourquoi as-tu besoin de paramétrer son nom ??? Il s'agit d'une variable locale à la méthode. Tu peux donc utiliser n'importe quel nom...

a++
20/09/2010, 16h01
akrom

Regarde du coté des ArrayList ici

Citation:

Regarde du coté des ArrayList ici

Code:

1
2
3
4
5
6
7
8
9
10
private boolean CreerCheckBox(String nom, String texte, int x, int y)
{
    javax.swing.JCheckBox   onSentBalance   = new javax.swing.JCheckBox();
    onSentBalance.setName(nom);//<--- inutile
    onSentBalance.setText(texte);
    onSentBalance.setBounds(x, 22, y, 22);
    onSentBalance.addActionListener......
    maList.add(onSentBalance);//<--- Ligne en plus : maList = ArrayList
    panel.add(onSentBalance);
}

Citation:

Envoyé par adiGuba

Salut,
Pourquoi as-tu besoin de paramétrer son nom ??? Il s'agit d'une variable locale à la méthode. Tu peux donc utiliser n'importe quel nom...
a++

D'abord merci

Oui j'ai essayé, mais cela ne marche pas. Seule la première case est ajoutée Sans doute une erreur de programmation. Voici mon code.
Merci pour les suggestions.
Code:

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 import javax.swing.JPanel; public class Courses extends javax.swing.JFrame { private static final long serialVersionUID = 1L; JPanel Panneau = null; public Courses() { super(); initialize(); } private void addCheckBox(String texte, int ligne, int colonne, javax.swing.JPanel pan) { javax.swing.JCheckBox x = new javax.swing.JCheckBox(); x.setName("Check"+ ligne + colonne); int posX = 18 + 40*ligne; int posY = 97 + 140*colonne; x.setText(texte); x.setBounds(posX, 22,posY, 22); pan.add(x); } private void initialize() { setName("Courses"); setDefaultCloseOperation(javax.swing.WindowConstants.DISPOSE_ON_CLOSE); setSize(800, 600); Panneau = new javax.swing.JPanel(); Panneau.setName("Panneau"); Panneau.setLayout(null); Panneau.setBounds(15, 69, 785, 423); addCheckBox("Hello AA", 1,1, Panneau ); addCheckBox("Hello BB", 1,2,Panneau); addCheckBox("Hello CC", 2,1,Panneau); this.add(Panneau); setVisible(true); } public static void main(java.lang.String[] args) { new Courses(); } }
Citation:

Envoyé par Akrom

Regarde du coté des ArrayList ici

Oui je connais, mais je ne vois pas comment implémenter mes cases dynamiquement à des positions différentes.

20/09/2010, 17h16
adiGuba

Les JCheckBoxs sont bien ajoutées au panel. Le problème vient du fait que tu les ajoutes les unes sur les autres avec des tailles bien trop grande (tu peux vérifier cela avec un setOpaque(false))

:arrow: Le layout null c'est le mal !!!!

Il serait préférable d'utiliser un layout adapté à ce que tu veux obtenir ;)

a++

PS : Pour rendre le code plus lisible, essayes de bien respecter les conventions de nommages ;)
20/09/2010, 19h18
GérardMartinelli

Citation:

Envoyé par adiGuba

Les JCheckBoxs sont bien ajoutées au panel. Le problème vient du fait que tu les ajoutes les unes sur les autres avec des tailles bien trop grande (tu peux vérifier cela avec un setOpaque(false))

:arrow: Le layout null c'est le mal !!!!

Il serait préférable d'utiliser un layout adapté à ce que tu veux obtenir ;)

a++

PS : Pour rendre le code plus lisible, essayes de bien respecter les conventions de nommages ;)

Merci, j'ai trouvé mon erreur. J'avais inversé y et "width" dans setBound.
Après avoir revu également l'algorithme de placement des cases, ça marche impec.

A propos de Layout null, c'est une excellente remarque. En utilisant GridLayout, j'ai fait en une instruction ce que je voulais faire, et ça marche.

Pour ce qui concerne les standards de nommage, je suis bien sûr entièrement d'accord. Mais le code précédent était juste un test écrit en quelques minutes (d'où mon erreur) Comme quoi il faut prendre son temps pour réfléchir. .

Merci encore.

Gégé

Citation:
Envoyé par Ivelios
+1
Code:

1 2 3 4 5 6 7 8 9 10 private boolean CreerCheckBox(String nom, String texte, int x, int y) { javax.swing.JCheckBox onSentBalance = new javax.swing.JCheckBox(); onSentBalance.setName(nom);//<--- inutile onSentBalance.setText(texte); onSentBalance.setBounds(x, 22, y, 22); onSentBalance.addActionListener...... maList.add(onSentBalance);//<--- Ligne en plus : maList = ArrayList panel.add(onSentBalance); }
Je ne vois pas ce que ArrayList ajoute au code. En particulier le lien entre le Panneau et cette table. Je dois sans doute être un peu bouché.
Merci quand même pour la réponse.

20/09/2010, 23h50
Ivelios

Pour ton cas, tu n'as en effet pas besoin de liste pour stocker tes CheckBox. J'aurais du regarder à deux fois :oops:
Par contre ça peut être pratique pour faire beaucoup d'ajout et beaucoup de suppression de CheckBox ( dynamique ) et ce sur plusieurs panel.
Il est alors préférable de tous les stocker dans une même liste.

Après, j'ai pris l'habitude (bonne ou mauvaise) de souvent utiliser les ArrayList pour stocker mes éléments de fenêtre. Voila pour les petits éclaircissement des idées qui traine dans mon cerveau :aie:
21/09/2010, 19h39
GérardMartinelli

Citation:

Envoyé par Ivelios

Pour ton cas, tu n'as en effet pas besoin de liste pour stocker tes CheckBox. J'aurais du regarder à deux fois :oops:
Par contre ça peut être pratique pour faire beaucoup d'ajout et beaucoup de suppression de CheckBox ( dynamique ) et ce sur plusieurs panel.
Il est alors préférable de tous les stocker dans une même liste.

Après, j'ai pris l'habitude (bonne ou mauvaise) de souvent utiliser les ArrayList pour stocker mes éléments de fenêtre. Voila pour les petits éclaircissement des idées qui traine dans mon cerveau :aie:

Merci, c'était déjà super sympa de me répondre.

Gégé