Discussion :

[Diablo143]

Bonjour à tous,

j'ai une petite méthode qui lit un fichier excel contenant 3 feuilles

Code :

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

1
2
3
4
5
 
final Workbook classeurExcel = WorkbookFactory.create(fichier);
final Sheet feuilleEquipe1 = classeurExcel.getSheet("Equipe_1");
final Sheet feuilleEquipe2 = classeurExcel.getSheet("Equipe_2");
final Sheet feuilleScore = classeurExcel.getSheet("Score");

et pour les lires, je passe par une boucle while différente par page ce qui donne le résultat suivant

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
 
int index = 1;
Row lignesEquipe1 = feuilleEquipe1.getRow(index++);
 
while (lignesEquipe1 != null) {
 
        final JoueurHandball joueur = rowToJoueur(lignesEquipe1);
    equipe1.add(joueur);
        lignesEquipe1 = feuilleEquipe1.getRow(index++);
}
index = 1;
Row lignesEquipe2 = feuilleEquipe2.getRow(index++);
while (lignesEquipe2 != null) {
 
    final JoueurHandball joueur = rowToJoueur(lignesEquipe2);
    equipe2.add(joueur);
    lignesEquipe2 = feuilleEquipe2.getRow(index++);
}

techniquement ça fonctionne, mais je voudrait savoir s'il y'a pas un moyen de faire ça de manière plus propre et optimisé.

merci pour vos réponse.

[tchize_]

Ben oui, vu que c'est le même code qui est répété, tu met ce code dans une méthode, qui prend en paramètre la feuille et te retourne l'équipe. De plus cette lecture supplémentaire d'un row avant la boucle n'a pas vraiment de sens.

Code :

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

1
2
3
4
5
6
7
8
9
public List<JoueurHandball> getEquipe(Sheet feuille){
    List<JoueurHandball> result = new ArrayList<>();
 
    Row lignesEquipe = null;
    while ( (lignesEquipe =  feuille.getRow(index++)) != null) {
        equipe1.add(rowToJoueur(lignesEquipe));
    }
    return result;
}

Code :

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

1
2
equipe1 = getEquipe(feuilleEquipe1);
equipe2 = getEquipe(feuilleEquipe2);

[Diablo143]

Oh que je suis c**, la solution la plus simple et j'y ai pas pensé.
Mais oui effectivement ça fait beaucoup plus propre, merci