Problem mit dem Klassen aus dem java.io Paket

[math.iso@live.de]

Hallo liebe java-programierer


ich bin grad am java -lernen und wollte aus dem Buch "Einstieg ins Java " ein Programm kompilieren:

package ch08.io;

import java.io.BufferedReader;
import java.io.FileNotFoundException;
import java.io.FileReader;
import java.io.IOException;

public class FiletReader {

  public static void main (String[] arguments) {

    BufferedReader file = null;
    String text;

    try {
      file = new BufferedReader(new FileReader("Filet.txt"));
    }
    catch (FileNotFoundException ex) {
      // (...)
    }
    try {
      while ((text = file.readLine()) != null) {
        System.out.println(text);
      }
    }
    catch (IOException ex) {
      // (...)
    }
  }
}

Filet.txt

Filet mit Salat
Man nehme ein 3 cm dickes Filet, salze und pfeffere
es von einer Seite, brate es in der Pfanne 5 Minuten,
wende es anschliessend, salze und pfeffere es erneut,
brate es wieder 5 Minuten und serviere es mit Salat.

Das Problem ist ,nachdem kompilieren des Programm wollte ich das Programm ausführen ,leider kommt es zu einer fehlermeldung:

Exception in thread "main " java.lang.NullPointerException
at ch08.io.FiletReader.main(FiletReader.java:24)

ich hoffe ihr könnt mir helfen ,glaub der autor hat das programm nicht gut programmiert

 

[Ark]

Schreib mal den Code wie folgt um (ist auch sauberer!):

package ch08.io;

import java.io.BufferedReader;
import java.io.FileNotFoundException;
import java.io.FileReader;
import java.io.IOException;

public class FiletReader {

  public static void main (String[] arguments) {

    BufferedReader file = null;
    String text;

    try {
      file = new BufferedReader(new FileReader("Filet.txt"));
      while ((text = file.readLine()) != null) {
        System.out.println(text);
      }
    }
    catch (IOException ex) {
      ex.printStackTrace();
    }
  }
}

Ich vermute mal, die Datei Filet.txt existiert in dem Arbeitsverzeichnis nicht.

Ark

 

[Landei]

Wer auch immer das

while ((text = file.readLine()) != null) {...}

erfunden hat, sollte mit 10 Jahren verschärfter Cobol-Programmierung bestraft werden.

for (String text = file.readLine(); text != null; text = file.readLine()) {...}

Zugegeben, es ist ein paar Zeichen länger, dafür braucht man text nicht vorher zu definieren, und es meiner Meinung nach viel klarer, was hier Initialisierung, Abbruchbedingung und Schleifen-Increment ist, oder nicht?

 

[Ark]

Da bin ich leider überhaupt nicht deiner Meinung (@Landei: Wie wär's mit einer Umfrage? ). DRY! (Don't repeat yourself!) Und wenn die Anweisung mal etwas schwieriger wird als readLine(), tja ...

Sauberer in deinem Sinne wäre wohl aber auch dieser Code:

while(true){
    text = file.readLine();
    if(text == null) break;
    // weiterer Code
}

Es würde mich auch nicht wundern, wenn der Bytecode identisch aussehen sollte. Ich halte den Einzeiler einfach für handlicher.

Ark

 

[Landei]

Yo, mit der break-Version könnte ich leben. Der eigentliche "Skandal" ist, dass BufferedReader kein Iterable ist, so man leider nicht einfach schreiben kann:

for(String text : file) {
  ...
}

Da haben die Sunny-Boys echt geschlafen...

 

[math.iso@live.de]

warum klappt das prog von meinen java einstiegsbuch nicht ?

da muss doch ein grund geben ,oder ein hakan geben ...

 

[math.iso@live.de]

file.readline() gibt doch eine zeile des Textes aus

warum


while ((text = file.readLine()) != null) {
System.out.println(text);

gibt er hier den gesamten text aus , denn woher soll das programm wissen warum er beim 2 durchlauf der while schleife die 2 zeile ausgeben soll ?

 

[Ark]

gibt er hier den gesamten text aus , denn woher soll das programm wissen warum er beim 2 durchlauf der while schleife die 2 zeile ausgeben soll ?

Warum ist da eine while-Schleife drum?

Reader sind Zeichenströme. Nach dem Lesen eines Zeichens ist dieses "futsch", und es geht mit dem nächsten Zeichen weiter.

Du solltest übrigens aufhören, Ansprüche an die Sprache bzw. mitgelieferte APIs zu stellen. Und deine Form solltest du auch mal überdenken.

Ark

 

[math.iso@live.de]

sorry Arks ich hab es nicht so gemeint ,ich meinte das alles sehr nett

 

[Landei]

Das Prinzip "ein Stückchen ausspucken und vergessen, das nächste Mal das nächste Stückchen ausspucken und vergessen..." zieht sich durch ganz Java. So arbeiten:
- Streams, Reader
- Iterator, Enumeration
- StringTokenizer
- (SQL-) ResultSet
- SAX-Parser