Bufferedwriter code Nullpointerexception

[Timon110]

//Methode soll alle Daten aus Kontakte (eine information pro zeile) in temporäre datei schreiben außer die zu löschenden informationen
  BufferedReader reader = new BufferedReader(new InputStreamReader(new FileInputStream("kontakte.txt")));
        BufferedReader reader1 = new BufferedReader(new InputStreamReader(new FileInputStream("kontaktetemp.txt")));

        BufferedWriter writer = new BufferedWriter(new OutputStreamWriter(new FileOutputStream("kontaktetemp.txt", true)));
        BufferedWriter finish = new BufferedWriter(new OutputStreamWriter(new FileOutputStream("kontakte.txt", true)));
        String line = "";
        BufferedWriter tdel = new BufferedWriter(new OutputStreamWriter(new FileOutputStream("kontaktetemp.txt", false)));
tdel.write("");    //löscht alles im temporären txt
tdel.close();

        while (line != null) {
            line = reader.readLine();
            if (line == name) {line = reader.readLine(); line = reader.readLine(); line = reader.readLine();} //wenn zu löschende info = line ist werden 3 zeilen übersprungen (alles was zu löschen ist)
            writer.write(line); //schreibt in temp file

        }
        writer.close();
        reader.close();
        String line1 = "";

        BufferedWriter kdel = new BufferedWriter(new OutputStreamWriter(new FileOutputStream("kontakte.txt", false)));
        kdel.write("");    //löscht alles im hauptfile
        kdel.close();
        while (line1 != null) {
            line1 = reader1.readLine();
            finish.write(line1);    //schreibt alles aus temp file in kontakt file
        }
        reader1.close();
        finish.close();

Warum tritt eine nullpointer exception auf?

 

[mrBrown]

An welcher Stelle tritt die denn auf?

Falls du schon Java 14 nutzt, übergib -XX:+ShowCodeDetailsInExceptionMessagesbeim starten.

 

[kneitzel]

Also wenn Du sowas postest, dann bitte auch direkt angeben, wo eine Exception auftritt und den StackTrace. Sonst muss man nur unnötig lange suchen.

Und in der while Schleife hast Du in beiden den Fall, dass readLine irgendwann null zurück gibt und Du dennoch damit arbeitest. Das wäre jetzt meine Vermutung. Und Strings vergleicht man mit equals und nicht mit ==.

Und Einiges kannst Du direkt löschen, weil Du es nicht brauchst in deinem Code. Und generell solltest Du Variablen nicht wie in einem Script am Anfang definieren sondern immer erst da, wo man sie wirklich benötigt. Und Du solltest try with resources verwenden.

Das sieht dann so in der Art aus für das Kopieren (etwas gekürzt):

try (reader = ...; writer= ...) {
  while ((line=reader.readLine()) != null) {
    if (line.equals(name) {
      reader.readLine();
      reader.readLine();
      reader.readLine();
      continue;
    }
    writer.write(line); //schreibt in temp file
  }
}

Die zweite Schleife würde ich ähnlich aufbauen.

Generell wäre es aber auch schöner, Daten generell korrekt einzulesen. Also eine Klasse Adresse oder so in der die Daten enthalten sind und dann kann man Adresse von einem Reader lesen und in einen Writer schreiben ... und schon hätte man es auch etwas sauberer. Aber das ist evtl. etwas, das erst später beim Java Lernen kommen wird, wenn die ersten Grundlagen etwas besser sitzen.

 

[LimDul]

Auf die gleiche Datei gleichzeitig zwei Writer bzw einen reader & einen Writer aufzumachen sieht mir mindestens fragwürdig aus.

 

[Timon110]

Also wenn Du sowas postest, dann bitte auch direkt angeben, wo eine Exception auftritt und den StackTrace. Sonst muss man nur unnötig lange suchen.

Und in der while Schleife hast Du in beiden den Fall, dass readLine irgendwann null zurück gibt und Du dennoch damit arbeitest. Das wäre jetzt meine Vermutung. Und Strings vergleicht man mit equals und nicht mit ==.

Und Einiges kannst Du direkt löschen, weil Du es nicht brauchst in deinem Code. Und generell solltest Du Variablen nicht wie in einem Script am Anfang definieren sondern immer erst da, wo man sie wirklich benötigt. Und Du solltest try with resources verwenden.

Das sieht dann so in der Art aus für das Kopieren (etwas gekürzt):

try (reader = ...; writer= ...) {
  while ((line=reader.readLine()) != null) {
    if (line.equals(name) {
      reader.readLine();
      reader.readLine();
      reader.readLine();
      continue;
    }
    writer.write(line); //schreibt in temp file
  }
}

Die zweite Schleife würde ich ähnlich aufbauen.

Generell wäre es aber auch schöner, Daten generell korrekt einzulesen. Also eine Klasse Adresse oder so in der die Daten enthalten sind und dann kann man Adresse von einem Reader lesen und in einen Writer schreiben ... und schon hätte man es auch etwas sauberer. Aber das ist evtl. etwas, das erst später beim Java Lernen kommen wird, wenn die ersten Grundlagen etwas besser sitzen.

Ich habs dank dir hinbekommen.
Vielen Dank!