Problem bei Schiffeversenken-Spiel

[Masipulami]

Hallo zusammen,

ich habe mich die letzten Tage mal daran gesetzt ein Schiffeversenken-Spiel zu programmieren. Das Spiel ansich läuft so weit prima.

Jetzt habe ich das Ganze auch noch so programmiert, dass der aktuelle Score immer zwischengespeichert wird und dann am Ende ausgegeben wird. Der Score wird auch korrekt gespeichert.

Bei der Ausgabe klappt allerdings etwas noch nicht.

Ich wollte das folgendermaßen machen:

Nachdem das letzte Schiffsfeld aufgedeckt wurde, wird der Benutzer aufgefordert seinen Name einzugeben.

Dieser Name wird dann mit dem Score im aktuellen Fenster ausgegeben (also dort wo vorher noch das Spielfeld war (hat gestern auch schon mal funktioniert; das Spielfeld wird dann einfach nur "überzeichnet") und in der Datei highscore.txt gespeichert.

Dabei soll nach jedem Spiel der Name und Score in einer neuen Zeile in der Datei gespeichert werden.

Beim nächsten Klick soll dann diese Datei wieder ausgelesen werden und deren Inhalt (also alle gespeicherten Highscores) wieder im aktuellen Fenster (wieder mittels "Überzeichnen") ausgegeben werden.

Von der Funktionalität her sollte das ja kein Problem sein. Nur leider habe ich noch einen Fehler im Code und der Inhalt der Datei bleibt leer. Ganz egal, was ich eingebe.

Die IF-Anweisungen in der Paint-Methode müssten so ja eigentlich laufen.

Dass der Score im Fenster angezeigt wurde hat heute Abend schon auf diese Weise geklappt.

Es würde mich sehr freuen, wenn ihr euch mal den Code ansehen würdet.

Ich finde den/die Fehler einfach nicht.

Viele Grüße,

Masipulami

import java.io.*;

import java.util.*;

import java.awt.*;

import java.awt.event.*;


class SchiffeVersenken extends Frame

{

   public int zaehler = 0;

   final int MAXSPALTE = 16, MAXZEILE = 16; // Geometrie des Spielfelds


   Image bild0, bild1, bild2, bild3, bild4, bild5, bildT; // Referenzen auf die zu ladenden Bildchens


   int maxX = 0; // maximaler x-Wert, dient zur Errechnung der notweindigen Fenstergröße


   Point aktMausPunkt = new Point(0,0); // Koordinaten des letzten Mausklicks


   // Sichtbarkeitsmatrix - zeigt an ob an dieser Stelle das Spielbrett sichtbar ist (true) oder nicht (fals)

   boolean sm[][] = new boolean[MAXSPALTE][MAXZEILE];


   // Wertematrix des Spielfelds. Wird mit Hilfe von Zufallszahlen im Kostruktor erzeugt.

   int spielFeld[][] = null;


   int feld1 = 0; int feld2 = 0;

   int scoreZähler = 0; // Variable, die den aktuellen Score speichert


   final int ABSTAND=4; // Abstand des Bildes von oben, unten, links rechts


   int schiffsfeldzaehler = 0;  // zählt alle Felder aller Schiffe zusammen

   int wert;  // Wert, der an jeweiligen Stelle des mit Random() erzeugten Arrays steht

   Font f;  // für Highsoreausgabe

   public int x=50;  // Position der Textausgabe

   public int y=50;  // Position der Textausgabe

   String ausgabe; // Hier wird Aufgabestring von Punktestand des Scorzählers gespeichert

   BufferedReader stdin = new BufferedReader(new InputStreamReader(System.in));

   String name; // für eingegebenen Name (für Highscore)

   String highsc; // wird später so verwendet: highsc = name + " " + ausgabe;

   int c;


   public static void main (String[] args)

   { 

   SchiffeVersenken re = new SchiffeVersenken();

   }


   public SchiffeVersenken()

   {

     super("Bild Anzeigen");


     f = new Font("Verdana", Font.PLAIN, 24);


     addWindowListener(new WindowClosingAdapter(false));


      // Schliesst den MouseListerner mit eigener Klasse (in der gleichen Datei unten!) an

     addMouseListener(new myMausListener());


     setBackground(new Color(0,22,0));   // Hintergrundfarbe setzen


     setSize(200,200);   // die Größe ist nur vorläufig, wird in paint() auf den endgültigen Wert geändert


     Random zufallsGenerator = new Random();

     int[][] spielFeld = new int[16][16];

     int zufallsZahl = 0;    // Variable speichert die Zufallszahl

     int hilfsVariable = 0;  // Hilfsvariablen sollen einen reibungslosen Ablauf der

     int hilfsVariable2 = 0; // Arrayaufnahme der unterschiedlichen Schiffe sichern.

     int zähler = 0;         // Zähler, der die Anzahl der Schiffe zählt.



     // Hier werden im Array mittels Zufallsgenerator die Schiffe platziert.

     // Somit sind die Schiffe bei jedem Start an einer anderen Position.

     for(int i = 0; i <16; i++)

     {

      for(int j = 0; j < 16; j++)

      {

            // Hier werden die 2er, 3er, 4er und 5er Felder im Array aufgenommen.

         if(hilfsVariable == 2 | hilfsVariable == 3 | hilfsVariable == 4 | hilfsVariable == 5)

         {

            spielFeld[i][j] = zufallsZahl;

            hilfsVariable2--;


            if(hilfsVariable2 == 1)

            {

            hilfsVariable = 0;

            zähler++; // Zähler der die Anzahl der Schiffe hochzählt,

            }        // wenn ein 2er, 3er, 4er oder 5er Schiff platziert wird.

         }


         else

         {

            // Eine Zufallszahl wird generiert. Die Zufallszahö ist deshalb so hoch damit nicht

            // zu viele Schiffe auf dem Spielfelsd sind.

            zufallsZahl = zufallsGenerator.nextInt(30);                                  


                // Wenn die Zufallszahl größer 5 ist dann wird sie genullt, also leeres Feld.

            if(zufallsZahl > 5)

            zufallsZahl = 0;


                // Sorgt dafür, dass kein zu großes Schiff vor Ende der Spalte gezeichnet wird.

            if(zufallsZahl + j > 15)

            zufallsZahl = 0;


            if(zufallsZahl == 1)

                // Zähler der die Anzahl der Schiffe hochzählt, wenn ein 1er Schiff platziert wird.

            zähler++;

            spielFeld[i][j] = zufallsZahl; // Hier werden die leeren Felder und die 1er Felder ins Array aufgenommen.

            hilfsVariable = zufallsZahl; //Hilfsvariablen die einen reibungslosen Ablauf,

            hilfsVariable2 = zufallsZahl;// der 2er, 3er, 4er und 5er Felder bei der Arrayaufnahme sicherstellen.

         }

      }

     }


     // Nachdem der Spielfeldarray nun mit Zufallswerten gefüllt wurde, werden die

     // Schiffsfelder ausgezählt, also alle Felder, auf denen ein Teil eines Schiffes ist.

     for(int k = 0; k <16; k++)

     {

      for(int l = 0; l <16; l++)

      {

         int wert = spielFeld[k][l];


         if(wert == 1 | wert == 2 | wert == 3 | wert == 4 | wert == 5)

         {

            schiffsfeldzaehler++;

         }

      }

     }


    System.out.println("============================");

    System.out.println("==    Schiffeversenken    ==");

    System.out.println("============================");

    //    Bei der Erstellung des Frames wird die Anzahl der erzeugten Schiffe ausgegeben 

    System.out.println("Anzahl der Schiffe: " +zähler);

    // GIbt die Anzahl der Schiffsfelder aus

    System.out.println("Anzahl der Schiffsfelder: " + schiffsfeldzaehler);

    System.out.println("============================\n");

    //System.out.println();


    // Weist das Konstruktorarray dem Klassenarray zu

    this.spielFeld = (int[][]) spielFeld.clone();


    // Bilder werden geladen

    bild0=getToolkit().getImage("feld0.gif");

    bild1=getToolkit().getImage("feld1.gif");       

    bild2=getToolkit().getImage("feld2.gif");

    bild3=getToolkit().getImage("feld3.gif");

    bild4=getToolkit().getImage("feld4.gif");

    bild5=getToolkit().getImage("feld5.gif");

    bildT=getToolkit().getImage("feldT.gif");


    setVisible(true);   

  }


  // wird von myMausListener  aufgerufen, und setzt die Referenz der Variablen aktMausPunkt

  // auf das Point-Objekt mit den aktuellen Daten des letzten Mausklicks

  public void addPunkt(Point p)

  {

     this.aktMausPunkt = p;

  }

...FORTSETZUNG SIEHE UNTEN

[Masipulami]

HIER GEHTS WEITER...

  private void feldBerechnen()

  {

     int insOben = getInsets().top;

     int insLinks = getInsets().left;

     int bildBreite = bild0.getWidth(this);

     int bildHoehe = bild0.getHeight(this);         

     int x=-1,y=-1;


   /*

         Nachdem die Insets des Fensters und Höhe und Breite der Button-Bilder ermittelt wurden, werden nun aus den Mauskoordinaten

         die Koordinaten des angeklickten Feldes errechnet.


         Algorithmus:

         der x-Wert (d.h. die linke Kante) des beispielsweise 5. Bildes errechnet sich aus

         insetLinks + (5*Abstand-zwischen-Bildern) + 4*Bildbreite (5 Abstände und 4 Bilder liegen rechts des 5. Bildes)

         Dieser Wert ist die Linke Kante und muss kleiner sein als der x-Wert des Mausklicks, damit das 5. Bild angeklickt sein kann


         Die rechte Kante des beispielsweise 5. Bildes errechnet sich aus

         insetLinks + (5*Abstand-zwischen-Bildern) + 5*Bildbreite       ( d.h. Linke Kante + Bildbreite)   

         Dieser Wert muss größer sein als der x-Wert des Mausklicks, damit dieses Bild angeklickt sein kann


         .. analoges gilt für die y-Werte


         Wurde zwischen die Felder geklickt wird für beide Koordinaten -1 ausgegeben


        */



     for ( int i =0; i < MAXSPALTE; i++)

      for ( int j =0; j < MAXZEILE+1; j++)

      if ( aktMausPunkt.x > insLinks +  (i+1)* ABSTAND + i*bildBreite &&  aktMausPunkt.x < insLinks + (i+1)*ABSTAND + (i+1)* bildBreite &&

          aktMausPunkt.y > insOben + (j+1)* ABSTAND + j*bildHoehe &&  aktMausPunkt.y < insOben + (j+1)*ABSTAND + (j+1)*bildHoehe )

     {

       x = i;

       y = j;

       this.feld1 = x; // Speichert den angklickten Punkt in der Klassenvariablen

       this.feld2 = y; //   Speichert den angklickten Punkt in der Klassenvariablen

     }   


    // wenn kein Feld getroffen wurde (d.h. es wurde ins Wasser neben den Schiffen geklickt) wird auch nix eingetragen

    // anderfalls wird das jeweilige Feld in der Sichtbarkeitsmatrix auf true gesetzt, d.h. an dieser Stelle kann der

    // Eintrag im Spielfeld (sp-Matrix) ausgegeben werden

    if ( x != -1) // Feld getroffen

        // der angelickte Punkt wird als true in der Sichtbarkeitsmatrix wird auf true gesetzt

       sm[x][y] = true;


    System.out.println("getroffen: "+ (x+1) + " - " + (y+1));


  }


    // Gibt die Klassenvariable zurück. Hab das müssen so programmieren da der Compiler es bei

    // mir anders nicht gemacht hat

     private int xKoordinate()

   {

      return this.feld1;

   }


    // Gibt die Klassenvariable zurück.

   private int yKoordinate()

   {

      return this.feld2;

   }


   private void scoreBerechnen()

   {

        // Gibt den Wert des Arrays zurück auf den geklickt wurde

      int bildKoordinate = spielFeld[xKoordinate()][yKoordinate()];


        // Jetzt wird noch je nach Klick der Score ausgerechnet und der Schiffsfeldzähler

      // um 1 herabgesetzt, damit später (wenn der Schiffsfeldzähler bei 0 angekommen ist

      // die Eingabeaufforderung, das Ausgaben des Scores und das Speichern des Scores

      // in einer Datei ausgeführt werden kann -> siehe Paint()

      if(bildKoordinate == 0)

      {

         scoreZähler = scoreZähler -2;

      }


      if(bildKoordinate == 1)

      {

         scoreZähler = scoreZähler +1;

         schiffsfeldzaehler--;

      }


      if(bildKoordinate == 2)

      {

         scoreZähler = scoreZähler +2;

         schiffsfeldzaehler--;

      }


      if(bildKoordinate == 3)

      {

         scoreZähler = scoreZähler +3;

         schiffsfeldzaehler--;

      }


      if(bildKoordinate == 4)

      {

         scoreZähler = scoreZähler +4;

         schiffsfeldzaehler--;

      }


      if(bildKoordinate == 5)

      {

         scoreZähler = scoreZähler +5;

         schiffsfeldzaehler--;

      }


      System.out.println("Aktueller Score: " + scoreZähler);   

      System.out.println("Verbleibende Schiffsfelder: " + schiffsfeldzaehler);

      System.out.println();


      ausgabe = "Punktestand: " + scoreZähler;

   }



  public void paint(Graphics gra)

  {         

     if(schiffsfeldzaehler > 0)

   {

         //   die x-Koordinate des lo-Ecks des Bildes wird errechnet: Inset-Wert plus (selbstdefinierter) ABSTAND

       int aktX=ABSTAND + getInsets().left; 

        // die y-Koordinate des lo-Ecks des Bildes wird errechnet: Inset-Wert plus (selbstdefinierter) ABSTAND                                                      

       int aktY = ABSTAND + getInsets().top;   


       // nun werden für alle möglichekn Koordinatenwertepaare das jeweilige bild angezeigt

       // für alle Zeilen und für alle Spalten einer Zeile wird mit bildSuchen das für

       // den Koordinateneintrag passende Bild gesucht und an der errechneten Stelle ausgegeben

       for ( int j = 0; j < MAXZEILE; j++)

       { 

           for (int i = 0; i < MAXSPALTE; i++)

           {

              //   bildSuchen gibt das passende Bild für die Koordinaten zurück

              // aktX und aktY sind die errechneten Pixel-Koordinaten des Bildecks links oben    

              gra.drawImage( bildSuchen(i,j),aktX, aktY,this);


              aktX += bild0.getWidth(this) + ABSTAND; // x weiterschalten 

              maxX = aktX;

           }

       aktY += bild0.getHeight(this) + ABSTAND;// y weiterschalten

       aktX=ABSTAND + getInsets().left;           

       }     


      aktX = maxX + getInsets().right;

      aktY = aktY + getInsets().bottom;


      this.setSize(aktX, aktY); // und das Fester wird auf die neue Größe eingestellt

   }


   if(schiffsfeldzaehler == 0)

   {

      gra.setColor(new Color(188,0,0));

       gra.setFont(f);


       //ausgabe = "Punktestand: " + scoreZähler;


       gra.drawString(ausgabe, x, y);

       this.setSize(200, 200);


       try

      {

          BufferedWriter f = new BufferedWriter(new FileWriter("highscore.txt"));


          System.out.print("Bitte geben Sie Ihren Name ein: ");

          name = stdin.readLine();

          highsc = name + " " + ausgabe;

          f.write(highsc);

          f.newLine(9;

          f.flush();


          f.close();


      }

       catch (IOException ex) {};

   } 


   if(schiffsfeldzaehler < 0)

   {

      gra.setColor(new Color(188,0,0));

       gra.setFont(f);

       this.setSize(200, 200);


       try

      {

          FileReader f = new FileReader("highscore.txt");


          while ((c = f.read()) != -1)

            {

             int z = 0;

             char highscArr[];

             highscArr[z]=(char)c;

             z++;

            }

            f.close();



            gra.drawString("HIGHSCORELISTE", x, y);

            gra.drawString(ausgabe, x, y);


      }

       catch (IOException ex) {};

   }



  }


   /*

         aus paint() ausgegliederte Methode - um paint() nicht riesig werden zu lassen

         errechnet aus den x, und y Koordinaten in der Spiele-Matrix - nicht den Mauskoordinaten(!!)


   */

   Image bildSuchen(int x, int y)

   {

      int matrixWert = spielFeld[x][y]; 


      if( sm[x][y]) // Eintrag in Sichtbar-Matrix ist true - Feld soll sichtbar sein

        switch(matrixWert)

        {

          case 0: return bildT;

          case 1: return bild1;

          case 2: return bild2;

          case 3: return bild3;

          case 4: return bild4;

          case 5: return bild5;     

       }

      else

       return bild0;



      return bild0;


   }


class myMausListener  extends MouseAdapter

{



      public void mousePressed(MouseEvent event)

      {


           SchiffeVersenken fenster = (SchiffeVersenken) event.getSource();


        Point aktPoint = new Point(event.getX(),  event.getY());


        fenster.addPunkt(aktPoint);

        fenster.feldBerechnen();

        fenster.scoreBerechnen();

        fenster.repaint();


        zaehler++;

      }

}

}

Hier in dem Teil wird mir auch ein Fehler angezeigt:

try

      {

          FileReader f = new FileReader("highscore.txt");


          while ((c = f.read()) != -1)

            {

             int z = 0;

             char highscArr[];

             highscArr[z]=(char)c;

             z++;

            }

            f.close();



            gra.drawString("HIGHSCORELISTE", x, y);

            gra.drawString(ausgabe, x, y);


      }

       catch (IOException ex) {};

Und zwar in dieser Zeile:

highscArr[z]=(char)c;

Schiffe versenken.rar

[Masipulami]

Der Teil ist auch noch nicht ganz fertig.

Ich stell mir das so vor, dass mit der while-Schleife oben wieder alles, was in der Datei steht, in den Array gespeichert wird. Dieser danach in nen String umgewandelt wird und dann als gesamte Highscoreliste ausgegeben wird.

Oder gibt es eine Möglichkeit einfach Zeilenweise aus der Datei zu lesen, diese Zeilen dann jeweils in nem StringBuffer zu speichern und halt mit append immer an den StringBuffer anzuhängen, diesen StringBuffer dann in nen String umwandeln und den dann im Fenster mittels drawString auszugeben.

Nur wie muss dazu mein Code aussehen?

Komisch ist auch, dass

a) Die Datei leer bleibt und

Die Aufforderung den Name einzugeben immer wieder kommt. Also wie in einer Endlosschleife, obwohl die Anweisung den Name einzugeben nicht in einer Schleife steht.

Muss das Programm halt unbedingt in den nächsten Tagen zum Laufen bringen. Ist für die Uni und wird bewertet.

Hab die benötigten Dateien mal noch in den Anhang meines zweiten Postings gepackt. Musste das in drei Postings aufteilen, weil die Beitragslänge ja auf 10.000 Zeichen begrenzt ist.

Zum Testen kann man die Größe des Spielfelds ja verkleinern.

Einfach überall, wo 16 steht z.B. ne 4 draus machen.

Dann muss man nicht mehr so viel klicken.

Viele Grüße,

Masipulami

[Kaeyu]

also erstma :uli

ich progge auch seit einem halben jahr mit java, und ich bin ma kurz über deinen code drüber geflogen.

dazu is mir bis jetzt nur aufgefallen( vielleicht hab ich es auch übersehen), aber in deinen bedingungen vom 2. teil, wird die variable "schiffsfeldzaehler" überprüft. dazu muss ich sagen, ich habe (noch) nicht gesehen, dass du diese variable vorher deklariert und definiert hast.

(kann auch sein, dass ich da falsch liege, aber naja...einen versuch war es wert.

wenn ich etwas mehr zeit habe check ich das mal gründlicher^^")

un da die variable ja nicht definiert wurde, erhält sie ja den wert 0, und weil sie dann diesen wert immer hat, kann man auch immer und immer wieder den namen eingeben.

variable = 0, dann name eingeben.

dadurch entsteht dann ja automatisch eine schleife, weil sich der variablen wert nicht ändert.

[Masipulami]

Danke schon mal für die Antwort.

Doch doch. Die Variable hab ich oben im Kopf deklariert:

int schiffsfeldzaehler = 0; // zählt alle Felder aller Schiffe zusammen

[geloescht_Newlukai]

Klar, daß 'ne Exception geworfen wird. Ich vermute mal ... eine IndexOutOfBoundsException?

Liegt wohl daran, daß Du zwar das Array highscArr initialisierst, aber keine Größe angibst.

Das mehrfache Auftreten der "Bitte Namen eingeben"-Meldung könnte daran liegen, daß Du das ganze in die paint-Methode geschrieben hast.

[Masipulami]

So, habs hinbekommen.

Hab die Dateien mal zum Testen in den Anhang gepackt.

Die Namenseingabe musste ich aus der Paint-Methode rausnehmen.

Keine Ahnung warum. So gehts jetzt aber.

Einziges Problem. Nach jedem Spiel wird die Highscoreliste überschrieben.

Vielleicht findet ja einer von euch den Fehler. Kann ja nur ne Kleinigkeit sein.

Normalerweise hat das Spielfeld auch 16x16 Felder.

Das hab ich jetzt nur zum Testen auf 4x4 beschränkt.

Bei 16x16 ist ein Schiff auch immer in einer Spalte.

Noch ne Frage:

Ich hab mal gesehen, dass man das mit der Eingabe auch so machen kann, dass ein kleines Fenster dafür aufgeht, in dem man den Name eingeben kann.

Waren nur 3 Zeilen Code oder so.

Jmd. ne Ahnung wie das geht? Würde das Ganze noch ein bisschen verschönern.

Schiffe versenken.rar

[geloescht_Newlukai]

Die Namenseingabe musste ich aus der Paint-Methode rausnehmen.

Keine Ahnung warum. So gehts jetzt aber.

paint() wird imer dann ausgeführt, wenn das Fenster gezeichnet werden muß, also bei jedem Verschieben, etc ...

Einziges Problem. Nach jedem Spiel wird die Highscoreliste überschrieben.

Vielleicht findet ja einer von euch den Fehler. Kann ja nur ne Kleinigkeit sein.

Richtig. Ist nur eine Kleinigkeit. Die findet man in der Java-API. Tip: Schau' Dir die verschiedenen Konstruktoren von FileWriter an.

Ich hab mal gesehen, dass man das mit der Eingabe auch so machen kann, dass ein kleines Fenster dafür aufgeht, in dem man den Name eingeben kann.

Waren nur 3 Zeilen Code oder so.

Jmd. ne Ahnung wie das geht? Würde das Ganze noch ein bisschen verschönern.

Schau' mal, wenn Du grad in der API rumblätterst, nach JOptionPane.

[Masipulami]

Die Filewriter-Konstruktoren hab ich mir angesehen.

Find den Fehler trotzdem nicht.

Wenn du weißt, was falsch ist, könntest du mir ja einfach sagen, was ich ändern muss.

[Masipulami]

Zu der JOptions-Geschichte:

Wenn ich das jetzt so machen würde:

import javax.swing.*;

import java.awt.*;

public class test

extends JFrame

{

public test()

{


System.out.println(JOptionPane.showInputDialog(

        this, "Geben Sie Ihren Namen an"));


}

public static void main(String[] args)

{

test wnd = new test();

}

}

Wie speicher ich denn jetzt den Wert in nem String? *blödfrag*

Könntest du mir mal den entsprechenden Code schreiben?

[Kaeyu]

aaaaaaaalllsoooo. ich hab mal den anhang runtergezogen, und...

beim ersten mal, war nirgendswo ein "schiff". also alle felder hatten bei mir später einen roten punkt. beim zweiten mal, hatte ich 2 "schiffe" getroffen mit zahl 2 und 3.

dann mit der highscore....dazu erstma nachdem man den namen eingegeben hat, wenn man dann in der liste klickt, geht das proggi noch weiter. die score wird immernoch berechnet und im dos fenster angezeigt. natürlich im minus bereich.

dann jetzt mal zu dem highscore, mit dem thema namen. du hast ja nur im code angegeben, dass der name gespeichert werden soll, und das is dann halt der fehler, würd ich sagen.

da du nur das neue speicherst, überschreibt das programm den alten highscore automatisch.

das könnte man ganz leicht lösen, indem man einfach den inhalt der datei ließt und in eine variable speichert, sodass du nur noch einen vergleich der punkte einproggen musst, und dann wird es ganz normal in der reihenfolge gespeichert.

ach ja, wieviele scores sollen eigentlich angezeigt werden? platz 1-3 oder 1-10? das müsstest du dann natürlich schon wissen. dann brauchste ja nur eine variable machen, die so viele elemente besitzt, wie du plätze in der scorelist brauchst.

mfg

Kaeyu

[Jaraz]

Die Filewriter-Konstruktoren hab ich mir angesehen.

Find den Fehler trotzdem nicht.

/**

     * Constructs a FileWriter object given a file name with a boolean

     * indicating whether or not to append the data written.

     *

     * @param fileName  String The system-dependent filename.

     * @param append    boolean if <code>true</code>, then data will be written

     *                  to the end of the file rather than the beginning.

     * @throws IOException  if the named file exists but is a directory rather

     *                  than a regular file, does not exist but cannot be

     *                  created, or cannot be opened for any other reason

     */

    public FileWriter(String fileName, boolean append) throws IOException {

	super(new FileOutputStream(fileName, append));

    }

Gruß Jaraz

[Masipulami]

Danke. Das ist mir wie gesagt schon klar.

Nur wie bau ich das jetzt richtig in meinen Code ein?

@keiu:

Die Highscoreliste soll zunächst einmal nur alle Einträge von highscore.txt anzeigen.

Das mit der Sortierung mach ich dann später.

[geloescht_Newlukai]

Die Filewriter-Konstruktoren hab ich mir angesehen.

Find den Fehler trotzdem nicht.

Wenn du weißt, was falsch ist, könntest du mir ja einfach sagen, was ich ändern muss.

Klar, könnte ich das, aber Jaraz hat Dich ja schon drauf gestoßen. Google hat einen Übersetzer, der sagt Dir, was 'append' bedeutet.

Wie speicher ich denn jetzt den Wert in nem String? *blödfrag*

Was gibt Dir denn

JOptionPane.showInputDialog(this, "Geben Sie Ihren Namen an")

zurück?

PS: Ja, ich könnt' Dir alles vorkauen. Ich bin aber nicht im geringsten dazu bereit. Das alles hier gehört zum Thema "Grundlagen" und wenn ich Dir die vorkaue, lernst Du nichts dabei.

[Masipulami]

Also, ich glaube kaum, dass es bei mir daran scheitert, dass ich die Grundlagen nicht verstanden hätte. Schließlich habe ich das Programm ja auch bis zu diesem Punkt alleine geschrieben und inzwischen läuft es ja auch.

Ich habe nur Probleme bei diesen 2 Sachen und habs schließlich auch die ganze Zeit schon selbst versucht, aber ich bekomms einfach nicht zum Laufen.

Da hat das doch also nix mit Vorkauen zu tun, wenn ich nach dem richtigen Code frage. Ich kann da jetzt natürlich noch Tage selbst dran sitzen und dann evtl. irgendwann mal selbst drauf kommen, aber das Programm ist wie gesagt für die Uni und muss so schnell wie möglich fertig werden.

[Masipulami]

So, das mit dem append hat jetzt geklappt.

Das mit der Fenstereingabe bekomm ich aber immer noch nicht hin.

[geloescht_Newlukai]

Also, ich glaube kaum, dass es bei mir daran scheitert, dass ich die Grundlagen nicht verstanden hätte.

Dann weißt Du wie man den Rückgabewert von JOptionPane.showInputDialog() in einem String speichert.

Und Du weißt, was der Konstruktor FileWriter(String fileName, boolean append) bedeuten könnte. Zur Not hilft auch dieser JavaDoc-Auszug des genannten Konstruktors:

@param append (boolean): if true, then data will be written to the end of the file rather than the beginning.

Da hat das doch also nix mit Vorkauen zu tun, wenn ich nach dem richtigen Code frage. Ich kann da jetzt natürlich noch Tage selbst dran sitzen und dann evtl. irgendwann mal selbst drauf kommen, aber das Programm ist wie gesagt für die Uni und muss so schnell wie möglich fertig werden.

Wahrscheinlich habe ich meine Erwartungen an die Uni immer zu hoch gesteckt. Ich dachte immer, Studenten, die irgendwas programmieren, wissen, was eine Funktion, eine Prozedur, eine Parameterliste, ein Rückgabetyp und eine Zuweisung sind. Mehr mußt Du nicht wissen, um die 2 Probleme zu beheben.

[Masipulami]

Habs jetzt so gemacht und es läuft auch:

if(schiffsfeldzaehler == 0)

			{

				try

			      {

					fenster.hide();

					BufferedReader stdin = new BufferedReader(new InputStreamReader(System.in));  


					BufferedWriter f = new BufferedWriter(new FileWriter("highscore.txt", true)); 



					System.out.print("Bitte geben Sie Ihren Name ein: ");

			        name = stdin.readLine();

			        fenster.show();

			        highsc = "Name: " + name + " | Punkte: " + scoreZähler;

			        f.write(highsc);

			        f.newLine();

			        f.flush();

			        f.close();


			        fenster.repaint();

			      }

			      catch (IOException ex) {};

			   } 

Problem nur:

Die jeweiligen Daten werden innerhalb der Datei nicht in einer neuen Zeile gespeichert, sondern einfach angehängt.

newLine() müsste doch dafür sorgen, dass immer in ne neue Zeile geschrieben wird.

btw.:

Nicht jeder kann alles gleich gut. Java ist bei uns nur Nebenfach. Mein Studienschwerpunkt liegt wo anders. Abgesehen davon hab ichs ja wie gesagt bis hier hin auch geschafft.

[geloescht_Newlukai]

Problem nur:

Die jeweiligen Daten werden innerhalb der Datei nicht in einer neuen Zeile gespeichert, sondern einfach angehängt.

newLine() müsste doch dafür sorgen, dass immer in ne neue Zeile geschrieben wird.

Stimmt.

public void newLine() throws IOException

Write a line separator. The line separator string is defined by the system property line.separator, and is not necessarily a single newline ('\n') character.

Aber zur Not schreibst Du statt f.newLine einfach f.write("\n").

btw.:

Nicht jeder kann alles gleich gut. Java ist bei uns nur Nebenfach. Mein Studienschwerpunkt liegt wo anders. Abgesehen davon hab ichs ja wie gesagt bis hier hin auch geschafft.

Richtig. Meiner Meinung nach gehört es aber immer noch in die Rubrik "Grundlagen" den Rückgabewert (der übrigens ein String ist, falls Du noch nicht in die API geschaut haben solltest) von JOptionPane.showInputDialog() einem String zuweisen zu können.