Mit FileReader lesen, in ArrayList speichern

[SpaceToon]

Hey Leute,
ich lerne momentan für eine Klausur und habe mir mal folgende zwei Szenarien gedacht:
1) Ich sol aus einer Text-Datei, die z.B. aus 10 Zeilen besteht und in jeder Zeile eine Zahl steht eine Zahl. Anschließend soll will ich diese Werte dann in eine ArrayList speichern. Mein Code dazu:

import java.io.FileReader;
import java.io.IOException;
import java.util.ArrayList;
import java.util.List;

public class CopyFile {

    public static void main(String[] args) throws IOException {

        FileReader reader = null;
        List<Integer> list = new ArrayList<>();

        try {
            reader = new FileReader("kopiere.txt");
            int c;

            while ((c = reader.read()) != -1) {

                list.add(c);

            }
        } catch (IOException e) {
            e.printStackTrace();
        } finally {

            if (reader != null) {
                reader.close();
            }
        }

        System.out.println(list);
    }

}

So, in meiner Text-File sind die Zahlen 1-10 zeilenweise geschrieben. Wenn ich mir nach dem ganzen die Liste ausgebe, erhalte ich folgende Einträge:

[49, 13, 10, 50, 13, 10, 51, 13, 10, 52, 13, 10, 53, 13, 10, 54, 13, 10, 55, 13, 10, 56, 13, 10, 57, 13, 10, 49, 48]

Wie kann das aber sein? Ich habe doch nur 10 Zeilen und die Zahlen 1 - 10 in der File, ich hätte erwartet, dass die ArrayList auch nur diese anzeigt.

2) Ich möchte aus derselben Datei lesen, die zahlen jeweils mit 2 multiplizieren und anschließend in eine andere Datei schreiben. Mein Code:

import java.io.FileReader;
import java.io.FileWriter;
import java.io.IOException;

public class CopyFile {

    public static void main(String[] args) throws IOException {

        FileReader reader = null;
        FileWriter writer = null;
        try {
            reader = new FileReader("kopiere.txt");
            writer = new FileWriter("einfuegen.txt");
            int c;

            while ((c = reader.read()) != -1) {
                int i = Integer.parseInt(String.valueOf(c));
                i = i * 2;
                writer.write(i);

            }
        } catch (IOException e) {
            e.printStackTrace();
        } finally {
            if (writer != null) {
                writer.close();
            }
            if (reader != null) {
                reader.close();
            }
        }

    }

}

Das Problem ist, dass ich dann in der Datei "einfuegen.txt" folgenden Eintrag habe:

bdfhjlnprb`

Das ist ja eigentlich mein Ergebnis, aber nur in Buchstaben (b=2, d=4 etc.)

Wo liegt mein Fehler?
Kann mir jemand mal genau erklären, wie vor allem die while Schleife funktioniert, die werte von c=reader.read() sind doch zufällig oder? Was hat das dann mit dem Zeilenweise auslesen zu tun? Verstehe das Prinzip nicht ganz.

 

[Meniskusschaden]

Wie kann das aber sein? Ich habe doch nur 10 Zeilen und die Zahlen 1 - 10 in der File, ich hätte erwartet, dass die ArrayList auch nur diese anzeigt.

Die (10, 13)-Paare sind die Zeilenschaltungen (Carriage Return / Line Feed).

Das ist ja eigentlich mein Ergebnis, aber nur in Buchstaben (b=2, d=4 etc.)

49*2 ergibt 98 und das ist der ASCII-Code von b.

Was hat das dann mit dem Zeilenweise auslesen zu tun?

Gar nichts. Du liest zeichenweise aus.

 

[Tarrew]

1). Du kriegst die ASCII-Werte deiner Zahlen und Zeichen wieder.
49 = '1'
13 = Wagenrücklauf
10 = Zeilenumbruch

Dann gehts weiter mit '2' und dem Zeilenumbruch "\r\n" und so weiter.
Die Zahl zu deinem ASCII-Wert kriegste zum Beispiel mit

Character.getNumericValue(c)

In deinem Fall ist vllt. eher ein BufferedReader sinnvoll, der die ganze Zeile liest:

Spoiler

    public static void copy() throws IOException {

        FileReader reader = null;
        List<Integer> list = new ArrayList<>();

        try {
            reader = new FileReader("kopiere.txt");

            BufferedReader br = new BufferedReader(reader);
            String line;

            while ((line = br.readLine()) != null) {
                list.add(Integer.parseInt(line));
            }
        } catch (IOException e) {
            e.printStackTrace();
        } finally {

            if (reader != null) {
                reader.close();
            }
        }

        System.out.println(list);
    }

2).
Ähnliches Problem.
Deine erste Zahl ist 1 (ASCII 49) -> String.valueOf(49) = "49" -> Wird zu int 49 geparsed und verdoppelt (98). Das ist dann der Buchstabe b der zurückgeschrieben wird.

Am besten das Ganze wieder mit der obigen Methode parsen und dann als String zurückschreiben, damit auch Zahlen weggeschrieben werden, die mehr als 1 Zeichen haben:

Spoiler

 public static void multiplyAndWrite() throws IOException {

        FileReader reader = null;
        FileWriter writer = null;
        try {
            reader = new FileReader("kopiere.txt");
            writer = new FileWriter("einfuegen.txt");
            int c;

            while ((c = reader.read()) != -1) {
                int i = Character.getNumericValue(c);
                i = i * 2;

                writer.write(""+i);
                writer.write("\n");

                //\r\n überspringen
                reader.skip(2);
            }
        } catch (IOException e) {
            e.printStackTrace();
        } finally {
            if (writer != null) {
                writer.close();
            }
            if (reader != null) {
                reader.close();
            }
        }

    }

Diesmal eine andere Variante um den Zeilenumbruch zu überspringen. Ist jetzt ein kleiner Bug drin, weil nur einstellige Zahlen jetzt richtig gelesen werden, aber ich denke, das kriegste auch gelöst

 

[Robat]

Dein Reader liest ein char ein und du wandels diesen char in ein int.. Was du da siehst sind nicht die Zahlen sondern der dezimal ASCII Wert (vgl. Ascii Tabelle). 13 10 musste CR LF sein (also der Zeilenumbruch).

Beim schreiben das selbe Problem nur umgekehrt.

 

[SpaceToon]

Vielen Dank für die Hilfe. Ich habe es nun verstanden und nun funktioniert auch der Code.