Hilfe, wieso funzt das net?

  • Themenstarter Themenstarter Guest
  • Beginndatum Beginndatum
Status
Nicht offen für weitere Antworten.
G

Guest

Gast
kann mir bitte jemand sagen wo der Fehler liegt?

die Frage wir korrekt importiert und ausgegeben aber wenn ich sie abspeichern will, dann löscht er immer die gesamte Text datei und schreibt nix rein. ich bin ratlos! Wo liegt der Fehler?

Code:
 import java.io.BufferedReader;
import java.io.FileReader;
import java.io.IOException;
import java.io.BufferedWriter;
import java.io.FileWriter;

public class AdventEingabe {

	public static void main(String[] args) {

		String[][] frageListe = new String[24][3];
		try {
			BufferedReader f = new BufferedReader(new FileReader("Rätsel"));
			String line = null;
			int zeile = 0;
			while ((line = f.readLine()) != null) {
				int pos = zeile / 4; //Türchen
				int typ = zeile % 4; // Zeile im einzelnen Rätsel
				if (typ != 3) {
					frageListe[pos][typ] = line;
				}
				zeile++;
			}
			f.close();
		} catch (IOException e) {
			System.out.println("Fehler beim Lesen der Datei");
		}

		int frageNr;
		System.out.println("Welche Frage möchtest du bearbeiten. Waehle zwischen 1 und 24.");
		frageNr = Tastatur.readint();
		System.out.println(frageListe[frageNr - 1][0]);
		System.out.println(frageListe[frageNr - 1][1]);
		System.out.println(frageListe[frageNr - 1][2]);

		boolean speichern = true;

		System.out.println("Bitte gib die neue Frage ein.");
		String eingabeNeu = Tastatur.readString();
		System.out.println("Hier ist nochmal die eingegebene Frage: ");
		System.out.println(eingabeNeu);
		System.out.println("Möchtest du die Frage so abspeichern? j/n: ");
		speichern = (Tastatur.readchar() == 'j');
		if (speichern == true) {
			 eingabeNeu = frageListe[frageNr - 1][0] ;

			try {
				BufferedWriter wr = new BufferedWriter(new FileWriter("Rätsel"));
				wr.write(frageListe[frageNr - 1][0]);
			} catch (IOException e) {
				e.printStackTrace();
			}
			if (speichern == false)
				System.out.println("Die Frage wurde nicht gespeichert.");

		}
	}
}
 
Man muss unterscheiden:

Wenn schon etwas in der Datei steht, wirds etwas komplizierter:

Alle Zeilen die drin stehen lesen und im Speicher halten (z.B. in einer ArrayList) und dann die gesamte ArrayList mit z.B. einem BufferedWriter auf Platte + Dein neuen Zeilen dranhängen.
Nachteil: Es wird immer das komplette File geschrieben.
Wenn die Datei leer ist:
Code:
BufferedWriter bw;
bw.write("ueberschrift");
bw.newLine();                       //Nicht vergessen, bei Dir ueberschreibt es immer wieder die Zeile
bw.close();                           //Und auch wieder schließen, sonst Datenverlust möglich

Aber vielleicht weiß noch jemand was besseres.

Gute Nacht
 
zeile 45 sollte wohl die zuweisung umgedreht sein.

Die Datei wird gelöscht weil du nicht appendest. Wenn du willst musst du einen anderen Konstruktor benutzen: FileWriter(File file, boolean append)

Aber du musst ja auch nicht nur die eine frage schreiben sondern alle, also in einer Schleife durchs array gehn.

hmm und warum da nichts drin steht: schon mal ein flush oder close am stream versucht?
 
Hm, ich denke eure Tips helfen mit nicht wirklich.
Es soll tatsächlich immer nur die Frage überschrieben werden, die man zuvor ausgewählt hat, der Rest soll bestehen bleiben. Die Frage soll nicht angehängt werden.
Hat noch jemand ne Idee?
 
am einfachsten ist es, die Datei komplett neu zu schreiben,
also alles vorherige auslesen, in Java verändern und dann wieder zurück
 
Also entweder versteht ihr meine Frage nicht, oder ich eure Antworten. :bahnhof:

Ich habe eine Datei mit 24 Fragen. Dann soll ich eine der Fragen aufrufen und dafür eine neue eingeben können. die anderen 23 Fragen sollen bestehen bleiben.
 
Code:
...
        if (speichern == true) {
            eingabeNeu = frageListe[frageNr - 1][0] ;
            try {
                doSave(frageListe);
            } catch (IOException ex) {
                ex.printStackTrace();
            }
...
    private static void doSave(String[][] frageListe ) throws IOException{
        BufferedWriter wr = new BufferedWriter(new FileWriter("Rätsel"));
        for (int i = 0; i < frageListe.length; i++) {
            for (int j = 0; j < frageListe[i].length; j++) {
                wr.write(frageListe[i][j]+System.getProperty("line.separator"));
            }
        }
        wr.close();
    }
...
 
naja er will ja die alte frage nicht mehr mitschreiben, und das geht nun mal nur in dem er alles neuschreibt
 
Danke, jetzt gehts
auch wenn diese Zuweisung

eingabeNeu = frageListe[frageNr - 1][0] ;

falsch rum ist. Wenn man sie umdreht funzt es.
Vielen DANK!
 
Anonymous hat gesagt.:
Danke, jetzt gehts
auch wenn diese Zuweisung

eingabeNeu = frageListe[frageNr - 1][0] ;

falsch rum ist. Wenn man sie umdreht funzt es.
Vielen DANK!
Das hat mit dem eigentlichen Problem des Abspeicherns aber gar nix zu tun:
Die Variable "eingabeNeu" wird nämlich überhaupt nicht in die Datei abgespeichert :wink:
 
André Uhres hat gesagt.:
Code:
if (speichern == true)

:shock:

Schreibst du etwa auch

Code:
if ((x==42) == true)
oder, um ganz sicher zu gehen, sogar
Code:
if (((x==42) == true) == true)

Geh' in die Ecke und schäm dich! ( :bae: )
 
Status
Nicht offen für weitere Antworten.

Zurück
Oben