(csv)Datei lesen FindBug findet mgl. NullPointer - wie lösen

[Guest]

Hallo,

ich hatte diese Methode um eine CSV-Datei einzulesen:

private void readCSVFile() throws IOException, ParseException {
		String lineInFile = new String();
		String[] fieldsPerLine = null;
		try {
			inputFile = new BufferedReader(new FileReader(csvFileName));
			readfile:                        // <<== a labeled statement
			while ( (lineInFile = inputFile.readLine()) != null ) {
				cntLines++;
				lineInFile = lineInFile.trim();
				// leere Zeilen nicht beachten
				if ( lineInFile.equals("") ) {
					continue readfile;
				}
				// Kommentarzeilen nicht beachten
				test4comment:                // <<== a labeled statement 
				for ( int i = 0; i < POSSIBLE_COMMENT_CHAR.length; i++ ) {
					if ( lineInFile.startsWith(POSSIBLE_COMMENT_CHAR[i]) ) {
						continue readfile;
					}
				}
				// Zeileninhalt als String Array aufnehmen
				fieldsPerLine = lineInFile.split(String.valueOf(delimiter));
				// prüfen ob der Eintrag aufgenommen wurde
				if ( ! allItemsInFile.add(fieldsPerLine) ) {
					throw new RuntimeException(" Aufnahme des Eintrages [ " + 
							Arrays.toString(fieldsPerLine) + " ] in Zeile [" + 
							cntLines + "] war nicht mgl.");
				} 
			}
		} catch (FileNotFoundException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		} finally {
			if ( inputFile != null ) {
				inputFile.close();
			}
		}
	}

Den Java Code hab ich unter Eclipse mit dem FindBug Plugin prüfen lassen und so geändert:

private void readCSVFile() throws IOException, ParseException {
		String[] fieldsPerLine = null;
		try {
			inputFile = new BufferedReader(new FileReader(csvFileName));
			readfile:                        // <<== this is a labeled statement
			for ( cntLines = 1 ; true; cntLines++ ) {
				String lineInFile = inputFile.readLine();
				if ( lineInFile != null ) {
					lineInFile = lineInFile.trim();
					// leere Zeilen nicht beachten
					if ( lineInFile.equals("") ) {
						continue readfile;
					}
				}
				// Kommentarzeilen nicht beachten
				checkcomment:                // <<== this is a labeled statement 
					for ( int i = 0; i < POSSIBLE_COMMENT_CHAR.length; i++ ) {
						if ( lineInFile.startsWith(POSSIBLE_COMMENT_CHAR[i]) ) {
							continue readfile;   // hier die aeussere for-Schleife beendet
						}
					}
				// Zeileninhalt als String Array aufnehmen
				fieldsPerLine = lineInFile.split(String.valueOf(delimiter));
				// prüfen ob der Eintrag aufgenommen wurde
				if ( ! allItemsInFile.add(fieldsPerLine) ) {
					throw new RuntimeException(" Aufnahme des Eintrages [ " + 
							Arrays.toString(fieldsPerLine) + " ] in Zeile [" + 
							cntLines + "] war nicht mgl.");
				} 
			}
		} catch (FileNotFoundException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		} finally {
			if ( inputFile != null ) {
				inputFile.close();
			}
		}
	}

Nun findet FindBug in der Zeile 18

if ( lineInFile.startsWith(POSSIBLE_COMMENT_CHAR[i]) ) {

den Fehler/Hinweis:

<NP> Null value is guaranteed to be dereferenced <NP_GUARANTEED_DEREF>
There is a statement or branch that if executed guarantees that a value is null at this point, and that value that is guaranteed to be dereferenced (except on forward paths involving runtime exceptions).

Ich prüf' doch schon in der Zeile 8 ob die Variable lineInFile möglicherweise null sein könnte. Macht hier eine nochmalige Prüfung noch einen Sinn ???:L Leider hab ich aktuell keine Idee was ich hier machen könnte, damit FindBug zufrieden ist.

Gruß & vielen Dank für Eure Hilfe

 

[tfa]

Möglicherweise kann Findbugs nicht mit dem continue umgehen. Du könntest dem if aus Zeile 8 noch ein else spendieren.

 

[SlaterB]

Zeile 7, lineInFile wird eingelesen, kann null sein,
Zeile 8, if wird übersprungen,

Zeile 17, for Schleife beginnt,
garantiert Zugriff auf lineInFile -> NullPointerException,

die erste Version war besser

 

[Guest]

Hallo tfa,
hallo slaterB,

Möglicherweise kann Findbugs nicht mit dem continue umgehen. Du könntest dem if aus Zeile 8 noch ein else spendieren.

jup - so ist FindBugs zufrieden! :applaus:

...
						continue readfile;
					}
				} else {
					break readfile;
				}
				// Kommentarzeilen nicht beachten
				checkcomment:                // <<== this is a labeled statement 
...

Zeile 7, lineInFile wird eingelesen, kann null sein,
Zeile 8, if wird übersprungen,

Zeile 17, for Schleife beginnt,
garantiert Zugriff auf lineInFile -> NullPointerException,

die erste Version war besser

danke - das hab ich im ersten Moment so nicht kapiert
Mit der ersten Version war FindBugs auch nicht 100%ig zufrieden, da hat die Initialisierung der Variable linInFile in der Zeile 2 nicht gepasst:

<Dm> Method invokes inefficient new String() constructor <DM_STRING_VOID_CTOR>
Creating a new java.lang.String object using the no-argument constructor wastes memory because the object so created will be functionally indistinguishable from the empty string constant "". Java guarantees that identical string constants will be represented by the same String object. Therefore, you should just use the empty string constant directly

Deshalb hab ich dann das so umgebaut. Allerdings war es in der ersten Version nur dieser Hinweis.

In der zweiten Variante hatte ich jedoch einen möglicher Fehler eingebaut. Von dem her gesehen, hast Du natürlich recht. Jetzt wenn ich mir den Hinweis genau durchlese, dann hilft das hier in der erste Variante bei der Initialisierung der Variable linInFile in Zeile 2 ebenso:

 String lineInFile = "";

Ich werd' wieder die erste Variante nehmen - FindBugs ist damit 100%ig zufrieden - dank Euch!

vielen Dank Euch zwei für die Hilfe & Erklärung.