Elemente einer .txt hinzufügen und nicht überschreiben

[RapiM]

Hallo allerseits,

ich befasse mich gerade mit einer HighScore Liste für mein Spiel! Bisher funktioniert es ziemlich gut, außer dass beim Schreiben neuer Einträge die komplette Liste erst gelöscht wird und dann der neue Eintrag eingetragen wird.

Ich möchte aber gerne die Datei auslesen und dann einfach "hinzufügen" können, sodass die Liste immer länger wird.

Ist das möglich? Hier die "hinzufügen"-Funktion:

 public static void saveScore(final float score, final String player){
    	try {
    		out = new FileOutputStream("highscore.txt");
    	} catch (FileNotFoundException ex) {
    		ex.printStackTrace();
    	}
    	p.put(player, String.valueOf(score));
       
    	try {
    		p.store(out, "");
    	} catch (IOException ex) {
    		ex.printStackTrace();
    	}
}

Danke für jegliche Hilfe...:toll:

 

[frapo]

Das geht mit dem zweiten Konstruktor von FileOutputStream. Wenn der zweite Parameter true ist wird an die Datei angehangen. Näheres in der API.

FileOutputStream(File file, boolean append)

 

[Developer_X]

ich sag dir mal was, du könntest mit folgender methode:

static public String getContents(File aFile) {
		    StringBuilder contents = new StringBuilder();  
		    try {
		      BufferedReader input =  new BufferedReader(new FileReader(aFile));
		      try {
		        String line = null; 
		        while (( line = input.readLine()) != null){
		          contents.append(line);
		        }
		      }
		      finally {
		        input.close();
		      }
		    }
		    catch (IOException ex){
		      ex.printStackTrace();
		    }
		    
		    return contents.toString();
		  }

den inhalt deiner datei aus´einem File laden, ihn in eine TextArea geben,
wenn du dann was hinzufügen willst, dann mit textarea.append("");

JTextArea area = new JTextArea();
String inhalt = getContents(new File("C:/Ergebnisse.txt"));//du musst den pfad selbst angeben
area.setText(inhalt);
area.append("neuestes Ergebnis 1:0");

und dann ganz bequem mit dieser methode:

 static public void setContents(File aFile, String aContents)
		                                 throws FileNotFoundException, IOException {
		    if (aFile == null) {
		      throw new IllegalArgumentException("File should not be null.");
		    }
		    if (!aFile.exists()) {
		      throw new FileNotFoundException ("File does not exist: " + aFile);
		    }
		    if (!aFile.isFile()) {
		      throw new IllegalArgumentException("Should not be a directory: " + aFile);
		    }
		    if (!aFile.canWrite()) {
		      throw new IllegalArgumentException("File cannot be written: " + aFile);
		    }
		    Writer output = new BufferedWriter(new FileWriter(aFile));
		    try {
		      output.write( aContents );
		    }
		    finally {
		      output.close();
		    }
		  }

den inhalt der datei überschreiben, und zwar mit dem was im textfield ist,
also:

setContents(new File("C:/Ergebnisse.txt"),area.getText());

die gesamte klasse als vorschlag von mir:

public class ErgebnissApp
{
public static void main(String[]args)
{
JTextArea area = new JTextArea();
String inhalt = getContents(new File("C:/Ergebnisse.txt"));//du musst den pfad selbst angeben
area.setText(inhalt);
area.append("neuestes Ergebnis 1:0");//hier musst du in die Java klasse was neues eintragen
//geht auch indem du diese App erweiterst, mit einem JFrame und buttons usw...
setContents(new File("C:/Ergebnisse.txt"),area.getText());

}
 static public void setContents(File aFile, String aContents)
		                                 throws FileNotFoundException, IOException {
		    if (aFile == null) {
		      throw new IllegalArgumentException("File should not be null.");
		    }
		    if (!aFile.exists()) {
		      throw new FileNotFoundException ("File does not exist: " + aFile);
		    }
		    if (!aFile.isFile()) {
		      throw new IllegalArgumentException("Should not be a directory: " + aFile);
		    }
		    if (!aFile.canWrite()) {
		      throw new IllegalArgumentException("File cannot be written: " + aFile);
		    }
		    Writer output = new BufferedWriter(new FileWriter(aFile));
		    try {
		      output.write( aContents );
		    }
		    finally {
		      output.close();
		    }
		  }
static public String getContents(File aFile) {
		    StringBuilder contents = new StringBuilder();  
		    try {
		      BufferedReader input =  new BufferedReader(new FileReader(aFile));
		      try {
		        String line = null; 
		        while (( line = input.readLine()) != null){
		          contents.append(line);
		        }
		      }
		      finally {
		        input.close();
		      }
		    }
		    catch (IOException ex){
		      ex.printStackTrace();
		    }
		    
		    return contents.toString();
		  }
}

hoffe ich konnte dir helfen

 

[RapiM]

Oh vielen vielen Dank! Das hat mir wirklich sehr geholfen! ;-)

 

[Developer_X]

no problem

 

[Lexi]

@Developer_X

Da ist der Vorschlag von frapo, dem überladenen Konstruktor zusätzlich ein true zu übergeben, aber ein wenig einfacher, oder nicht ?

 

[RapiM]

Ja einfacher auf jeden Fall! Nur finde ich es immer ziemlich hilfreich, wenn man mehrere Lösungen hat und dann auch abwägen kann, welche für einen persönlich am Besten ist

 

[RapiM]

Da hier immer sehr hilfreich geantwortet wird habe ich noch eine Frage ;-)

Ich hab das nun mit FileOutputStream("highscore.txt", true); gemacht....

Wenn ich jetzt bei meinem Spiel in das Ziel komme, dann speichert er die Daten in die Textdatei, allerdings sieht man diese erst beim nächsten Start. Es fehlt also ein refresh der List, bevor sie benutzt wird!!!

Wie kann ich sowas lösen??

Danke ;-)

 

[Schandro]

Indem du die List refreshst? Du hast fast keinen Code von dir gepostet, deswegen können wir dir nicht mehr sagen.

 

[RapiM]

Hier der Code:

  public static  void initHighScore(){
    	   	
    	 p = new Properties();
    	 
    	 try {
             inStream = new FileInputStream("highscore.txt");
             p.load(inStream);
         } catch (IOException ex) {
             ex.printStackTrace();
         }
         Enumeration players =  p.propertyNames();
         playerMap = new TreeMap<Float, String>();
       while(players.hasMoreElements()){
             String player = (String) players.nextElement();
             float score = Float.parseFloat(p.getProperty(player));
             playerMap.put(score, player);
         }
       
       
       String[]array = new String[6];
       int i = 0;
       //Sortierte Liste ausgeben
			for (Float score : playerMap.keySet()){       	 
		        array[i] = "" + score + " sec  - " + playerMap.get(score);
		        i++;		       
		} 
			player1 = array[0];
			player2 = array[1];
			player3 = array[2];
			player4 = array[3];
			player5 = array[4];
			
		//	saveScore(77, "Lalala");
			
     }
     
   
    public static void saveScore(final float score, final String player){
    	try {
    		out = new FileOutputStream("highscore.txt", true);
    	} catch (FileNotFoundException ex) {
    		ex.printStackTrace();
    	}
    	p.put(player, String.valueOf(score));
       
    	try {
    		p.store(out, "");
    	} catch (IOException ex) {
    		ex.printStackTrace();
    	}
}

Die Datei heißt HighScoreStata.java!

Nun speichere ich in meinem Spiel beim Ziel folgendes :

 HighScoreState.saveScore(Timer.getTimer().getTimeInSeconds(), SwingMenuState.CurrentPlayerName);

Ich hoffe das ist hilfreich ...

 

[SlaterB]

dein Code enthält nichts, was auf eine List oder sonstige Ausgabe wie andeutet ("sieht man diese erst beim nächsten Start. Es fehlt also ein refresh der List") schließen lässt

die playerMap oder das array könntest du ersetzen, wo ist denn aktuell die Schwierigkeit?
vielleicht auch komplett initHighScore(); neu aufrufen