TextField-Inhalt mit Button in .txt-Datei speichern

[xzlukas_]

Hallo,
Ich versuche mit einem Button den Inhalt von TextFields in eine .txt-Datei zu speichern

Hier der Code von dem Button zum Speichern:

button1.addActionListener(new ActionListener() {
          public void actionPerformed(ActionEvent e) {
              try {
                    boolean append = true;
                    PrintWriter out = new PrintWriter(new BufferedWriter(new FileWriter("outfilename", append)));
                    out.println(textbox1.getText());
                    out.println(textbox2.getText());
                    out.println(textbox3.getText());
                    out.println(textbox4.getText());
                    out.println(textbox5.getText());
                    out.println(textbox6.getText());
                    out.close();
                } catch (IOException e1) {
                }
          }
        });

Wenn ich das Programm jetzt starte, die Textfelder fülle und den Button drücke passiert nichts und ich finde den Fehler in meinem Code nicht. Kann jemand meinen Code korrigieren oder mir sagen was ich falsch gemacht habe und mir erklären wie ich es korrigiere?

Wenn der restliche Code benötigt wird einfach fragen!

 

[Robat]

Entweder PrinterWriter#flush aufrufen, oder im Konstruktor vom PrintWriter als letztes Argument true übergeben.

 

[xzlukas_]

oder im Konstruktor vom PrintWriter als letztes Argument true übergeben.

Mache ich das nicht bereits hier?

boolean append = true;
                    PrintWriter out = new PrintWriter(new BufferedWriter(new FileWriter("outfilename", append)));

Also ich habe ja diese boolean-variable der ich den wert true gegeben habe.

Und wo müsste ich flush hin schreiben? vor oder hinter dem Schreiben?

 

[Robat]

Nein der boolean muss beim PrintWriter in den Konstruktor. Warum hast du eigentlich so ein komisches Konstrukt? new PrintWriter(new FileWriter(File), true); würde doch ausreichen. Wozu der BufferedWriter zwischendrin?

 

[xzlukas_]

Wozu der BufferedWriter zwischendrin?

Den Writer hab ich von nem anderen Beitrag in diesem Forum da ich versucht habe mein Problem erst so zu lösen.

Ich habe den Code jetzt wie du gesagt hast angepasst und er funktioniert jetzt. Kannst du mir auch verraten wie ich den Inhalt einer Textdatei (Der Name der Textdatei ist Variabel also am besten mit einem Auswahlfenster) wieder in die Textboxen lade?

 

[Robat]

"Auswahlfenster => JFileChooser für Swing oder FileChooser für JavaFX
Naja du liest die Datei ein und musst den Inhalt irgendwie auswerten. Wie kommt auf den Inhalt an.

 

[xzlukas_]

Ich will die Datei zeilenweise auslesen
meine txt-Datei sieht ca. so aus

123
1234
12345
123456
1234567
12345678

und ich will das Zeile 1 in TextField1, Zeile 2 in TextField 2 usw. geladen wird.
Wie genau mache ich das?

 

[Robat]

Naja du liest die Datei zeilenweise ein und packst dann jeweils die Zeile n in TextField n mit setText(String). Sollten die TextFields in nem Array o.ä. gespeichert sein, kann man das ganze natürlich noch etwas generischer machen.

 

[Robat]

Sollten zum Zeitpunkt des Einlesens noch keine TextFields existieren, kannst du natürlich auch pro eingelesene Zeile ein Textfield erstellen, dort den Text setzen und es der GUI hinzufügen.

 

[xzlukas_]

button3.addActionListener(new ActionListener() {
          public void actionPerformed(ActionEvent e) {
             JFileChooser datei = new JFileChooser();
             int returnVal = datei.showOpenDialog(getComponent(0));
          }
        });

Das ist bis jetzt mein Button zum laden wie bekomme ich jetzt den Dateipfad damit ich dem Reader sagen kann wo er die Datei findet die er lesen soll

die infos zum erstellen meines FileChoosers habe ich hierher:
https://docs.oracle.com/javase/tutorial/uiswing/components/filechooser.html

 

[Robat]

JFileChooser#getSelectedFile()

 

[xzlukas_]

button3.addActionListener(new ActionListener() {
          public void actionPerformed(ActionEvent e) {
             JFileChooser datei = new JFileChooser();
             int returnVal = datei.showOpenDialog(getComponent(0));
            
             BufferedReader in;
            try {
                in = new BufferedReader(new FileReader(datei.getSelectedFile()));
            } catch (FileNotFoundException e2) {
                // TODO Auto-generated catch block
                e2.printStackTrace();
            }

             String line;
            try {
                line = in.readLine();
                textbox1.setText(line);
            } catch (IOException e1) {
                // TODO Auto-generated catch block
                e1.printStackTrace();
            }
            
             textbox1.setText(line);
          }
        });

Ich weiß nicht wie ich diesen Fehler korrigiere.

 

[Robat]

Du liest momentan auch nur eine Zeile ein. Schau dir noch mal ein Beispiel an, wie man eine Datei komplett einliest

 

[xzlukas_]

Ich finde kein Beispiel alle Beispiele die ich bisher gefunden habe sehen so aus. Wie kann ich dem Reader sagen das er z.B. in eine 2. Variable den Inhalt von Zeile 2 schreibt?

 

[xzlukas_]

ok Habe das Problem gelöst Mein LoadButton sieht jetzt so aus:

button3.addActionListener(new ActionListener() {
          public void actionPerformed(ActionEvent e) {
             JFileChooser datei = new JFileChooser();
             int returnVal = datei.showOpenDialog(getComponent(0));
            
             BufferedReader in;
            try {
                in = new BufferedReader(new FileReader(datei.getSelectedFile()));
                String line1 = in.readLine();
                String line2 = in.readLine();
                String line3 = in.readLine();
                String line4 = in.readLine();
                String line5 = in.readLine();
                String line6 = in.readLine();
                textbox1.setText(line1);
                textbox2.setText(line2);
                textbox3.setText(line3);
                textbox4.setText(line4);
                textbox5.setText(line5);
                textbox6.setText(line6);
            } catch (FileNotFoundException e2) {
                // TODO Auto-generated catch block
                e2.printStackTrace();
            } catch (IOException e1) {
                // TODO Auto-generated catch block
                e1.printStackTrace();
            }
            
            
          }
        });

ist zwar unübersichtlich aber funktioniert!

Danke für deine Hilfe Robat!

 

[Robat]

Es wäre "schöner" die Textboxen in einer Liste abzuspeichern.
Dann könntest du auch sowas machen.

try(BufferedReader in = new BufferedReader(new FileReader(datei.getSelectedFile()))) {
    String line = "";
    List<String> lines = new ArrayList<>();
    while((line = in.readLine()) != null) {
        lines.add(line);
    }
    for(int i = 0; i < textboxes.size(); i++) {
         textboxes.get(i).setText(lines.get(i));    
    }
} catch(IOException e) {
   e.printStackTrace();
}

Oder mit Java 8:

try(BufferedReader in = new BufferedReader(new FileReader(datei.getSelectedFile()))) {
    List<String> lines = in.lines().collect(Collectors.toList());
    IntStream.range(0, textboxes.size()).forEach(i -> textboxes.get(i).setText(lines.get(i)));
} catch(IOException e) {
   e.printStackTrace();
}

 

[mihe7]

@xzlukas_ Du solltest Dir dann aber überlegen, ob Du beim Speichern wirklich an die Datei anfügen willst.