Datei lesen und Char für Char wiedergeben

[manuche]

Ich habe mir eine Klasse erstellt die ein BackUp für mich verwalten. Das funktioniert allerdings auch ganz gut...
Jetzt wollte ich das BackUp wieder einspielen! Dafür muss allerdings Charakter für Character gelesen werden.
Ich habe es mir so gedacht, dass an die Methode aus der Klasse BackUp übergeben wird, der wievielte Character es sein soll... Die Methode wiederum spuckt dann den Character aus...
Dazu habe ich mir einen FileReader angelegt. Das Problem beim FileReader ist allerdings, dass es nur die Methode read() gibt und die einen Integerwert liefert...
Bis jetzt siehts so aus:

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

public class BackUp {

  private static BackUp instance;
  private File sudoku = new File("backup.txt");
  private FileWriter fw;
  private FileReader fr;

  public static BackUp getInstance() {
    if (instance == null){
      instance = new BackUp();
    }
    return instance;
  }

  private BackUp() {
    try {
      fw = new FileWriter ("backup.txt", true);
    }catch (IOException e){
      System.err.println ("IOException beim Zugriff auf das BackUp-File: "+ e.getMessage());
    }
  }

  public void println (String s) {
    ...
  }
  
  public void print (String s) {
    ...
  }
  
  public boolean sudokuexists(){
    ...
  }
  
  public char aktchar(int stelle){
    char c;
    c = //wiedergabe aus einer methode anhand von x
    return c;
  }
}

Und die Abfrage dem entsprchend:

BackUp.getInstance().aktchar(int x);

 

[SlaterB]

die Rückgabe sollte für deine Zwecke ausreichen, wie schreibst du die chars denn in den Writer?

char c = (char) int;

um die vorherigen Zeichen zu überspringen: skip(int);

wenn du mehrere Zeichen lesen willst, verwende einen BufferedReader,
beim Schreiben solltest du unbedingt einen BufferedWriter verwenden, falls du zeichenweise schreibst

 

[manuche]

Das mit dem Writer ist kein Problem... Ich bekomme es nicht auf die Reihe char für char zu lesen. Wäre es sinnvoller den ganzen String zu lesen, dann zu übergeben und anschließend zu teilen?

 

[SlaterB]

> Ich bekomme es nicht auf die Reihe char für char zu lesen.

wenn du dazu noch Fragen hast, dann stelle sie,
du kennst read() und aus dem int kannst du char machen,
mehr kann man dazu erstmal nicht sagen

> Wäre es sinnvoller den ganzen String zu lesen, dann zu übergeben und anschließend zu teilen?

wenn schon dann lies ein char[] ein, die Operation gibts ja auch,

falls du einen BufferedReader verwendest, kannst du aber auch char für char lesen,
dann wird praktisch vom BufferedReader intern ein großer Teilstring (bzz. char[]) geladen,
was du sonst manuell machen müsstest

 

[manuche]

public char aktchar(int stelle){
    /*
    br = new BufferedReader(fr);

    return br.readLine();
    */
    char c;
    try{
      c = (char) fr.skip(stelle);
    }catch(IOException e){
      System.err.println ("IOException beim Zugriff auf das BackUp-File: " + e.getMessage());
    }
    return c;
  }

hab ich daraus jetzt gemacht...
leider mit der Meldung, dass c evtl nicht initialsiert ist... wie bekomm ich den Fehler weg? c = '' geht ja nicht!

 

[SlaterB]

c = ' ';

 

[manuche]

Wenn ich jetzt mit

BackUp.getInstance().aktchar(stelle);

drauf zurück greife kommt die fehlermeldung Exception in thread "main" java.lang.NullPointerException
Irgendwas ist nicht belegt aber ich weiss nicht was...

 

[SlaterB]

schön, dass du deine Gedanken anderen mitteilst

falls andere dir helfen sollen musst du Fragen stellen oder Code zu Exceptions posten,
falls obiger Code noch aktuell ist: Java sagt dir die Zeile in der der Fehler auftaucht,
prüfe alle Objekte, ob sie nicht null sind,

fr ist z.B. sehr verdächtig:
System.out.println("fr ist: "+fr);

 

[manuche]

Ok das hat geklappt... Hab vergessen dem FileReader fr eine Datei zuzuweisen! Also funktionieren tut es zwar jetzt fehlerfrei nur soll der eine char aus der Datei zurück in ein Textfeld geschrieben werden:

      int stelle = 1;
      for (int i = 0; i <= 8; i++){
        for (int j = 0; j <= 8; j++){
          char c = BackUp.getInstance().aktchar(stelle);
          matrix[i][j].setText("" + c);
          stelle++;
        }
      }

ich hab ein system.out.print eingabaut aber auch da: nichts!

 

[SlaterB]

wie nichts? wo nichts?
wenn du keinen einzigen char auslesen kannst,
wozu dann eine doppelte for-Schleife?

probiere erstmal EINEN Aufruf
BackUp.getInstance().aktchar(1);
ohne viel drumherum,

wenn das nicht klappt, dann musst du allen relevanten Code zusammentragen,
so dass man nachvollziehen kann, wie du die Daten einliest,
und eine nichtleere Datei brauchst du auch

 

[manuche]

Steht eigentlich schon alles da: also die Datei besteht quasi aus 81 Zahlen von 0 bis 9 hintereinander weggeschrieben.
Diese sollen dann in 9 x 9 Textfelder geschrieben werden soll.Die liegen in einem Arry... Deswegen die beiden for schleifen.
Aus der Datei lese ich wie folgt:

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

public class BackUp {

  private static BackUp instance;
  private File sudoku = new File("backup.txt");
  private FileWriter fw;
  private FileReader fr;
  private BufferedReader br;// = new BufferedReader(fr);

  public static BackUp getInstance() {
    if (instance == null){
      instance = new BackUp();
    }
    return instance;
  }
  
  private BackUp() {
    try {
      fw = new FileWriter ("backup.txt", false);
    }catch (IOException e){
      System.err.println ("IOException beim Zugriff auf das BackUp-File: "+ e.getMessage());
    }
  }

  public void println (String s) {
    ...
  }

  public void print (String s) {
    ...
  }

  public boolean sudokuexists(){
    boolean exists;
    if (sudoku.exists()){
      exists = true;
    }else{
      exists = false;
    }
    return exists;
  }

  public char aktchar(int stelle){
    char c;
    try{
      fr = new FileReader ("backup.txt");
      c = (char) fr.skip(stelle);
    }catch(IOException e){
      System.err.println ("IOException beim Zugriff auf das BackUp-File: " + e.getMessage());
      c = ' ';
    }
    return c;
  }
}

 

[SlaterB]

die Zeile
c = (char) fr.skip(stelle);
stand bestimmt noch nicht da, und die ist auch ganz schön daneben,
verwende

fr.skip(stelle-1); // überspringe stelle-1 chars
c = (char) fr.read(); // lies ein Zeichen

wieviel du überspringen musst, musst du erst testen, sind nur grobe Befehle, die ich da nenne,

--------
aber generell:
wenn du 81 Zeichen lesen willst, wieso dann 81x die Datei öffen, zur richtigen Stelle laufen und char lesen?
öffne lieber nur EINEN Reader und lies einfach nach und nach die Zeichen aus,
skip ist wahrscheinlich zu schwer für dich (mein Fehler das vorzuschlagen)

-------

und nochmal: auch wenn du diese Änderungen nun hast,
denke nicht daran, irgendeine GUI zu befüllen, irgendeine Matrix zu befüllen
oder irgendwelche Schleifen aufzubauen, gar zwei ineinander(!)

versuche ganz ganz simpel
char c1 = BackUp.getInstance().aktchar(0);
System.out.println("Zeichen an Stelle 0 ist: "+c1);

sowie das gleiche noch für ein paar andere Stellen, bzw. du sollst die Zeichen ja eh in Reihenfolge auslesen,
also

char c1 = BackUp.getInstance().aktchar();
System.out.println("Zeichen an Stelle 0 ist: "+c1);
usw.

wenn das zu funktionieren scheint, dann langsam langsam langsam komplizierteres bauen

 

[manuche]

Wenn ich dich jetzt richtig verstehe, soll ich direkt den Reader deklarieren und initialisiern....
also:
privat FileReader xyz = new FileReader("backup.txt");

Habs jetzt mal mit immerwieder aufrufen probiert so wie du es mir gesagt hast! Ergebnis war eine Ausgabe, nur leider ? statt Zahlen xD

 

[SlaterB]

hmm hmm,
was soll man da alles erklären,

mal ganz anders angefangen: vergewissere dich, dass die Datei vorhanden ist und merke dir die erste Ziffer dort,
was kommt bei

class Test
{
    public static void main(String args[])
        throws Exception
    {
        BufferedReader br = new BufferedReader(new FileReader("backup.txt"));
        int a = br.read();
        char c = (char)a;
        System.out.println("a: " + a + ", c: " + c);
    }
}

für eine Ausgabe (a und c)?

 

[manuche]

a: 48 c: 0
erste ziffer aus der textdatei ist 0 womit es wohl geglückt ist...

 

[SlaterB]

und nun bitte deinen aktuellen Code
public class BackUp {
..
}

+ den Aufruf

und ändere die erste Ziffer mal in 7 oder so, 0 kann ja überall rauskommen (a = 48 ist aber ziemlich gut)

 

[manuche]

mit einer anderen zahl geht es auch...
problem ist nur, dass wenn ich deinen codeschnipsel in meine methode einfüge werden die chars nicht übergeben... bei meinem aufruf kommen dann nur ? raus!

  public char aktchar(int stelle) throws Exception{
    BufferedReader br = new BufferedReader(new FileReader("backup.txt"));
    int a = br.read();
    char c = (char) a;
    //System.out.println("a: " + a + ", c: " + c);
    return c;
  }

 

[SlaterB]

was heißt beim Aufruf?
dort mit System.out.println ausgegeben oder unerlaubt kompliziert weiterverarbeitet?

wenn du meinst, dass es am Aufruf liegt,
dann wirf den Reader komplett raus und teste erstmal

public char aktchar(int stelle) throws Exception{
return '3';
}

 

[manuche]

mhh jetzt übergibt er die 3... musst aber bei zwei methoden in meiner Klasse vom Programm ein "throws Exception" dran hängen... man man man garnich so einfach zum einstieg -.-
sobald ich allerding return c sage übergibt er wieder nur Fragezeichen...

 

[Rogge]

upss... falscher beitrag *fg

Sorry

 

[SlaterB]

dann noch mal im direkten Vergleich:
schreibe in die Datei 3 an erste Stelle,

prüfe dein Programm mit

public char aktchar(int stelle) throws Exception{
   // oder auch ohne Exception mit try/catch

  char c = '3'

  if (c != '3') {
    throw new RuntimeException("alles doof");
  } else {
    System.out.println("char ist: "+c);
    return c; 
  }
}

vs

public char aktchar(int stelle) throws Exception{
   // oder auch ohne Exception mit try/catch

  BufferedReader br = new BufferedReader(new FileReader("backup.txt")); 
    int a = br.read(); 
    char c = (char) a; 
    System.out.println("a: " + a + ", c: " + c); 

  if (c != '3') {
    throw new RuntimeException("alles doof");
  } else {
    System.out.println("char ist: "+c);
    return c; 
  }
}

was kommt bei beiden jeweils für Ausgaben und was kommt beim Aufrufer an?

Aufruf bitte per
char c = ...aktchar(0);
System.out.println("Aufrufer erhält: "+c);

 

[manuche]

beim ersten kommt "char ist 3" auf der Konsole raus...
beim zweiten:

a: -1, c: ?
Exception in thread "main" java.lang.RuntimeException: alles doof

 

[SlaterB]

na dann klappt doch das Einlesen nicht,

das widerspricht
> mit einer anderen zahl geht es auch...
> problem ist nur, dass wenn ich deinen codeschnipsel in meine methode einfüge werden die chars nicht übergeben...
von vorhin, oder du hast es vorhin gar nicht in deinem Programm getestet!!

du hast also ein Problem mit dem Einlesen bzw. mit der Datei,
vielleicht wird die im Programm überschrieben..

ist nach der Programmausführung immer noch die 3 am Anfang der Datei,
so wie du sie reingeschrieben hast?

 

[manuche]

Ich hab nen bisschen rum probiert und jetzt gehts... leider kann ich nicht mehr sagen, woran genau es jetzt lag! vermutlich wie gesagt am reader oder so...
hab allerdings ein neues problem! und zwar funktioniert im folgenden die methode filedelete nicht! also sie gibt immer false wieder:

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

public class BackUp {

  private static BackUp instance;
  private File sudoku = new File("backup.txt");
  private FileWriter fw;
  private FileReader fr;
  private BufferedReader br;// = new BufferedReader(fr);

  public static BackUp getInstance() {
    if (instance == null){
      instance = new BackUp();
    }
    return instance;
  }

  public void print (String s) {
    try{
      fw = new FileWriter (sudoku, true);
      fw.write (s);
      fw.flush();
    }catch (IOException e) {
      System.err.println ("IOException beim Zugriff auf das BackUp-File: " + e.getMessage());
    }
  }

  public boolean sudokuexists(){
    boolean exists;
    if (sudoku.exists()){
      exists = true;
    }else{
      exists = false;
    }
    return exists;
  }
  
  public char aktchar(int stelle) throws Exception{
    br = new BufferedReader(new FileReader (sudoku));
    br.skip(stelle);
    int a = br.read();
    char c = (char) a;
    return c;
  }
  
  public void filecreate(){
    try{
      sudoku.createNewFile();
    }catch (Exception e){
      //
    }
  }
  
  public void filedelete(){
    try{
      sudoku.delete();
    }catch (Exception e){
      //
    }
  }
}

Weiss jemand woran es liegen könnte? eigentlich ist immer ein file vorhanden, dass gelöscht werden kann!

Also laufzeittechnisch siehts so aus:
das programm liest beim start aus dieser datei soweit sie vorhanden ist und löscht sie anschliessend. falls sie nicht vorhanden ist, wird sie beim beenden erzeugt und in sie hineingeschrieben.
leider wird die datei nicht gelöscht sondern immer weiter in sie reingeschrieben...

 

[20mithrandir]

1. close() fehlt (Writer)

2. filedelete() liefert "false" -> Wie muss man das verstehen? Die Methode hat ja keinen Rückgabewert...

3. Exceptions fangen und ignorieren -> phöse

 

[manuche]

> 1. close() fehlt (Writer)
Ok, vergessen...

> 2. filedelete() liefert "false" -> Wie muss man das verstehen? Die Methode hat ja keinen Rückgabewert...
Da war ich wohl wieder zuvoreilig....
sudoku.delete() gibt false wieder!

> 3. Exceptions fangen und ignorieren -> phöse[/quote]
habs auch erst mit "throws Exception" probiert aber da hat der Compiler gemeckert wenn ich die Methode aus einer anderen Klasse heraus aufgerufen wurde!

 

[SlaterB]

> habs auch erst mit "throws Exception" probiert

ziemlich schräge Begründung, wenn du doch an anderer Stelle bereits das deutlich bessere und genauso endgültige

}catch (IOException e) {
System.err.println ("IOException beim Zugriff auf das BackUp-File: " + e.getMessage());
}

hast

---------

> sudoku.delete() gibt false wieder!

auch jetzt noch nach dem close() (und wenigstens ein paar ms Pause)?

 

[manuche]

also ich hab jetzt zumindest das System.out.print(...); im catch-zweig!
es läuft nach dem close alles so wie ichs mir vorgestellt hab!
super!!!
Und auf jeden Fall vielen Dank für eure Geduld und Hilfe!!

Gruß manuche

 

[Leroy42]

Na das ist ja mal 'ne geniale Methode:

  public boolean sudokuexists(){ 
    boolean exists; 
    if (sudoku.exists()){ 
      exists = true; 
    }else{ 
      exists = false; 
    } 
    return exists; 
  }

:shock:

Ob man da wohl noch etwas straffen könnte?

  public boolean sudokuexists(){ 
    boolean exists; 
    exists = sudoku.exists();
    return exists; 
  }

So! Ähh, noch ein Wenig...

  public boolean sudokuexists(){ 
    boolean exists = sudoku.exists();
    return exists; 
  }

Eine Kleinigkeit...

  public boolean sudokuexists(){ 
    return sudoku.exists();
  }

...noch ne Winzigkeit...

  sudoku.exists();

Uppss! Jetzt ist mir die Methode selbst bei der
gesamten Herumstrafferei abhanden gekommen! :shock: