FileWriter bzw. Reader fehlerhaft

[Cev Droon]

Hi Leute,

Ich habe ein Problem bezüglich der Serialisierung von Strings mit dem FileWriter und FileWriter. In folgendem Code kommt nach dem kompilieren und ausführen nur ein "?" heraus!

package Speichern;

import java.io.*;

public class Klasse {

	public static void main(String[] args) {
		new Klasse().savePlayerCoo();
		new Klasse().readPlayerCoo();

	}

	int xPlayer = 800;
	int yPlayer = 160;

	Writer fw;

	Reader reader;

	public void savePlayerCoo() {

		int xxPlayer;
		int yyPlayer;

		xxPlayer = xPlayer;
		yyPlayer = yPlayer;

		try {
			fw = new FileWriter("PlayerCoo.txt");
			fw.write(xxPlayer);
			fw.write(yyPlayer);
			fw.append(System.getProperty("line.separator")); // e.g. "\n"
		} catch (IOException e) {
			System.err.println("Konnte Datei nicht erstellen");
		} finally {

			if (fw != null)
				try {
					fw.close();
				} catch (IOException e) {
					e.printStackTrace();
				}
		}
	}

	public void readPlayerCoo() {
		try {
			reader = new FileReader("PlayerCoo.txt");

			for (int c; (c = reader.read()) != -1;)
				System.out.print((char) c);
		} catch (IOException e) {
			System.err.println("Fehler beim Lesen der Datei!");
			e.printStackTrace();
		} finally {
			try {
				reader.close();
			} catch (Exception e) {
				System.out.println(xPlayer + yPlayer);
			}
		}
	}

}

Weiß jemand, wieso nicht die x - und y - Koordinaten ausgegeben werden? Falls es bei der Problemlösung hilft, ich arbeite nach dem Buch "java head first".

Danke :toll:

Cev Droon

 

[fisherman]

for (int c; (c = reader.read()) != -1;)
	System.out.print((char) c);

[STRIKE]Die increment expression fehlt.[/STRIKE] (Quatsch.) [STRIKE]Abgesehen davon verwendest du hier besser eine while-Schleife, weil du nicht wissen kannst, wie viele Zeichen der Reader lesen muss/kann.[/STRIKE] Naja, while kommt als Konstrukt in diesem Fall häufiger vor als for.

 

[Harry Kane]

Ersetze einfach "(char)c" durch "c" in deiner Ausgabemethode.
Die char´s in der Zeichentabelle an Position 160 und 800 sind wahrscheinlich nur sehr eingeschränkt druckbar.

 

[Cev Droon]

Erst mal danke, Harry und fisherman, das Ergebnis hat sich schon mal geändert :toll:

Nur leider hat sich das Ergebnis von "?" nach "631601310" geändert

public void readPlayerCoo() {
		try {
			reader = new FileReader("PlayerCoo.txt");

			for (int c; (c = reader.read()) != -1;)
				System.out.print(c);
		} catch (IOException e) {
			System.err.println("Fehler beim Lesen der Datei!");
			e.printStackTrace();
		} finally {
			try {
				reader.close();
			} catch (Exception e) {
				System.out.println(xPlayer + yPlayer);
			}
		}

Die Methode zum Lesen der Koordinaten des Spielers aus der Datei habe ich, wie von Harry empfohlen, geändert, nur leider kommt immer noch nicht das gewünschte Ergebnis ;(

Weiß jemand weiter?

Cev Droon

 

[kaoZ]

Öhm nur am Rande, du redest hier von Serialisierung , ich sehe hier allerdings nirgends die Nutzung eines

FileOutputStreams

bzw. eines

ObjectOutputStreams

Du schreibst lediglich die Strings in eine Datei wenn ich mich jetzt nicht vollkommen verlesen habe.

Zudem nutzt du nicht die funktion die Ein- und Ausgaben zu Buffern und in einem Rutsch in die Datei zu schreiben bzw. zu lesen.

wenn dann würde ich zumindest einen

BufferedWriter

/

BufferedReader

verwenden .

Sinnvoller wäre es allerdings , dementsprechend das Objekt (Player) mit samt seinen Attributen und deren Zuständen in eine Datei zu Schreiben, und somit Persistent zu Speichern. (Serialisierung)

Wenn du dazu Fragen haben solltest , helfe ich dir gern weiter.:toll:

 

[Cev Droon]

Ja, ich habe eine Frage, kaoZ:

Du hast geschrieben, du würdest die BufferedReader- und writer verwenden.

Soll, ich das jetzt tun, und hilft es zur Problemlösung, oder war es bloß eine Empfehlung?

Danke,
Cev Droon

 

[kaoZ]

Dies war lediglich ein Ratschlag da es sinniger ist die Ausgabe als vollkommenes ganzes statt in jedes char für sich einzeln in die Datei zu schreiben.

Der BufferedReader und Writer verwenden ein charArray zur pufferung bzw. als cache , der es ermöglicht Eingaben und Ausgaben erst vollständig in den Assoziativspeicher zu laden und dann in einem rutsch aus diesem zu verarbeiten.

Bei Nutzung eines normalen Readers oder Writers, wird jeder Buchstabe für sich allein in die Datei geschrieben oder aus ihr gelesen, dies verursacht wesentlich mehr Auslastungen als eben die Nutzung eines puffers.

Zu deinem Problem:

Wenn du die Belegung der Attribute deines Objektes in PlainText in eine Datei Schreiben möchtest kannst du dies wie in diesem Beispiel machen ( ich habe um es übersichtlich zu halten auf Exception Handling verzichtet , dies sollte mal bei einer konkreten Implementierung niemals machen!)

Spoiler: Writer

package tests;

import java.io.BufferedWriter;
import java.io.File;
import java.io.FileWriter;
import java.io.IOException;

public class Writer{
	
	String text; // Lazy initialisation
	File file;
	
	public Writer(String filename) {
		text = "Hallo Welt";
		file = new File(filename);
	}
	
	public void writeTextToFile() throws IOException{
		BufferedWriter writer = new BufferedWriter(new FileWriter(file));
		
		writer.write(text);
		writer.close();
	}

	public static void main(String[] args) throws IOException{
		new Writer("H:/Apps/HalloWelt.txt").writeTextToFile();
	}
}

und ein Reader könnte so aussehen:

Spoiler: Reader

package tests;

import java.io.BufferedReader;
import java.io.File;
import java.io.FileReader;
import java.io.IOException;

public class Reader {
	
	File file;
	
	public Reader(String filename) {
		file = new File(filename);
	}
	
	public void readDataFromFile() throws IOException{
		BufferedReader reader = new BufferedReader(new FileReader(file));
		
		String line = "";
		
		while((line = reader.readLine()) != null){
			System.out.println(line);
		}
		
		reader.close();
		
	}
	
	public static void main(String[] args) throws IOException{
		new Reader("H:/Apps/HalloWelt.txt").readDataFromFile();
	}
}

falls du Fragen haben solltest, auch bezüglich der Realisierung zur Persistenten Speicherung des Objektes (Serialisierung) und nicht nur dessen Wertbelegungen als PlainText, immer raus damit