Problem beim Einlesen einer Datei (Endlosschleife ?)

[HoMeRPoWeR]

Hiho! Hab da folgendes Problem, und zwar muss ich ein Programm schreiben, welches eine text (eingabe.txt) mit einem Wörterbuch (woerterbuch.txt) vergleicht. Jetzt habe ich folgendes Problem. Ich hab das Wörterbuch eingelesen und in einer Liste gespeichert, funktioniert auch, jetzt wollte ich als nächstes den Text Wort für Wort einlesen. Klappt auch soweit, nur wird das letzte Wort nicht eingelesen, und das Programm hängt glaube ich in einer Endlosschleife fest.

Hier mal meine Methode dafür

	public static void checkFile() {
		FileReader f1, f2;
		FileWriter w1;
		char space = ' ';
		String newLine = "\n";
		char zeichen;
		char tmp, satzzeichen;
		String wort = "";
		
		try {
			f1 = new FileReader("eingabe.txt");
			f2 = new FileReader("eingabe.txt");
			w1 = new FileWriter("ausgabe.txt");
			while ((tmp = (char) f1.read()) != -1 ) {
				while ((zeichen = (char) f2.read()) != space) {
					if (zeichen == '.') {                //falls am Wortende ein Punkt ist, Punkt merken
						satzzeichen = '.';
					}
					else if (zeichen == ',') {         //falls am Wortende ein Komma ist, Komma merken
						satzzeichen = ',';
					}
					else {
						wort += zeichen;
					}
					
				}
				System.out.println(wort);
				wort = "";
			}
			
			f1.close();
			f2.close();
			w1.close();
		}
		catch (FileNotFoundException e) {
			System.out.println("Datei nicht gefunden!");
			System.out.println(e.getMessage());
		}
		catch (IOException e) {
			System.out.println("Fehler beim Lesen der Eingabedatei");
			System.out.println(e.getMessage());
		}
	}//Ende checkfile

Danke schonmal für die Hilfe!!

 

[Ark]

Das Problem ist in Zeile 15: Durch das Casten auf char geht das Vorzeichen verloren, aus -1 wird dann \uFFFF, und das ist eben nicht mehr -1, daher die Endlosschleife. Das Casten musst du also später vornehmen, falls es überhaupt nötig ist.

Außerdem ist es sinnfrei, zwei gleich funktionierende Reader auf dieselbe Datei anzuwenden. Versuche, f2 zu umgehen; das kannst du z. B. mit tmp machen. Und sieh ja zu, dass du einen StringBuffer oder StringBuilder verwendest, um dein wort zusammenzubasteln!

Ark

 

[Gelöschtes Mitglied 5909]

verwende doch readLine() und split() oder regex, geht denke ich besser

 

[homerpower]

Okay Programm läuft, funzt alles, hat mich aber doch länger beschäftigt als erwartet.

Danke für die Hilfe