Discussion :

[Diablo143]

Bonjour à tous,

je rencontre une nullPointer exception quand j'essaie de lire une colonne dans une ligne d'un fichier excel

mon classeur excel est composé de plusieurs ligne qui ont chacune 3 colonnes de renseigné, qui correspondent aux information d'un joueur (exemple en dessous)

Erwan

Capitaine

22

j'arrive bien à récupérer les informations contenues dans la première colonne, mais pas dans la seconde, ni dans la troisième

voici ma méthode de lecture

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
private static JoueurHandball rowToJoueur(final Row row) { 
 
    JoueurHandball joueur = new JoueurHandball(); 
    joueur.setNom(row.getCell(0).getStringCellValue()); 
    joueur.setPoste(row.getCell(1).getStringCellValue()); 
    joueur.setNumero((int) row.getCell(2).getNumericCellValue()); 
 
    return joueur; 
 
 }

la ligne qui récupère le nom me renvoie la valeur "Erwan", mais celle censé récupérer le poste me renvoie un NullPointerException, et idem pour le numéro.

merci d'avance pour vos réponses.

[wax78]

Est-tu sure que ton fichier XLS(x) est correcte parce qu'avec le peu de code que je vois, tout semble OK.

[Diablo143]

comme j'arrive à récupérer la première colonne je pense que le fichier est correct, après en log j'ai la trace suivante quand il exécute la ligne de récupération du poste
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at traitement.TraitementSport.rowToJoueur(TraitementSport.java:115)
la ligne 115 étant

Code :

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

1
2
 
joueur.setPoste(row.getCell(1).getStringCellValue());

après j'avoue ne pas avoir essayé de changer le format de mes cellules en les mettant en pure Texte ou Nombre dans Excel

[wax78]

Refile un bout de fichier (ou tout le fichier si c'est pas confidentiel) car c'est impossible à voire d'ou peut venir le problème ainsi. (dans le bon format)

Mais a priori

Code :

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

row.getCell(1)

retourne un null comme indiqué par le stacktrace.

[Diablo143]

Voilà le fichier utilisé.
Si ça peut aider voici aussi un lien vers le code complet que j'utilise
https://github.com/ErwanLT/Sports/bl...mentSport.java
la méthode le lecture de ligne Excel ce trouve dans la méthode traitementHandball

[wax78]

Chez moi je n'ai aucun nullpointer, ça marche ... par contre je remarque un truc bizarre que je n'ai pas encore compris.

Edit : Ok, je viens de voire une truc qui peut être pourrait expliquer des choses, l'index était à 1 c'était voulu ? (voir le commentaire)

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
private static List<JoueurHandball> feuilleToEquipe(final Sheet sheet){
        List<JoueurHandball> equipe = new ArrayList<JoueurHandball>();
        int index = 0; // Pourquoi avoir mis 1 avant ?
        Row lignesEquipe = sheet.getRow(index++);
 
        while (lignesEquipe != null) {
 
            final JoueurHandball joueur = rowToJoueur(lignesEquipe);
            equipe.add(joueur);
 
            lignesEquipe = sheet.getRow(index++);
        }
 
        return equipe;
 
    }

[Diablo143]

bizarre.

l'index qui commence à 1 car à la base je comptait mettre une ligne d'entête, mais qu'au final elle est pas présente dans le fichier.
je vais réessayer ce soir en rajoutant de la log en console pour mieux voir

[joel.drigo]

Salut,

A noter, qu'on peut aussi faire :

Code :

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

1
2
3
for(Row row : sheet) {
         equipe.add(rowToJoueur(row));
}

pas besoin de se coltiner un index, de faire des tests sur les "null", des mi while mi do/while , tout ça.

Pour sauter la ligne de titre on peut faire :

Code :

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

1
2
3
4
5
6
7
8
9
boolean skipHeader=true;
for(Row row : sheet) {
       if ( skipHeader ) {
            skipHeader = false;
       }
       else {
              equipe.add(rowToJoueur(row));
       }
}

ou

Code :

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

StreamSupport.stream(sheet.spliterator(), false).skip(1).forEach( row-> equipe.add(rowToJoueur(row)) );

[Diablo143]

J'ai changé le format des colonne sous Excel et ça passe sans générer l'exception.
ensuite je vais mettre en place la solution proposer par joel.drigo

Code :

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

1
2
3
4
5
6
7
8
9
10
 
boolean skipHeader=true;
for(Row row : sheet) {
       if ( skipHeader ) {
            skipHeader = false;
       }
       else {
              equipe.add(rowToJoueur(row));
       }
}

merci à tous pour vos réponses et votre aide.