Discussion :

[Shadowritter]

Bonjour

J'essaye d'exécuter une requête qui me renvoi une liste de noms (des OS pour être précis), j'aimerais les récupérer dans une ArrayList pour ensuite les mettre dans une Jcombobox pour le moment

La requête ne me renvoi qu'une valeur, hors le résultat devrait être "Windows" ; "Linux" ; "Test", j'ai surement fais une erreur toute bête mais j'avoue ne pas l'a trouver

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
    public ArrayList Combo(String query)
    {
        ArrayList Array = new ArrayList();
        try
        {
            rs = sta.executeQuery(query);
            rs.last();
            int nbRow = rs.getRow();
            rs.beforefirst();
            int i = 0;
            while (rs.next())
            {
                i++;
                Array.add(rs.getString(i));
                System.out.println(rs.getString(i));//j'affiche pour vérifier
                rs.next();
            }
        }
        catch (SQLException queryE)
        {
            System.out.println("Erreur de requête : " + queryE);
        }
        return Array;
    }

Je suis encore assez débutant en JAVA, donc si vous avez des avis sur ma façon de faire, n'hésitez pas je me doute que c'est pas au point

Merci d'avance

[snoop]

Pas besoin de refaire un rs.next() puisqu'il est effectué dans le while :
El il faut fair eune boucle dans le while pour récupérer toutes les colonnes... Regardes avec les ResultSetMetadata pour récupérer les noms des colonnes et ensuite effectuer un getString("nomColonne") ...

Code :

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

1
2
3
4
5
6
7
 
        while (rs.next())
            {
              //  i++;
                Array.add(rs.getString(i));
                System.out.println(rs.getString(i));//j'affiche pour vérifier
            }

[Miss Gaelle]

Pourquoi tu utilises une variable nbRow ? Tu ne la réutilises pas par la suite?
Fais voir la requête.
C'est quelle valeur qu'il renvoit?

Pour le code je ferai comme ça :

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
 
public ArrayList Combo(String query) 
    { 
        ArrayList Array = new ArrayList(); 
        try 
        { 
            rs = sta.executeQuery(query); 
            rs.beforefirst(); 
            int i = 0; 
            while (rs.next()) 
            { 
                i++; 
                Array.add(rs.getString(i)); 
                System.out.println(rs.getString(i));//j'affiche pour vérifier 
            } 
        } 
        catch (SQLException queryE) 
        { 
            System.out.println("Erreur de requête : " + queryE); 
        } 
        return Array; 
    }

[romano21]

Je pense que ta query marche au niveau base.

Code :

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

1
2
3
4
 
 rs.last();
 int nbRow = rs.getRow();
 rs.beforefirst();

Ca, ca ne sert a rien, tu peux y supprimer de ton programme.

Code :

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

1
2
3
4
5
6
7
8
9
 
int i = 0;
while (rs.next())
{
                i++;
                Array.add(rs.getString(i));
                System.out.println(rs.getString(i));//j'affiche pour vérifier
                rs.next();
}

Ton rs.getString(i) n'est pas bon . Le i indique le numéro de colonne du résultat de ta requete.
Or, je suppose ke ta requete ramène une colonne, donc tu dois mettre rs.getString(1) et ton indice i ne sert à rien ici.

Voili voilou
Bon courage pour la suite


[/quote]

[fabszn]

Hello,

En fait le probleme vient de ce que j'ai mis en gras :

ArrayList Array = new ArrayList();
try
{
rs = sta.executeQuery(query);
rs.beforefirst();
int i = 0;
while (rs.next())
{
i++;
Array.add(rs.getString(i));
System.out.println(rs.getString(i));//j'affiche pour vérifier
}
}
catch (SQLException queryE)
{
System.out.println("Erreur de requête : " + queryE);
}
return Array;

En fait, à chaque rs.next() tu es positionné sur une occurence ramenée par ta requete.
Lorsque tu fais rs.getString(i), tu selectionnes une colonne sur l'occurence courante.
J'imagine que ta requete ne ramene qu'un seul champ.
Lors de ta premiere boucle, i est egale à 1 donc pas de probleme... mais pour les tours suivant, i prend des valeurs qui ne correspondent à aucun champs ramené par ta requete.

Dans ton code il faudrait que tu remplaces i par 1, ou que tu initialises i à 1 sans l'incrementer.

J'espere que cela t'aidera , si tu as des questions n'hesite pas!

[venegan]

le getString s'utilise comme ceci :


rs.getString("NOM DE COLONNE DE LA REQUETE") ou
rs.getString("NUMERO DE COLONNE")

et non pas numero de l'enregistrement

le rs.next() suffit a incrementer ton parcours

[Shadowritter]

Ah oui effectivement c'était tout con, donc merci à tous, j'aurais pus mettre longtemps à trouver tout seul

Pour le bout de code inutile, c'était parce que j'étais parti sur un for pour le compteur

En tout cas merci bien !

[NGY]

Salut,

Juste une petite remarque un peu hors sujet :

Code :

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

public ArrayList Combo(String query)

C'est dommage de retourner un ArrayList. Il vaut mieux déclarer que ta méthode retourne une interface Collection, voire une List si l'ordre est important.
Ca te permettra de changer l'implémention de ta liste si jamais tu en as besoin sans avoir à modifier le code dans les méthodes qui utilisent Combo().