NullPointerException beim Schreiben einer ArrayList in eine Datei

[ZxSpectrum]

Hallo liebe Community,

Ich stehe verzweifelt vor einem Problem, das ich leider nicht eigenständig lösen kann. Ich habe schon viele Wege ausprobiert, aber leider lande ich immer am selben Ausgangspunkt.

Mein Problem ist folgendes:
1. Ich möchte eine Datei zeilenweise einlesen und in ArrayList ablegen.

    public static ArrayList<String> schreiben(String element){

        file = new File("Highscore.txt");

        try {

            writer = new FileWriter(file ,true);//Hängt ans Ende der Datei, wenn vorhanden

            writer.write(element); //Element aus der ArrayList wird in die Datei geschrieben      
          
            writer.write(System.getProperty("line.separator"));//Zeilenumbruch

            writer.flush();

            // Schliesst den Stream
            writer.close();
        } catch (IOException e) {
            e.printStackTrace();
        }
        return highscore;
    }//schreiben

    public static ArrayList<String> lesen(String element) throws IOException{

        FileReader fr = new FileReader("Highscore.txt");
        BufferedReader br = new BufferedReader(fr);

        while (br.readLine()!=null){
            element = br.readLine();//jede Zeile im tmp String abspeichern
//            highscore.add(br.readLine());//War ein Versuch einer alternativen lösung
            highscore.add(element);//Jede Zeile wird zu Arraylist "highscore" geäddet.
        }//while
      
        br.close();
        return highscore;

    }//public static void lesen

2. Ich wollte diese Werte in dem ArrayList anschließend sortieren.

Collections.sort(highscore);

3. Und das ganze wieder in die Datei Highscore.txt geordnet reinschreiben.

public static ArrayList<String> schreiben(String element){

        file = new File("Highscore.txt");

        try {

            writer = new FileWriter(file ,false);//Hängt ans Ende der Datei, wenn vorhanden

            writer.write(element); //Element aus der ArrayList wird in die Datei geschrieben      
          
            writer.write(System.getProperty("line.separator"));//Zeilenumbruch

            writer.flush();

            // Schliesst den Stream
            writer.close();
        } catch (IOException e) {
            e.printStackTrace();
        }
        return highscore;
    }//schreiben

Und nu so sieht meine Main Methode aus:

        //***********HIGHSCORE*********
        ScoreWriter.lesen(element);//Hier sollten die Daten aus der Datei Highscore.txt eingelesen werden
      
        element = Methods.spielModus(prompt, zufallsZahl, element) + " " + auswahl;//tmp Speicher für die Werte aus der zuletzt gespielten Runde
        highscore.add(element);//Der aktuelle Wert vom Durchlauf des Programms sollte hinten an die Liste dran gehängt werden
      
        Collections.sort(highscore);//Die ArrayList sortieren

        System.out.println("inhalt der Liste");//Hier wird Kontrollweise der Inhalt der Liste ausgegeben
        for (String count: highscore) {
            System.out.println(count);
        }

        for (int i = 0; i < highscore.size(); i++) {
            element = highscore.get(i);
            ScoreWriter.schreiben(element);  
        }
        //***********HIGHSCORE*********

Nun, das Problem liegt daran, dass die Werte nicht sauber eingelesen werden, anstatt den ersten Eintrag erscheint "null" und beim schreiben in die Datei wird eine NullPointerException geschmissen.

Kann mir jemand von euch helfen?

Vielen Dank im Voraus

Liebe Grüße

ZxSpectrum

 

[Joose]

  public static ArrayList<String> schreiben(String element){

Ich finde es unsinnig das eine Methode die Daten in eine Datei schreibt etwas zurückgibt.
Wenn auch nur ob das schreiben erfolgreich war oder nicht, aber keine ArrayList.

Und wenn ich mir den Inhalt deiner "schreiben" Methode anschaue, hat es auch keinen Sinn.
==> Rückgabewert entfernen und void verwenden

Nun, das Problem liegt daran, dass die Werte nicht sauber eingelesen werden, anstatt den ersten Eintrag erscheint "null" und beim schreiben in die Datei wird eine NullPointerException geschmissen.

while (br.readLine()!=null) {
     element = br.readLine();   //jede Zeile im tmp String abspeichern
     highscore.add(element);     //Jede Zeile wird zu Arraylist "highscore" geäddet.
   }

Kann mir jemand von euch helfen?

Dein Problem liegt beim Einlesen in der while-Schleife.
In der Bedingung rufst du "readLine" auf und es wird eine Zeile eingelesen. Wenn diese != null ist betritts du die while-Schleife.
Dort liest du mit "readLine" wieder eine Zeile ein, nicht aber dieselbe wie bei der Bedingung.
=> Du prüfst jede 2.Zeile auf != null und speicherst nur jede 2.Zeile in deiner ArrayList.

Lösung: Deklariere eine String Variable welche als Zwischenspeicher für die ausgelesene Zeile dient. Diese Variable prüfst du in der Bedingung auf != null und mit dieser Variable arbeitest du in der Schleife. Am Ende der Schleife liest du die nächste Zeile.

 

[JStein52]

Du liest doch immer zwei Zeilen ein:

        while (br.readLine()!=null){
            element = br.readLine();//jede Zeile im tmp String abspeichern

einmal eine in der while-Bedingung und dann die nächste explizit ! Hängt dein Problem damit zusammen ?

 

[Jardcore]

String line = "";
while((line= br.readLine()) != null) {
    System.out.println(line);
}

Du kannst im Kopf der while Schleife direkt die Variable füllen

 

[kneitzel]

Und was mir noch aufgefallen ist: Du übergibst der Funktion lesen element, aber Du nutzt element dann nur in der while schleife zum Einlesen der Zeile. Da musst Du doch nichts übergeben sondern kannst auf den Parameter verzichten und element wird zu einer lokalen Variable.

 

[ZxSpectrum]

Danke euch allen! Das hat mir weiter geholfen, jetzt funktioniert´s fast, wie ich es mir vorgestellt hätte.

Eine Frage habe ich an Euch noch.

public static void schreiben(){

        String zeilenumbruch = System.getProperty("line.separator");
        file = new File("Highscore.txt");
       
        try {

            if (file.exists()) file.delete();
           
            writer = new FileWriter(file, false);//Hängt ans Ende der Datei, wenn vorhanden       
           
            for (String counter: highscore) {
                writer.write(counter);
                writer.write(zeilenumbruch);
            }

            writer.flush();

            // Schliesst den Stream
            writer.close();
        } catch (IOException e) {
            e.printStackTrace();
        }
    }//schreiben

Warum wird hier kein Zeilenumbruch in die Datei reingeschrieben und alle Werte werden in einer derselben Zeile ausgegeben? Hat jemand eine Idee?

 

[X5-599]

Kann ich hier nicht nachvollziehen. Bei mir werden die Zeilenumbrüche geschrieben. Am Code kann es eigentlich nicht liegen.
- Sicher, dass dieser Code ausgeführt wird. Evtl gibt's ja noch eine andere Methode.
- Schaust du dir die richtige Highscore.txt an?