Datei einlesen Fehler

[doktorschnaggel]

Hi zusammen.
Habe ein Problem und zwar will ich eine Textdatei in eine Hashmap einlesen.
Die Textdatei hat folgendes Format:
String;String
String;String
String;String
...
wobei der 1. String mein key für die Hashmap sein soll und der 2. dann der zum key gehärende Partner.
Nun habe ich eine Methode geschrieben, die aus der "antwort.txt" Datei lesen soll und dann die Zeile als String ausliest. Der String soll dann bei ";" gesplitet werden und der 1. Wert in die Hashmap als Key geschrieben werden und der 2. als Partner. Aber iwie hängt sich mein ganzes Java-Programm auf und ich hab keine Ahnung wieso. Hab auch schon rumgesucht aber ich bin auf keinen grünen Zweig gekommen.
Hoffe ihr könnt mir helfen.
Vielen Dank

public void vorbereiten()
{
  try{
	BufferedReader reader = new BufferedReader(new FileReader("antworten.txt"));
    String zeile = reader.readLine();
    ArrayList<String> values = new ArrayList<String>();
    while (zeile != null)
    {
    	values.add(zeile.split(";").toString());
    }
    for(int i = 0; values.size()>i;)
    {
    
    	speichern.put(values.get(i), values.get(i+1)); //speichern is der Name der Hashmap
    	i++;
    	i++;
    }
  }

 

[insane80]

Was meinst Du mit "hängt sich auf"?

Zuerst mal, fängst Du die Exception nicht.
Dann wirst Du in Deiner ArrayList vermutlich nicht das vorfinden, was Du erwartest, denn split() gibt ein String[] zurück.

Deine while-Schleife solltest Du auch nochmal überdenken, denn Du willst ja so lange aus der Textdatei lesen, wie sie Zeilen hat.

Wenn Du das hast, kannst Du in der gleichen Schleife auch die HashMap befüllen.

 

[Final_Striker]

Wie wäre es mit ein Paar

System.out.println(...)

in deinem Code?
Mach einen in deine while-Schleife, dann wirst du sehen woran es liegt.

 

[nillehammer]

- Der Aufruf von readLine() gehört natürlich in die while-Schleife. Ein einmaliger Aufruf liest nur die erste Zeile
- Die ArrayList is ihmo überflüssig. Du kannst Die Map doch direkt befüllen
- Mit split erhälst Du einen Array von Strings. Das nullte Element ist Dein Key, das erste Dein value

 

[doktorschnaggel]

Danke erstma für eure Hilfe.
Okay also hab die MEthode mal bisschen umgeschrieben, nur komm ich irgendwie nicht mit dem BufferedReader klar.

	while (reader.readLine() != null)
    {
    	  String zeile = reader.readLine();
    	  String[] key = zeile.split(";");
    	speichern.put(key[0],key[1]);
    	
    }

geht das in die richtige Richtung?

 

[HoaX]

Richtung ja, aber du verarbeitest so nur jede zweite Zeile.

 

[Camill]

So könnte dein Schleifenkopf aussehen, wie "insane80" bereits erwähnt hat würde ich das befüllen der HashMap direkt in der Schleife vornehmen.

String line = null;
while((line = reader.readLine()) != null) {
  //fill HashMap
}

 

[doktorschnaggel]

Okay und wie genau befülle ich die Liste jetzt?
Was ich nicht ganz verstehe ist, ob der Buffered Reader mit readLine() automatisch in der While Schleife die nächste Zeile raussucht oder man das noch irgendwie manuell eingeben muss.

  try{
	BufferedReader reader = new BufferedReader(new FileReader("antworten.txt"));
    
	String line = null;
	while((line = reader.readLine()) != null) {
	     String[] liste =  line.split(";");
	     speichern.put(liste[0], liste[1]);
	}

so hab das jetzt von Camill übernommen aber jetzt meckert er wieder und mir ist nicht genau klar warum.
split sollte mir doch ein Array von Strings übergeben und das sollte ich doch dann ganz normal auslesen können. Aber er meckert bei der Zeile [Java] String[] liste = line.split(";"); [/Java]. Ich verstehe nicht warum das so ist.

 

[Camill]

Poste mal den ganzen Code, Fehler liegt sicherlich an anderer Stelle.

 

[doktorschnaggel]

Also ich glaube nicht, dass der Fehler an einer anderen Stelle liegt. Wenn ich die Methode vorbereiten() am Anfang rauslasse funktioniert alles einwandfrei. Ich habe gleich zu Beginn einen Aufruf in den Konstruktor geschrieben, damit die HashMap sofort bei Programmstart gefüllt wird.

Hashmap habe ich ebenfalls im Konstrukur intialisiert:

HashMap<String,String> speichern = new HashMap<String,String>();

und hier nochmal die Methode

public void vorbereiten()
{
  try{
	BufferedReader reader = new BufferedReader(new FileReader("antworten.txt"));
    
	String line = null;
	while((line = reader.readLine()) != null) {
		String neu = reader.readLine();
	 liste =  neu.split(";");
	     speichern.put(liste[0], liste[1]);
	}
    	
   
}

 catch(IOException m )
 {
	 JOptionPane.showMessageDialog(null,"falsche datei");
 }

 }

Also ich sehe so keinen Fehler. Er gibt mir eine NullPointerException aus, in der Zeile, wo der String eigentlich gesplitet werden sollte.

 

[Camill]

Jetzt hast du aber etwas anderes gepostet als in deinem letzten post, was genau steht denn nun im Code ?

 

[insane80]

[JAVA=]liste = neu.split(";");[/code]

Hast Du das String-Array

liste

vorher deklariert? Ist in Deinem zuletzt geposteten Code zumindest nicht zu sehen...

 

[doktorschnaggel]

@Camill. Das ist doe Komplette Methode mit dem catch und dem Rest der Methode.
@insane80. Ja habe nun gemacht funktioniert trotzdem nicht.
Die Zeile sieht jetzt so aus:

	String[] liste =  neu.split(";");

Leute, dass gibts doch nicht, dass der dumme Code nicht laufen will.
Eigentlich muss das doch so jetzt funktionieren. Ich checks einfach nicht.

 

[timbeau]

Lass dir mal den String neu ausgeben. Ich tippe eher darauf, dass der Split nicht klappt.

 

[doktorschnaggel]

Ah, jetzt gibt er mir null aus für den String neu. Also liest er die Datei nicht ein. Komisch, denn ich habe sie im selbern Verzeichnis. Jemand ne Ahnung warum er das nicht liest.

 

[insane80]

Bekommst Du die Fehlermeldung, dass er die Datei nicht findet? Wenn ja, stimmt mit dem Pfad vermutlich doch was nicht.

Lass das

String neu...

weg, sonst werden nicht alle Zeilen in der HashMap gespeichert.

  String line;
    while((line = reader.readLine()) != null) {
        //String neu = reader.readLine();
     liste =  neu.split(";");
         speichern.put(liste[0], liste[1]);
    }

 

[nillehammer]

Und das neu.split muss dann natürlich line.split heißen.

 

[insane80]

Richtig, danke, hab ich übersehen...