write / read Problem

[JavaFreak94]

Hallo
Ich bin begeisterter Java Programmierer und schreibe mir gerade ein paar Einstellungen für mein Programm.
In diesem verbinde ich mehrere Programmparts über ein Textdokument. Es werden die einzelnen Einstellungen in je ein Textdokument geschrieben.
Das Problem ist, sobald man auf den Button zum Speichern klickt, schreibt er jedesmal etwas in ein Textdokument.
Egal ob für ein neues Hintergrundbild ein Pfad ausgewählt ist, oder nicht, ob ein neuer Name angegeben wurde, oder nicht.
Ich habe den Fehler seit einigen Tagen und weiß da einfach net weiter 😛
Das ist der ganze Code, der mit dem Problem in Verbindung stehen kann.

private void buttonActionPerformed(java.awt.event.ActionEvent evt) {
        //Frage "Programm wirklich beenden?" anzeigen 
            try {
                BufferedWriter bw = new BufferedWriter(new FileWriter("documents/exit question.txt"));

                if(ja.isSelected() == true) {
                    bw.write("true");
                }
                if(nein.isSelected() == true) {
                    bw.write("false");
                }

                bw.flush();
            } catch(Exception error) {
                System.out.println(error);
            }

            // Hintergrundbild ändern
            try {
                BufferedWriter buWr = new BufferedWriter(new FileWriter("documents/background picture.txt"));
                String get = TextField.getText();
                if(get.equals("") == false  &&  pfad != null) {
                    buWr.write(String.valueOf(pfad));
                    buWr.flush();
                } else if(get.equals("")  ||  pfad == null) {
                    buWr.write("pics/Sonnenuntergang.jpg");
                }
            } catch(Exception error) {
                System.out.println("Fehler beim Ändern des Hintergrundbildes. Grund: " + error);
            }

            //Namen ändern
            try {
                String name = NameInput.getText();
                BufferedWriter bw = new BufferedWriter(new FileWriter("documents/name.txt"));
                if(name != null) {
                    bw.write(name);
                    bw.flush();
                }
            } catch(Exception error) {
                System.out.println(error);
            }

            TextField.setText("");
            NameInput.setText("");
            JOptionPane.showMessageDialog(this, "Einstellungen gespeichert!");
    }

    private void chooserButtonActionPerformed(java.awt.event.ActionEvent evt) {
        //deaktiviere die "Zeige Alle" Option
        chooser.setAcceptAllFileFilterUsed(false);
        //setze den Filter
        chooser.setFileFilter(new FileFilter() {
            //Dieser Text wird unter "Dateityp" angezeigt
            String description = "Bild Dateien (*.jpg)";
            //Dateiendung
            String extension = "jpg";
            public boolean accept(File file) {
                if(file == null) return false;
                //zeigt alle Ordner an
                if(file.isDirectory()) return true;
                //gibt nur Dateien mit .pdf Erweiterung aus
                return file.getName().toLowerCase().endsWith(extension);
            }
            //hier wird unsere Beschreibung gesetzt
            public String getDescription() {
                return description;
            }
        });
        chooser.showOpenDialog(chooserButton);
        pfad = chooser.getSelectedFile();

        if(pfad != null) {
            TextField.setText(String.valueOf(pfad));
        }
    }

    private void zurueckActionPerformed(java.awt.event.ActionEvent evt) {
        try {
            BufferedReader br = new BufferedReader(new FileReader("documents/background picture.txt"));
            String read = br.readLine();
            if(read == null) {
                BufferedWriter bw = new BufferedWriter(new FileWriter("documents/background picture.txt"));
                bw.write("pics/Sonnenuntergang.jpg");
                bw.flush();
            }
            br.close();
        } catch(Exception error) {
            System.out.println(error);
        }
        this.dispose();
        new MainFrame().setVisible(true);
    }

 

[Gast2]

Und was ist deine Frage? Und warum hälst die Konfiguration nicht einfach in einem Properties oder XML file?

 

[SlaterB]

> Das Problem ist, sobald man auf den Button zum Speichern klickt, schreibt er jedesmal etwas in ein Textdokument.

inwiefern ist das ein Problem? du hast doch jede Menge write-Befehle, nicht alle stehen in einem if oder wenn dann steht im else auch was,
es wird also jede Menge geschrieben, was soll denn stattdessen exakt passieren?

und achte auf korrekte Umsetzungen, Zeile 83-85 schreiben irgendwas, aber es fehlt ein close()!
lieber erstmal nur wenig programmieren und das dann korrekt

 

[JavaFreak94]

Ja, er soll ja etwas abspeichern, aber eben genau einen ausgewählten Pfad, der mit einem FileChooser ausgewählt wurde, oder eben den Namen, doch wenn die Text Felder leer bleiben, schreibt er eben einfach nix hinein, löscht also einfach den ganzen Text im Textdokument.
Und genau das bekomm ich nicht hin.
Jedesmal, wenn ich die Einstellungen speichere und die Textfelder leer sind, dann wird auch der Text in den Textdokumenten gelöscht.

Doch die write Befehle stehen alle in einer if - Anweisung.
Oder muss ich die zuweisungen bzw. definitionen der Writer sprich BufferedWriter bw = new BufferedWriter ... auch noch in eine Anweisung packen?

 

[JavaFreak94]

Und zu Zeile 83 - 85

try {
            BufferedReader br = new BufferedReader(new FileReader("documents/background picture.txt"));
            String read = br.readLine();
            if(read == null) {
                BufferedWriter bw = new BufferedWriter(new FileWriter("documents/background picture.txt"));
                bw.write("pics/Sonnenuntergang.jpg");
                bw.flush();
            }
            br.close();
        } catch(Exception error) {
            System.out.println(error);
        }

Das ist einfach der Zurück Button und mit diesem Writer fang ich praktisch den Fehler ab.
Denn, wenn ich die Einstellungen speicher und nichts drin steht, können
die anderen Programmteile den Pfad des Hintergrundbildes nicht mehr auslesen und es würde abstürzen.
So schreib ich dann, sofern im Dokument nichts drin steht einen Pfad für n Standard Bild rein.

 

[Michael...]

Ich würde mir mal Properties (Java Platform SE 6) anschauen. Ist für solche Zwecke gedacht.

 

[JavaFreak94]

Hab mir da schon einiges angesehen, auch schon mit meinem Informatik Professor gesprochen, hab jedoch keine brauchbaren Antworten bekommen... 😛

 

[SlaterB]

> Oder muss ich die zuweisungen bzw. definitionen der Writer sprich BufferedWriter bw = new BufferedWriter ... auch noch in eine Anweisung packen?

denkbar, allzu genaues kann ich aber nicht sagen, du hast allein 3 ActionListener mit jeder Menge Code drin,
das ist ja gar nicht alles zu überblicken,

konzentriere deine Fragen, sofern noch offen, auch wenige überschaubare Dinge und gib detailliere Informationen + Log des Ablaufs:
was für Textfelder (am besten zum Testen nur eins!),
was steht am Anfang in der Datei,
was genau machst du alles im Programmablauf, wann wird etwas eintippt, wann welcher Button gedrückt,
wie ändert sich jeweils die Datei, was soll stattdessen passieren?

 

[JavaFreak94]

Ich werd den Code mal versuchen so umzugestalten, dass die Zuweisungen der Writer auch in ner Anweisung stehn.
Vllt is des wirklich des ganze Problem.

PS: War mein erstes Posting, drum entschuldigung, wenn ich n paar Fehler gmacht hab. 🙂

Ich melde mich min. noch einmal 🙂

 

[JavaFreak94]

Gut, ich habe die Writer Zuweisungen jetzt auch in Anweisungen gepackt.
Dadurch lies sich das Problem zur Hälfte beheben. 🙂

Beim Hintergrundbild ist der Fehler behoben. Wenn kein neuer Pfad ausgewählt wird, behält er den alten Pfad.
Und somit auch das alte Hintergrundbild. Ansonsten ersetzt er den alten Pfad durch den neuen.
Doch der Name wird immer noch überschrieben. Da stand die Zuweisung jedoch schon in einer Anweisung...