Nullpoint Exception ähh?

[R3M4X]

Hallo Leute,

versuche gerade mit meinem Code aus einer Zeile, den Wert "USER" auszulesen und einfach zählen wieviele USER so in der Datei vorkommen. Und zwar liest er alles richtig aus, nur sobald der Buffer auf Null geht, liest er nicht mehr den letzten Wert von int User aus. Warum nicht und was kann ich da machen?

Mein Fehlercode: Exception in thread "main" java.lang.NullPointerException
at datei_V2.main(datei_V2.java:20)

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

public class datei_V2 {
	public static void main (String[]args) throws IOException{
		File datei = new File("C:\\Daten.txt");
		System.out.println("Dateiname: " + datei.getName());
		System.out.println("Dateigröße: " + datei.length());
		
		int USER= 0;
		FileReader datei_2 = new FileReader("C:\\Daten.txt");
		BufferedReader bf = new BufferedReader(datei_2);
		String test = bf.readLine();
		while (test !=  null){
			System.out.println(test);
			test = bf.readLine();
			
			if (test.contains("USER")){
				USER = USER + 1;
				System.out.println(USER);
			}
		
		}
		}
		
	}

}

 

[velaluka]

Deine Bedienung ist doch test != null und die wird am Ende auch erfüllt, also müsstes du vor dem contains auf null prüfen.
Ciao Velaluka

 

[ARadauer]

test = bf.readLine(); in zeile 18 gehört unter den if block...

 

[R3M4X]

test = bf.readLine(); in zeile 18 gehört unter den if block...

Hey, wie einfach, vielen Dank!

 

[tfa]

Du überliest die erste Zeile. Schlecht, wenn da auch ein USER drin steht.

 

[bygones]

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

public class datei_V2 {
	public static void main (String[]args) throws IOException{
		File datei = new File("C:\\Daten.txt");
		System.out.println("Dateiname: " + datei.getName());
		System.out.println("Dateigröße: " + datei.length());
		
		int USER= 0;
		FileReader datei_2 = new FileReader("C:\\Daten.txt");
		BufferedReader bf = new BufferedReader(datei_2);
		String test = bf.readLine();
		while (test !=  null){
			System.out.println(test);
			test = bf.readLine();
			
			if (test.contains("USER")){
				USER = USER + 1;
				System.out.println(USER);
			}
		
		}
		}
		
	}

}

oder einfach
[groovy]
def file = "C:\\Daten.txt" as File
println file.readLines().count { it.contains('USER') }
[/groovy]

bygones

 

[R3M4X]

oder einfach
[groovy]
def file = "C:\\Daten.txt" as File
println file.readLines().count { it.contains('USER') }
[/groovy]

bygones

Irgend wie klappt das nicht , habe Syntax error mit deinem Code Eclipse sagt: def kenn ich nicht, willst du Classe draus bauen?

Frage: Kann ich den readLine() irgend wie zerlegen? Habe z.B. in der nächsten Zeile verschiedene Freischaltungen die ein USER beinhaltet, würde gerne diese auch irgen wie auslesen können.

 

[bygones]

Irgend wie klappt das nicht , habe Syntax error mit deinem Code Eclipse sagt: def kenn ich nicht, willst du Classe draus bauen?

solange du nicht groovy bei dir installiert hast wird das nix ;-)

wollte nur mal wieder protzen wie einfach man Sachen in groovy loesen kann....

also bygones

 

[R3M4X]

oder einfach
[groovy]
def file = "C:\\Daten.txt" as File
println file.readLines().count { it.contains('USER') }
[/groovy]

bygones

solange du nicht groovy bei dir installiert hast wird das nix ;-)

wollte nur mal wieder protzen wie einfach man Sachen in groovy loesen kann....

also bygones

Ist es den üblich als Anfänger, gleich mit Groovy anzufangen? Sollte man lieber nicht erstmal den Pfaden, mit Java beibehalten um später bei größeren Sachen sich mit Groovy auszuhelfen?

Wie zerlege ich den nun meine readLine() am besten?

 

[schalentier]

public void readFile() {
   FileReader datei_2 = new FileReader("C:\\Daten.txt");
   BufferedReader bf = new BufferedReader(datei_2);

   String line;
   while( (line=bf.readLine()) != null ) {
      processLine( line );
   }
}

public void processLine( String line ) {
   if (line.contains("USER")){
      USER = USER + 1;
      System.out.println(USER);
   }
}

Das While-Konstrukt ist in der Tat haesslich und man sollte solches Zeug vermeiden, aber ich denke beim typischen Zeileneinlesen ist das okay. Der Vollstaendigkeit halber noch die Alternative:

...
String line;
do {
   line = bf.readLine();
   if( line!=null ) {
      processLine( line );
   }
} while( line!=null );
...

 

[bygones]

Ist es den üblich als Anfänger, gleich mit Groovy anzufangen? Sollte man lieber nicht erstmal den Pfaden, mit Java beibehalten um später bei größeren Sachen sich mit Groovy auszuhelfen?

groovy hilft eigentlich immer egal ob groesser oder kleiner sachen... aber bleib erstmal bei pur java ;-)

Wie zerlege ich den nun meine readLine() am besten?

nein readLine kannst du nicht zerlegen... wenn du aber einen USER gefunden hast kannst du natuerlich ueber weitere readLine aufrufe die darauffolgenden Zeilen einlesen...

 

[R3M4X]

Das nenne ich ein Support hier! Hätte ich Geld würd ich euch bezahlen!

 

[SlaterB]

man braucht groovy nicht, um sowas abzukürzen, von paar Variablen-Deklarationen + Schleifen-Klammern abgesehen:

        EasyFile file = new EasyFile("test.txt");
        Counter c = new Counter();
        for (String line : file.readLines())
        {
            c.count(line.contains("USER"));
        }
        System.out.println("count: " + c.getCount());

es braucht nur entsprechende Hilfsklassen

class Counter
{
    int count = 0;

    public void count(boolean bool)
    {
        if (bool)
        {
            count++;
        }
    }

    public int getCount()
    {
        return this.count;
    }
}


class EasyFile
{
    private String file;

    public EasyFile(String file)
    {
        this.file = file;
    }

    public Iterable<String> readLines() // provisorisch
    {
        final BufferedReader reader;
        try
        {
            reader = new BufferedReader(new FileReader(this.file));
        }
        catch (Exception e)
        {
            throw new RuntimeException(e);
        }

        final Iterator<String> iter = new Iterator<String>()
            {
                private String line = null;

                public boolean hasNext()
                {
                    try
                    {
                        line = reader.readLine();
                        if (line == null)
                        {
                            reader.close();
                        }
                    }
                    catch (Exception e)
                    {
                        throw new RuntimeException(e);
                    }
                    return line != null;
                }

                public String next()
                {
                    return line;
                }

                public void remove()
                {
                }

            };

        return new Iterable<String>()
            {
                public Iterator<String> iterator()
                {
                    return iter;
                }
            };
    }
}

wenn man so ungewöhnlich programmieren möchte, dann geht das auch in Java,
die Lehrbücher müssten das vielleicht bisschen mehr verbreiten