récuperer vector dans liste pour combobox

**BizuR** · 01/08/2006, 11h44

Pour la mise à jour de la liste, c'est pas bien dur :

Dans ton listener de combobox :

Code :

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

1
2
3
4
5
6
7
public void onStateChanged(...){
     if (jcombobox.getSelectedItem().equals("nouvelle entreprise")){
          ajoutEntreprise(); //methode qui ajoute une nouvelle entreprise
          MAJList();
     }
(...autres cas...)
}

Et ton MAJList()

Code :

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

1
2
3
4
MAJList(){
     jcombobox.removeAllItem();
     jcombobox.setModel(new DefaultComboBoxModel(vectorIHM);
}

Et voila, le tour est joué

NB: Ne pas oublier d'appeler cette méthode lors de modification d'entreprise mais aussi de suppression

**bnreb10** · 01/08/2006, 11h54

voici mon listener:

code:
ComboBoxEntreprise.addItemListener(new java.awt.event.ItemListener()
{
public void itemStateChanged(java.awt.event.ItemEvent e)
{
if(e.getItem() == "Nouvelle Entreprise")
{
//FenAjoutEntrep.getDialogAjoutEntrep().setModal(true);
FenAjoutEntrep.getDialogAjoutEntrep().setVisible(true);
}

else if (e.getItem()!="Nouvelle Entreprise")
FenAjoutEntrep.getDialogAjoutEntrep().setVisible(false);
}

});
ComboBoxEntreprise.addActionListener(new java.awt.event.ActionListener()
{
public void actionPerformed(java.awt.event.ActionEvent e)
{
int index;
if (e.getSource()== ComboBoxEntreprise)
index = ComboBoxEntreprise.getSelectedIndex();
}
});

listeEntreprise = TableauEntreprise.getListeEntreprise();

for(TableauEntreprise taben : listeEntreprise)
{
ComboBoxEntreprise.addItem(taben.toString());
}

Il faut que j'ajoute ce que tu m'a dit ici?!

Merci encore bizur

**BizuR** · 01/08/2006, 14h07

Voici ton code formaté :

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
[...]//debut de la creation de ta combobox
ComboBoxEntreprise.addItemListener(new java.awt.event.ItemListener() {
     public void itemStateChanged(java.awt.event.ItemEvent e){
          if(e.getItem() == "Nouvelle Entreprise"){
               //FenAjoutEntrep.getDialogAjoutEntrep().setModal(true);
               FenAjoutEntrep.getDialogAjoutEntrep().setVisible(true);
               MAJList(); //appel à la mise à jour de la liste
          }
          else if (e.getItem()!="Nouvelle Entreprise")
               FenAjoutEntrep.getDialogAjoutEntrep().setVisible(false);
     }
});

ComboBoxEntreprise.addActionListener(new java.awt.event.ActionListener(){
     public void actionPerformed(java.awt.event.ActionEvent e){
          int index;
          //PAS COMPRIS L'INTERET DE CES LIGNES ;)
          if (e.getSource()== ComboBoxEntreprise)
               index = ComboBoxEntreprise.getSelectedIndex();
     }
});

MAJList();//appel a la construction de la liste
} // fin de la creation de ta combobox

public void MAJList(){
     Vector listeEntreprise = TableauEntreprise.getListeEntreprise();
     for(TableauEntreprise taben : listeEntreprise){
         ComboBoxEntreprise.addItem(taben.toString());
     }
}

**bnreb10** · 01/08/2006, 15h24

heu...

Merci Bizur mais ça marche tjrs pas

Je vais voir si je peux changer un truc et réussir a débogger, je te remercie de ton aide et quand j'aurai réussi je mettré "Résolu" lol

**bnreb10** · 01/08/2006, 15h29

en fait dans ma combobox ça me met (aprés la mise a jour) 2 fois "Nouvelle entreprise" et 2fois aussi l'item vide que j'ai crée

**bnreb10** · 01/08/2006, 15h30

Mais bien sûr tjrs pas la nouvelle entreprise

**BizuR** · 01/08/2006, 15h58

A vrai dire, personnellement, je comprend pas tout a fait ton TableauEntreprise.java, il me semble un poil incomplet ... m'enfin.

J'aurai plutot vu un truc comme cela quitte a suivre ta démarche :

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
public class Entreprise {
     //declaration du tableau qui liste toutes tes instances entreprises
     private static Vector listeEntreprises=new Vector();
     //attributs d'une entreprise
     private String nomEtp;
     [...]

     public Entreprise(_nomEtp){
          this.nomEtp=_nomEtp;
          Entreprise.ajouterEntreprise(this);
     }

     [...] //getters, setters et autres méthodes diverses

     //ajoute une entreprise à la liste static
     //permet de verifier les non redondances
     private static void ajouterEntreprise(Entreprise etp){
          [...] //verifications diverses
          listeEntreprises.add(etp);
     }

     //renvoie un clone de la liste fonctionnelle des entreprises
     public static Vector getListeEntreprises(){
          return listeEntreprises.clone();
     }

     //renvoie un vecteur temporaire des labels de la liste des entreprises
     //utile pour afficher dans JList ou JComboBox
     public static Vector getLabelsEntreprises(){
          Vector vecteurLabels=new Vector();
          //ajout des valeurs par défaut
          vecteurLabels.add("");
          vecteurLabels.add("Nouvelle entreprise");
          //ajout des entreprises créées
          for(int i=0;i<listeEntreprises.size();i++){
               vecteursLabels.add((Entreprise)listeEntreprises(i).getNomEntreprise());
          }
          return vecteurLabels;
     }
}

En espérant que ca pourrait t'aider...

**bnreb10** · 01/08/2006, 16h03

En fait j'ai appellé ma classe TableauEntreprise car j'ai déja une classe entreprise:
http://cjoint.com/?ibqc7bhuxL

enfin c'est juste pour comprendre

**bnreb10** · 01/08/2006, 16h04

heu... bon... je crois que je vais reprendre et essayer de faire comme tu m'a dit Bizur

**bnreb10** · 01/08/2006, 16h10

a y'être voici aussi la classe où je crée mon interface et la combobox (avec son listener) en question
(ligne 980 ~)

http://cjoint.com/?ibqligOSL6

**BizuR** · 01/08/2006, 16h23

Hum un peu de refactoring de code serait le bienvenu

Si j'étais toi, dans un souci de maintenant et d'eovlutivité, je découperai tout de même un peu plus encore mes classes afin d'affiner réellement le modèle et assurer une clarté de code plus intéressante.

Par exemple, pourquoi ne pas externaliser la barre de menu ? Quelques panels principaux ?
Tu peux aussi rassembler toutes tes opérations dans des fonctions plus génériques comme lockTextFields(boolean value) (qui met tout en editable ou non).

En bref, tu y serais gagnant en maintenance et les personnes qui reprendraient ton appli seront également gagnante pour comprendre plus vite l'ensemble de ton organisation de code.

NB: evidemment, cela tient plus du conseil que de l'ordre hein ... si tu t'y retrouves, c'est aussi bien pour toi, mais quand tu codes, essaie de penser surtout à ceux qui passent après

.

**bnreb10** · 01/08/2006, 16h26

oki merci mais ya pas longtemps on m'a dit que je faisais trop de classes alors j'ai essayer d'en faire moins et de mettre (pour l'interface par exple) tout ce qui se rapporte a l'interface a l'intérieur.
Desormais je séparer plus, promis....