Remplir JComboBox à partir de List

Version imprimable

Bonjour les amis;
Je voudrais remplir ma JComboBox à partir d'une list. J'ai alors créer une class dans laquelle une ArrayList (list) est remplie à l'aide d'une requête sql (code 1).
j'ai mis en place une méthode (code 2) dans ma frame contenant le Combo et affecter cette list. Mais malheureusement, le combo n'est pas rempli à l'affichage de la frame au contraire je constate un gel du programme.

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
package ci.dataService;
 
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;
import java.util.ArrayList;
 
import ci.ressorces.FourEtNumero;
 
public class DAOFourEtNumero {
	private static BDConnectAS400 connexion = null;
 
	public DAOFourEtNumero(){
 
	}
 
	public static ArrayList<FourEtNumero> getlistFourEtNumero(){
		connexion = new BDConnectAS400();
		String requette  = "SELECT DISTINCT B33STFC.FTFOUL01.RSOCFO FROM B33STFC.FTFOUL01 ORDER BY RSOCFO ASC";
		Statement state;
		ResultSet res;
		ArrayList<FourEtNumero> list = new ArrayList<FourEtNumero>();
 
		try{
			state = connexion.getInstance().createStatement();
			res = state.executeQuery(requette);
			while(res.next()){
				FourEtNumero fourEtNum = new FourEtNumero();
				fourEtNum.setRSOCFO(res.getString(1));//raison sociale du fournisseur
				list.add(fourEtNum);
			}
			res.close();
			return list;
		}catch (SQLException e){
 
		}
 
		return null;
	}
}

Code:

1
2
public ArrayList<FourEtNumero> lister2 = null;  //  @jve:decl-index=0:
	public FourEtNumero tempo = null;

Code:

1
2
3
4
5
6
7
public void remplirFournisseur(){
		ArrayList<FourEtNumero> list = DAOFourEtNumero.getlistFourEtNumero();
		lister2 = list;
		for(int i=0;i<list.size();i++){
			ComboFournisseur.addItem(list);
		}
	}

Quelqu'un pourra-t-il me donner une piste pour résoudre l'erreur ? :oops:

09/07/2010, 17h50
thebloodyman
Salut,

Ici :
Code:

1 2 3 4 for(int i=0;i<list.size();i++){ ComboFournisseur.addItem(list); }
Tu rajoutes la liste à ta combo.
Tu devrais plutôt ajouter les éléments de ta liste (redéfini toString()de FourEtNumero si ce n'est fait).

tu peux utiliser le foreach java 5, c'est plus concis et tu gères moins de variable :
Code:

1 2 3 4 for(FourEtNumero currentFourEtNumero : list){ ComboFournisseur.addItem(currentFourEtNumero); }
Sinon ton getlistFourEtNumero() peut renvoyer null (si pas de résultat ou exception dans le traitement) et au final générer un nullPointerException dans remplirFournisseur(). Autant faire péter une exception parlante si le cas 'pas de résultat' est non permis ou renvoyer un tableau vide si le cas est permis.

Citation:

Envoyé par thebloodyman

Tu devrais plutôt ajouter les éléments de ta liste (redéfini toString()de FourEtNumero si ce n'est fait).

Merci thebloodyman de trouver la classe FourEtNumero

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
package ci.ressorces;
 
import java.io.Serializable;
 
public class FourEtNumero implements Serializable{
 
	private static final long serialVersionUID = 1L;
	private String NFOUFO;//Numéro du fournisseur (05)
	private String RSOCFO;//Raison sociale (11)
 
	public FourEtNumero(){
 
	}
 
	public void setNFOUFO(String nFOUFO) {
		NFOUFO = nFOUFO;
	}
 
	public String getNFOUFO() {
		return NFOUFO;
	}
 
	public void setRSOCFO(String rSOCFO) {
		RSOCFO = rSOCFO;
	}
 
	public String getRSOCFO() {
		return RSOCFO;
	}
}

Tu parles des éléments que j'ai déclarés ici ?

09/07/2010, 18h28
thebloodyman

Citation:

Tu parles des éléments que j'ai déclarés ici ?

Non, non je parle de FourEtNumero.

A la base tu as une liste d'objets de type FourEtNumero.
Et toi tu veux ajouter chacun de ses objets dans la JComboBox, il faut donc ajouter chaque FourEtNumero dans ton objet JComboBox et non la liste en entier.

Citation:
Envoyé par thebloodyman
Code:

1 2 3 for(FourEtNumero currentFourEtNumero : list){ ComboFournisseur.addItem(currentFourEtNumero); }
En procédant ainsi :
Code:

1 2 3 for(int i=0;i<list.size();i++){ ComboFournisseur.addItem(list.get(i).getRSOCFO()); }
le combo est remplir. Est-ce une bonne méthode ? Qu'en penses-tu ?

09/07/2010, 19h43
thebloodyman
Fonctionnellement, c'est une bonne solution si tu as n'as pas besoin d'accéder aux autres champs (Par exemple NFOUFO) de ta classe FourEtNumero durant les traitements sur tes comboBox.
Si tu as besoin d'accéder à d'autres champs de ton objet FourEtNumero il vaut mieux ajouter l'objet FourEtNumero dans ta comboBox.

Techniquement, tu peux avoir un code plus simple en utilisant la boucle foreach de Java.
Compare les deux :
Code:

1 2 3 4 for(int i=0;i<list.size();i++){ ComboFournisseur.addItem(list.get(i).getRSOCFO()); }
VS
Code:

1 2 3 4 for(ComboFournisseur comboFournisseur : list){ ComboFournisseur.addItem(comboFournisseur.getRSOCFO()); }
Dans la seconde solution, tu n'as pas a créer de variable i, ni à expliciter les conditions d'initialisation, de sortie et d'incrémentation de la boucle.
En gros, t'as moins de risques de faire des erreurs :ccool:
La boucle itère sur chaque objet de la liste et sort tout seul quand il n'y a plus d'objets sur lequel itéré.
12/07/2010, 12h22
the watcher
Citation:
Envoyé par thebloodyman

Code:

1 2 3 4 for(ComboFournisseur comboFournisseur : list){ ComboFournisseur.addItem(comboFournisseur.getRSOCFO()); }
En effet tu a pleinement raison thebloodyman. J'aurai besoin d'accéder aux autres champs comme le champ NFOUFO qui doit s'afficher lorsque je fais une sélection dans le Combobox. Ta procédure fonctionne à merveille.

Donc voici ma seconde préoccupation : comment mettre une procédure dans le listener de ma combofournisseur afin d'afficher le numéro (NFOUFO) du fournisseur correspondant dans un Jtextfield lorsque je sélectionne (RSOCFO) le nom du fournisseur du combofournisseur.

Merci encore.