Discussion :

[xeak2008]

Bonjour à tous,

J'ai une méthode readListComplet qui prend toutes les infos d'un BDD. (env. 9000 élèments). Afin d'éviter les aller/retour dans la BDD, je lis entièrement la liste. (ce qui marche très bien)

Mais voilà c'est pas très agréable d'avoir les 9000 élèments et j'ai donc décidé, de traiter la liste en affichant seulement les 100 premier et ensuite je gèrerais ça avec des boutons suivant et précèdent.

J'ai fait une méthode qui lis les 100 premier mais je ne comprend pas pourquoi elle ne marche pas.

voici mon code... y a t-il une erreur dans ma boucle for ???

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
public ArrayList readListPartielle(GenOdStandard pOd, Connection pCon, int pFrom, int pSize) throws AppException
 {
    try
    {
       ArrayList listePartielle = this.readListComplet(pOd, pCon, 0, 0);
       for (int i = 0; i < 100; i++)
       {
         listePartielle = (ArrayList)listePartielle .get(i);
       }
       return listePartielle ;
    }
    catch (Exception e)
    {
       throw new AppMException(
          new GenOdErreur(GenOdErreur.ERREUR_GRAVE,_DOMAINE,
          "Erreur X2", "SQL=" + SQL_SELECT_LISTE), e);
    }
 }

Merci de votre aide

[henolivier]

A vue de nez, je dirais que le problème vient de :

Code :

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

for (int i = 100; i < listePartielle .size(); i++)

essaie en mettant :

Code :

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

for (int i = 0; i < 100; i++)

[xeak2008]

Oui pardon,

c'est du à une erreur de ma part... j'ai corriger
c'est donc bien :

Code :

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

for (int i = 0; i < 100; i++)

mais ça ne marche pas...

[Heimdal]

Salut,

"ca ne marche pas" n'est pas vraiment un descriptif très utile, coller la stacktrace de l'exception serait plus productif.

Bref, je suppose que tu as affaire à une ClassCastException ou à une OutOfBoundsException.

Code :

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

listePartielle = (ArrayList)listePartielle .get(i);

Le ième élément de ta liste n'est sans doute pas un ArrayList.
Parlons bonne pratique...
- Si tu es en Java5 ou supérieur on peut typer les listes (? une liste de quoi?).
- S'il n'y a pas de méthodes particulières de telle ou telle implémentation que tu as besoin d'utiliser on préfére déclarer et travailler sur des interfaces.
Exemple sur ces deux points:

Code :

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

List<Patate> listeDePatates = new ArrayList<Patate>();

- Ensuite tes arguments pFrom et pSize sont inutiles (tu ne t'en sers pas). De plus le bon sens nous laisse penser qu'en mettant pSize à 0 ta liste résultat sera vide.

Pour finir réfléchis bien au sens de cette boucle:

Code :

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

1
2
3
4
for (int i = 0; i < 100; i++)
       {
         listePartielle = (ArrayList)listePartielle .get(i);
       }

et jette un oeil à la javadoc de l'interface java.util.List et notamment à la méthode add(E e): http://java.sun.com/javase/6/docs/api/

[thelvin]

En ce qui me concerne, je ne comprends pas ce que tu essaies de faire, et il me semble que toi non plus.

Si tu pouvais nous faire une description de ce que font exactement chacune des lignes de ta boucle... Alors tu comprendrais pourquoi ça ne marche pas. En ce qui me concerne, je ne peux pas t'aider tant que je ne sais pas ce que tu veux.

[xeak2008]

Ce que je veux faire c'est de retourner les 100 premiers élèment de la liste de ma méthode readListComplet.

Je vois en effet ma boucle for est complètement inutile.

[Alienor_Dhels]

C'est peut-être hors propos mais si c'est pour afficher seulement une partie de la liste, pourquoi ne pas utiliser la JTable ?

[thelvin]

Ce que je veux faire c'est de retourner les 100 premiers élèment de la liste de ma méthode readListComplet.

List.subList() ne fait pas simplement l'affaire ?

[Alain Defrance]

List.subList() ne fait pas simplement l'affaire ?

Tout-à-fait. De plus il me semble que cette liste possède les même références donc au niveau de la mémoire c'est très light.
Si on veux on peut aussi utiliser des WeakReferences ce qui permettrait d'éviter des fuites mémoires.