requete select ne marche pas dans jComobox

bonsoir à tous ,
jes deux jcombobox dont l'une dépende de l'autre je m'explique :
la premiere Jcombobox est remplie depuis la BD des type de matériel (imprimante,scanners...ect);je veux l'orsque je choisi un type de matériel par exemple imprimante qu'il m'affiche dans la deuxieme jcombobox les numéro de série des imrprimantes qui sont en panne .voici ce ma requête à chaque selection d'un type il m'affiche un seul numéro de seri de ce type dans la combo et la liste comprend les numéro de série de tous les types en panne au lieu de ceux de celui que j'ai selectionné ,j' ignore pourquoi il prend pas en considération la condition WHERE en gras voici ma requête :

Code:

---

1 2 3 4 5 6 7 8 9

String requete= "SELECT NUMSERIMAT FROM PANNE INNER " + "JOIN MATERIEL ON " + " MATERIEL.NUMSERI= PANNE.NUMSERIMAT " + " where TYPEMAT='" + jComboType.getSelectedItem().toString() + "'"; Statement state = cnx.getConnection().createStatement(); ResultSet result = state.executeQuery(requete); if (result.next()) { jComboBox1.setSelectedItem(result.getObject(1).toString());

---

avez -vous des suggestions ?
merci d'avance

Bonsoir,
oui pour les suggestions :

• Mettre un while à la place du if pour parcourir tous les enregistrements du resultset et ne pas avoir uniquement le premier (while (result.next()){...})
• Remplacer result.getObject(1).toString() par result.getString(1)
• Penser à fermer ce qui est ouvert (resultset, statement...), mais tu l'as peut-être fait plus bas...
• Mais surtout ne pas faire des accès aux données dans l'interface en les délocalisant dans des classes spécifiques d'accès aux données

Citation:

---

Envoyé par kox2ee

Bonsoir,[*]Mettre un while à la place du if pour parcourir tous les enregistrements du resultset et ne pas avoir uniquement le premier ([B]while (result.next()){...}

---

merci de m'avoir répondu mais j'ai tous les enregistrement alors que je veux avoir seulement les enregistrement du matériel sélectionné avec la condition where : " where TYPEMAT='" + jComboType.getSelectedItem().toString() + "'";
c'est ça mon problème :cry:.

Citation:

---

Penser à fermer ce qui est ouvert (resultset, statement...), mais tu l'as peut-être fait plus bas...

---

oui je l'ai bien fermé en bas je voulais juste vous montrer la requête .

Bonjour,
le code :

Code:

---

1 2 3 4

if (result.next()) { jComboBox1.setSelectedItem(result.getObject(1).toString()); }

---

est quand même surprenant... il ne fait pas du tout ce dont tu parles.

Si tu veux parcourir le resultset, et ajouter dans la combobox les enregistrements trouvés , il faut déjà supprimer tous les éléments s'y trouvant, puis parcourir tout le resultset (while) pour ajouter les enregistrements via le model de la combo.

Exemple :

Code:

---

1 2 3 4 5 6

DefaultComboBoxModel<String> model = (DefaultComboBoxModel<String>) combo.getModel(); model.removeAllElements(); while (result.next()) { model.addElement(result.getString(1)); }

---

Sinon, si ta requête ne semble pas te fournir les résultats attendus, utilise les bonnes vieilles méthodes :
Tu affiches l'instruction sql, et tu l'exécutes directement dans la DB.

Citation:

---

Envoyé par kox2ee

Bonjour,
le code :

Code:

---

1 2 3 4

if (result.next()) { jComboBox1.setSelectedItem(result.getObject(1).toString()); }

---

est quand même surprenant... il ne fait pas du tout ce dont tu parles.

Si tu veux parcourir le resultset, et ajouter dans la combobox les enregistrements trouvés , il faut déjà supprimer tous les éléments s'y trouvant, puis parcourir tout le resultset (while) pour ajouter les enregistrements via le model de la combo.

Exemple :

Code:

---

1 2 3 4 5 6

DefaultComboBoxModel<String> model = (DefaultComboBoxModel<String>) combo.getModel(); model.removeAllElements(); while (result.next()) { model.addElement(result.getString(1)); }

---

Sinon, si ta requête ne semble pas te fournir les résultats attendus, utilise les bonnes vieilles méthodes :
Tu affiches l'instruction sql, et tu l'exécutes directement dans la DB.

---

bonsoir ;
je te remercie infiniment pour ton aide ça marche Dieu merci :ccool: