Programm tut was es soll, aber leider trotzdem durchgefallen !?

Frage zur Aufgabe

Gelöst

Hallo Ihr Lieben. Mein Programm liest eine Datei Zeile für Zeile ein. Sogar Leerzeichen und leere Zeilen dazwischen, machen im nichts aus. Aber die Validation sagt trotzdem durchgefallen. Was habe ich übersehen? Gibt es eigentlich für den static-Block auch eine Möglichkeit mit "throws Exception". Bei mir wird immer ein Fehler angezeigt. Ich finde die Variante nicht so schön, mit den vielen catch-Blöcken. (Anmerkung: im NullpointerException Block habe ich noch reader.close() hinzugefügt.)

1. Initialisiere die Variable Statics.FILE_NAME auf den vollständigen Pfad zu einer Datendatei, die mehrere Zeilen enthält.
2. Lies dann in einem static-Block alle Zeilen aus der Datei mit dem Namen Statics.FILE_NAME, und füge sie einzeln zu der List namens lines hinzu.

Anforderungen:

Die Konstante FILE_NAME darf nicht leer sein.
Im static-Block müssen alle Zeilen aus der Datei mit dem Namen FILE_NAME einzeln zur Liste namens lines hinzugefügt werden.
Das Feld FILE_NAME darf NICHT final sein.
Die Solution-Klasse muss eine Liste von Zeilen haben.

package de.codegym.task.task15.task1525; import java.io.*; import java.util.ArrayList; import java.util.List; /* Datei in einem statischen Block */ public class Solution { public static List<String> lines = new ArrayList<>(); static { try { BufferedReader reader = new BufferedReader(new FileReader(Statics.FILE_NAME)); boolean x = true; while (x) { String textzeile = reader.readLine(); try{ if (!textzeile.isEmpty()) lines.add(textzeile); }catch(NullPointerException e){ x = false; } } }catch (FileNotFoundException e){ e.printStackTrace(); }catch (IOException e) { e.printStackTrace(); } } public static void main(String[] args) { System.out.println(lines); } }

Kommentare (5)

Beliebt
Neu
Alt

Du musst angemeldet sein, um einen Kommentar schreiben zu können

Guadalupe Gagnon

Level 37 , Tampa, United States

22 September 2020, 19:54

This code would create an infinite loop because x would only be false (and break the loop) if a null pointer exception occurs. There is no point in that code where a null pointer would occur. If reader.readLine() returns null, which it would when reading past the lines in the file, the null value would be put in the ArrayList. This would eventually lead to the ArrayList being filled to the max (the max would be the max of an integer aka 2.2 billion) however you would probaly get an out of memory error occuring before then.

Claudia Theiling

Level 18 , Lüneburg, Germany

22 September 2020, 20:08

Thanks, but sorry I have to contradict you. The program is working. There is no infinite loop, because of the !x.isempty() - if clause. That leads to the NullPointerException and my Programm finishes at the end of the file. But maybe you have a nicer idea, how to get out of the while-loop with the BufferedReader.

Guadalupe Gagnon

Level 37 , Tampa, United States

22 September 2020, 20:37

Blah, I'm sorry. I read the code too fast without actually processing what was going on, my bad. As per how I read from a file, this is the strategy I use when reading lines in a loop.

String line;
while((line = reader.readline()) != null) // this line sets the value and breaks if it is null
   lines.add(line);

Though this method here is probably even cleaner and more per OOP standards:

while(reader.ready()) // returns true when more lines are to be read
    lines.add(reader.readLine());

Guadalupe Gagnon

Level 37 , Tampa, United States

22 September 2020, 21:00

My guess with the code that you shared is that line 22 is filtering out empty lines, which would be a valid line in the file and should be added to "lines". Unfortunately I don't work for codegym or have access to the data they use to test code or the specific code used to test; so I just have to give best guesses based on my experience.

Claudia Theiling

Level 18 , Lüneburg, Germany

23 September 2020, 17:47

Hi Guadalupe, thanks a lot. I took the second one and it worked fine. - I passed the validation. 😘 And the difference is, that now the empty line is really stored in the array. Before it jumped over it and proceeded further on.