Problem beim Lesen und Schreiben einer Datei

[Zcelo12]

Guten Morgen,
ich habe mich soeben registriert, da ich einfach nicht weiterkomme. Ich nutze Java lediglich für Modifikationen von "Java-Spielen". Nun möchte ich aus einer Datei, eine Zeile verändern. Dafür möchte ich am Ende der Zeile, die "S:LightItems" enthält, einen Integer anhängen. Mein Problem ist, dass dann die komplette Textdatei nur mit dieser Zeile überschrieben wird. Es soll lediglich aber nur eine Zeile verändert werden. Die Zeile muss auch in der selben Position bleiben, da sie in der Text/CFG Datei in einer Funktion steht.

Ich zeige euch mal meinen bisherigen Java-Code und die Text/CFG Datei.

public void run()
        {            
                try {
            		BufferedReader in = new BufferedReader(new FileReader("C:\\Users\\abt\\Desktop\\forge804\\mcp\\jars\\config\\DynamicLights_thePlayer.cfg"));
            		String line = null;
            		while ((line = in.readLine()) != null) {
            			if (line.contains("S:LightItems"))
                        {
            			if (!line.contains("4020"))
                           {
            				BufferedWriter out = new BufferedWriter(new FileWriter("C:\\Users\\abt\\Desktop\\forge804\\mcp\\jars\\config\\DynamicLights_thePlayer.cfg"));
            				String id = ",4020=10";
            				out.write(line+id);
            				out.close();
                           }
                        }
            		}
            		in.close();
            	} 
                catch (IOException e) {
            		e.printStackTrace();
                }
        }

# Configuration file

####################
# general
####################

general {
    # Item IDs that shine light while held. Armor Items also work when worn. [ONLY ON YOURSELF]
    S:LightItems=50,89=12,348=10,91,327,76=10,331=10,314=14

    # Item IDs that do not shine light when held in water, have to be present in LightItems.
    S:TurnedOffByWaterItems=50,327
}

 

[Keyone94]

die ganze datei auslesen, dann abändern, dann wieder speichern wäre ein weg

der BufferedReader und BufferedWriter sind nicht über einen magischen Faden verbunden, der Writer weiß also nicht wo der Reader gerade hängt

 

[Zcelo12]

Momentan lese ich die Datei ja zeilenweise. Wie lese ich die denn komplett und anschließend überprüfen in welcher Zeile sich "S:LightItems" befindet?

Gruß
Zcelo12 🙂

 

[mfernau]

Momentan lese ich die Datei ja zeilenweise. Wie lese ich die denn komplett und anschließend überprüfen in welcher Zeile sich "S:LightItems" befindet?

Gruß
Zcelo12 🙂

Das tust Du doch schon:

                    while ((line = in.readLine()) != null) {
                        if (line.contains("S:LightItems"))
                        {
                        if (!line.contains("4020"))
                           {
                            BufferedWriter out = new BufferedWriter(new FileWriter("C:\\Users\\abt\\Desktop\\forge804\\mcp\\jars\\config\\DynamicLights_thePlayer.cfg"));
                            String id = ",4020=10";
                            out.write(line+id);
                            out.close();
                           }
                        }
                    }

Du liest die Datei Zeilenweise ein bis sie zu ende ist. Das tut Deine While-Schleife. Was Du nun machen musst ist, dass Du während des Einlesens jede Zeile auch wieder heraus schreibst in eine temporäre zweite Datei. Bei der Zeile mit dem "S:LightItems" tust Du das, was Du tun willst und setzt danach fort bis Deine while zu ende ist.
Anschließend musst Du die tmporäre Datei umbenennen in die ursprüngliche Datei und fertig.

Hier Pseudo-Code:

Öffne Eingangsdatei A
Öffne temporäre Ausgangsdatei B (z.b. mit File.createTempFile(..,..))
Für jede Zeile x aus A tue
   IF x enthält "S:LightItems" THEN
      Hänge Integer an x an
   schreibe x nach B
END WHILE
lösche A
benenne B um nach A
ENDE

 

[GeoRage]

Es gibt teilweise Probleme beim Löschen von Dateien.
Also kannst du alternativ die Datei auch auslesen und jede Zeile in einer List speichern.

Dann hast du jede Zeile als Element in deiner List und kannst damit rumspielen.
Sobald deine Änderungen vorgenommen hast, speicherst/schreibst die List wieder in die Datei.

 

[mfernau]

Es gibt teilweise Probleme beim Löschen von Dateien

Echt? Wann denn? Ist mir noch nie unter gekommen.
Die gelesene Datei muss natürlich geschlossen werden. Sonst kann man sie nicht löschen. Aber sonst ist mir das noch nie passiert.
Hast Du nähere Infos für mich? Würde mich interessieren.

Schöne Grüße
Martin

 

[GeoRage]

Das ich das letzte mal damit gearbeitet hab is etwas länger her, aber da hat ich das problem auch, das ich die gewünschte Datei nicht löschen konnte. Ich hab dann auch auf nen zwischenspeicher in form von Liste oder Array zurückgegriffen

Habs hier aber immer wieder mal gelesen, das davon abgeraten wurde. (zB bei fragwürdigen Schreib-Rechten des Benutzers (falls auf Windowsoberfläche) etc)

 

[mfernau]

Hm - ehrlich gesagt kann ich das nicht nachvollziehen. Wenn Du mit einem Zwischenspeicher arbeitest (mal abgesehen davon, dass wir hier nicht wissen wie groß die Datei ist - bei vielen MBs bis GBs fände ich das wenig nützlich) und die ursprüngliche Datei überschreibst, musst Du ebenfalls Schreibrechte auf diese Datei besitzen.
Grundsätzlich ist es immer eine Frage der Zugriffsrechte weswegen bei Dateioperationen auch immer mit File.canWrite() und/oder File.canRead() abgefragt werden sollte, ob die nachfolgenden Operationen auch von Erfolg gekrönt sein werden.
In meinen bisherigen Projekten habe ich das immer getan und ich hatte bisher noch nie das Problem, dass ich eine Datei nicht löschen konnte. Und wenn doch, lag es daran, dass ich sie vergessen hatte zu schließen (was seit Java7 beim verlassen des umschließenden Blocks automatisch geschieht).

Jeder muss letztlich selbst entscheiden wie er ein Problem löst. Ich persönlich würde davon abraten eine Datei in den Speicher zu lesen und anschließend wieder raus zu schreiben:
1. Aufgrund des Speicherbedarfs (bei ein paar K natürlich zu vernachlässigen)
2. Aufgrund der Fehleranfälligkeit. Denn was machst Du, wenn Dein Programm während des Schreibens auf eine RuntimeException trifft und abbricht? Dann sind die Daten futsch, weil nur halb geschrieben.
IMHO ist die Lösung über eine temporäre Datei eine gängige Lösung. Nach dem Schreiben in eine Temp-Datei weißt Du, dass die Daten komplett sind. Sollte das Löschen+Umbenennen an irgend einer Stelle fehl schlagen, sind die Daten trotzdem alle vorhanden.

Aber wie gesagt - es muss jeder für sich selbst entscheiden wie er sein spezifisches Problem löst.

Schöne Grüße
Martin

 

[Matt]

Vor dem löschen mūssen die streams geschlossen sein.

In deinem fall der fileReader und bufferedReader.

 

[Matt]

Hier vllt. hilft dir das.

DummyProject bzw. das hier: "EditFile"

 

[Zcelo12]

Danke. Habe es hinbekommen! 🙂