Input/Output NullPointerException with Apache POI

[fabipfolix]

Hello,
I'm trying to code a little program which uses Excel as a little DB.
But my code which should read the Excel File got a NullPointerException.

Here's my code: (I've imported everything)

public class DBReader {
    List<Mini> miniList = new ArrayList<Mini>();
  
    public List<Mini> readSheet1() throws FileNotFoundException, IOException{
      
        HSSFWorkbook wb = new HSSFWorkbook(new FileInputStream("DB.xls"));
      
        HSSFSheet sheet = wb.createSheet();
        for(int i=0;i<=sheet.getLastRowNum();i++) {
            System.out.println(i);
            String name = "error";
            double age = 4;
          
            HSSFRow row = sheet.getRow(i);

            if(row.getCell(0).getCellTypeEnum()==CellType.STRING) { //ERROR
              
                name = row.getCell(0).getStringCellValue(); //ERROR if I remove if(row.getCell(0).getCellTypeEnum()==CellType.STRING)
              
            }
            if(row.getCell(1).getCellTypeEnum()==CellType.NUMERIC) {
                age = row.getCell(1).getNumericCellValue();
              
            }
            Mini m = new Mini(name, age);
          
          
            miniList.add(m);
          
        }
    wb.close();
    return miniList;
      
    }
}

 

[httpdigest]

In welcher Zeile tritt die Exception denn genau auf? Das siehst du im Stacktrace der Exception.
Ist es die Zeile mit dem Kommentar // ERROR if I remove if(row.getCell(0).getCellTypeEnum()==CellType.STRING)? Und, wenn ja, wieso solltest du die if-Anweisung dadrüber entfernen, die ja gerade prüft, ob der gelesene Wert ein String ist. Das heißt, der Code, so wie du ihn gepostet hast, wirft gar keine Exception?

Generell: Leute, die sagen, "dieser Code wirft eine Exception" sollten IMMER folgende Informationen mitgeben:
- in welcher Zeile tritt die Exception auf (natürlich Zeilennummern übersetzen für den Post oder einfach einen Kommentar über oder hinter die entsprechende Zeile im Code)
- den Exceptiontyp (bei dir ja NPE)
- falls gegeben: die Nachricht der Exception

EDIT: Oh, habe gerade erst das // ERROR hinter der if-Anweisung gesehen.
Schau dir mal https://stackoverflow.com/questions...exception-cannot-read-cell-if#answer-31191178 an.

 

[fabipfolix]

EDIT: Oh, habe gerade erst das // ERROR hinter der if-Anweisung gesehen.
Schau dir mal https://stackoverflow.com/questions...exception-cannot-read-cell-if#answer-31191178 an.

Daran dürfte mein Problem eigentlich nicht liegen, da ich die Zeile Text enthält. Ich erhalte diesen Fehler beim ersten Durchlauf, 5 Zeilen enthalten allerdings die richtigen Information. Dazu kommt, dass es funktioniert hat, bis ich etwas davon unabhängiges geändert habe.

Bei dem gegebenen Beispiel verstehe ich allerdings den Nutzen von Math.max und Math.min nicht.
Von was sind die eingesetzten Zahlen abhänig?

 

[httpdigest]

Naja, offensichtlich ist doch entweder row == null oder row.getCell(0) == null. Refactore diese eine Zeile doch erstmal in zwei Zeilen und prüfe dann mit einem Debugger, welche der zwei Fälle null ist.
Bei der SO Antwort ist ja eigentlich nur relevant, dass du (laut der Apache POI Dokumentation und den Benutzungsbeispielen (per Link genannt)) bei Iteration über ein Dokument immer prüfen musst, ob eine Zeile und Zelle null ist.

 

[fabipfolix]

Wenn ich getRow benutze dann ist row == null. Wenn ich createRow benutze scheint dies zu funktionieren. Dann liegt das nächste Problem allerdings hier:

  if(row.getCell(0).getCellTypeEnum()==CellType.STRING) {

Deswegen habe ich das ein wenig aufgesplitted und das Problem liegt bei dem Vergleich des CellTypes.

public List<Mini> readSheet1() throws FileNotFoundException, IOException{
       
        HSSFWorkbook wb = new HSSFWorkbook(new FileInputStream("DB.xls"));
       
        HSSFSheet sheet = wb.createSheet();
        for(int i=0;i<=sheet.getLastRowNum();i++) {
            System.out.println(i);
            String name = "error";
            double age = 4;
           
            Row row = sheet.createRow(i);
            if(row ==null) {
                System.out.println("row=null");
                continue;
            }
            Cell c = row.createCell(0);
            c.setCellType(CellType.STRING);
           
           
           
            if(c.getCellTypeEnum()==CellType.STRING) { //ERROR
               
                name = row.createCell(0).getStringCellValue();
               
            } else { //Wird nur ausgeführt wenn: if(row.getCell(0).getCellTypeEnum()==CellType.STRING) {
                System.out.println("Zeichenkette benötigt");
            }
            if(row.getCell(1).getCellTypeEnum()==CellType.NUMERIC) {
                age = row.createCell(1).getNumericCellValue();
               
            } else {
                System.out.println("nur Zahlen erlaubt");
            }
           
            Mini m = new Mini(name, age);
           
           
            miniList.add(m);
           
        }
    wb.close();
    return miniList;
       
    }

 

[httpdigest]

Wieso solltest du denn jetzt einfach createRow() und createCell() benutzen???
Das Sheet enthält offensichtlich tatsächlich keine Daten in der jeweiligen Zeile. Dann bringt es jetzt doch auch nichts, das Sheet einfach zu ändern und eine Zeile und Zelle anzulegen, deren Inhalt ja dann immer noch leer ist.
Kannst du zumindest verifizieren, dass du eventuell einfach Leerzeilen in dem tatsächlichen Excel-Sheet hast? Für solche Leerzeilen erhältst du dann nämlich einfach eine null Row.

 

[fabipfolix]

Ich wollte einfach mal ausprobieren, ob es so vielleicht funktioniert. Dank deiner Antwort ist mir dann aber aufgefallen, dass ich ja vorher schon das getSheetAt(0) zu createSheet(0) geändert habe (hatte da auch plötzlich einen Fehler, obwohl es vorher funktioniert hat).
Jetzt funktioniert das wieder.
Danke für deine Hilfe!

 

[httpdigest]

[...]dass ich ja vorher schon das getSheetAt(0) zu createSheet(0) geändert habe

Oh stimmt! Sorry, das ist mir die ganze Zeit auch nicht aufgefallen.