monitorjbl / excel-streaming-reader

An easy-to-use implementation of a streaming Excel reader using Apache POI
Apache License 2.0
953 stars 344 forks source link

ArrayIndexOutOfBoundsException in StreamingWorkbook.getSheet(name) when sheet with that name does not exist #94

Open marmys opened 7 years ago

marmys commented 7 years ago

version used: 1.1.0

stack trace: java.lang.ArrayIndexOutOfBoundsException: -1 at java.util.ArrayList.elementData(ArrayList.java:418) at java.util.ArrayList.get(ArrayList.java:431) at com.monitorjbl.xlsx.impl.StreamingWorkbook.getSheet(StreamingWorkbook.java:85)

monitorjbl commented 7 years ago

I believe this is fixed in the latest code, can you check it out and install it locally?

marmys commented 7 years ago

Hi, thanks for reply. In the latest code throwing ArrayIndexOutOfBoundException is fixed. But another problem is that now MissingSheetException is thrown when sheet is missing and according to documentation null should be returned.