Discussion :

[the watcher]

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 :

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
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 :

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

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

Code :

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

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 ?

[thebloodyman]

Salut,

Ici :

Code :

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

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 :

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

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.

[the watcher]

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 :

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
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 ?

[thebloodyman]

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.

[the watcher]

Code :

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

1
2
3
for(FourEtNumero  currentFourEtNumero : list){
  ComboFournisseur.addItem(currentFourEtNumero);
}

En procédant ainsi :

Code :

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

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 ?

[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 :

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

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

VS

Code :

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

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
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é.

[the watcher]

Code :

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

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.