gustav Hilfe !!!

[chriscool]

Hallo gustav,

ich habe vor ein paar Tagen das Thema für das lesen von Dateien und gewollte Strings ersetzten hier in das Forum gestellt und habe folgendes von Dir bekommen, was ich nun bei mir benutzen wollte. Aber es wird nicht ersetzt in der Datei, meine Datei ist leer!!!! Kannst Du mir dabei helfen.

Das war von dir!

public boolean Load(String FileName)
   {
     String buf = null ;
     int Zeile = 1 ;
     boolean back = false ;

// ##########
     String writeBack = null ;

     try
     {
       BufferedReader input = new BufferedReader(
                                new InputStreamReader(
                      new FileInputStream(FileName)), 1000) ;

// ########################
      FileOutputStream Fileoutput = new FileOutputStream(FileName) ;
      DataOutput outFile = new DataOutputStream(Fileoutput ) ;
// ########################

       while (input.ready() )
       {
         buf = input.readLine() ;
// ##########################
         writeBack = buf.replaceAll("§xxxx", "XXXXXXX") ;
         // oder ersetzen mit
         // index = buf.indexOf("§") usw. siehe Java Dokumentation für String

         Zeile++ ;
         outFile.writeBytes(writeBack +"\n") ;  // weiss jetzt nicht, ob "\n" nötig ist
       }
       input.close() ;
       back = true ;
     }
    catch (Exception e)
    {
      System.out.println(e);
      System.out.println("----------------------Exception laden/schreiben!");
      System.exit(1);
    }
    return back ;
   }
}

jptc.org: code tags eingefügt

 

[gustav]

Sorry,

dies liegt mit Sicherheit daran, das Du eine Datei gleichzeitig öffnest und beschreibst. Ich sehe hier auf die Schnelle 2 Möglichkeiten.

• 
  Entweder Du liest erst die komplette Datei ein und bearbeitest alles danach, oder
  Du benennst die Datei erst um, dann kannst du mit 2 Dateien arbeiten

Hab mal den Code für die 2. Variante zusammengestellt. Gefällt mir noch nicht so ganz - ABER es funktioniert.

import java.io.* ;

public class TestLoad
{

  public boolean Load(String FileName)
  {
    String buf = null ;
    int Zeile = 1 ;
    boolean back = false ;

    String writeBack = null ;

    try
    {
      String tmpFileName = FileName + ".tmp" ;

      File inFile = new File(FileName) ;
      File tmpFile = new File(tmpFileName) ;

        // Input File umbenennen
      if (inFile.renameTo( tmpFile))
      {
          // erfolgreich => ersetzen
        BufferedReader input = new BufferedReader(
            new InputStreamReader(
            new FileInputStream(tmpFileName)), 1000) ;

        FileOutputStream Fileoutput = new FileOutputStream(FileName) ;

        DataOutput outFile = new DataOutputStream(Fileoutput) ;

        while (input.ready())
        {
          buf = input.readLine() ;

          writeBack = buf.replaceAll("replaceMe", "XXXXXXX") ;

          Zeile++ ;
          outFile.writeBytes(writeBack + "\n") ; // weiss jetzt nicht, ob "\n" nötig ist
        }
        input.close() ;
        back = true ;

          // alte Input Datei löschen
        if (! tmpFile.delete())
        {
          System.out.println("Fehler beim Löschen der temporären Datei") ;
        }
      }
      else
      {
        System.out.println("Fehler beim Öffnen/Anlgen der temporären Datei") ;
      }
    }
    catch (Exception e)
    {
      System.out.println(e) ;
      System.out.println("----------------------Exception laden/schreiben!") ;
      System.exit(1) ;
    }
    return back ;
  }

  public static void main(String args[])
  {
    TestLoad test = new TestLoad() ;
    test.Load("in.dat") ;
  }
}

Hier wird am Ende die benutzte temporäre Datei wieder gelöscht. Als Sicherheitsoption brauchst Du dies vielleicht gar nicht machen.
Komischerweise meckert Java beim Umbenennen nicht wenn die Datei bereits vorhanden ist. Vielmehr wird sie ohne Nachfrage überbügelt. Zumindest ist es bei mir so, denn Du kannst das Löschen mal ausklammern und das Testprogramm einige Male starten. Es sollte immer ohne Fehler durchlaufen.

 

[chriscool]

hey vielen dank! compilieren kann ich es, wenn ich es nun ausführen will, dann bekomme ich folgenden fehler:

java.lang.NoSuchMethodError ?????

 

[Guest]

liegt das eventuell wieder an replaceAll() ?

Schau mal hier : http://www.java-forum.net/viewtopic.php?t=559
das sollte als Ersatz taugen

 

[gustav]

Sonst poste mal die komplette Fehlermeldung. Eigentlich sollte ab Java 1.4 auch replaceAll vorhanden sein.
Klingt aber verdammt nach einer falschen main - Methode. Überprüf das mal !

 

[chriscool]

ich stell dir nochmal meinen ganzen text zur verfügung. jetzt bekomme ich immer die excaption
"Fehler beim öffnen/Anlgen der temporären Datei"

import java.io.* ;

public class replaceStr {

private String File;

public void replaceStr(String Filename) {

File = Filename;

//replaceStr test = new replaceStr() ;
Load(File);
}

public void Load(String File)
{
String buf = null ;
int Zeile = 1 ;
boolean back = false ;
String FileName = File;

String writeBack = null ;

try
{
String tmpFileName = FileName + ".tmp" ;

File inFile = new File(FileName + ".txt") ; // <= der path von der file ist .....\email (ohne endung)
//Daher habe ich noch .txt anghängt. sonst erstellt er
//mir eine "emai.txt.tmp"

File tmpFile = new File(tmpFileName) ;

// Input File umbenennen
if (inFile.renameTo(tmpFile))
{
// erfolgreich => ersetzen
BufferedReader input = new BufferedReader(
new InputStreamReader(
new FileInputStream(tmpFileName)), 1000) ;

FileOutputStream Fileoutput = new FileOutputStream(FileName + ".txt") ;

DataOutput outFile = new DataOutputStream(Fileoutput) ;

while (input.ready())
{
buf = input.readLine() ;

writeBack = buf.replaceAll("%xx", "XXXXXXX") ;

Zeile++ ;
outFile.writeBytes(writeBack + "\n") ; // weiss jetzt nicht, ob "\n" nötig ist
}
input.close() ;
back = true ;

// alte Input Datei löschen
if (! tmpFile.delete())
{
System.out.println("Fehler beim Löschen der temporären Datei") ;
}
}
else
{
System.out.println("Fehler beim Öffnen/Anlgen der temporären Datei") ;
}
}
catch (Exception e)
{
System.out.println(e) ;
System.out.println("----------------------Exception laden/schreiben!") ;
System.exit(1) ;
}
//return back ;
}
}

 

[chriscool]

Exception in thread "main" java.lang.NoSuchMethodError
at replaceStr.Load(replaceStr.java:47)
at replaceStr.replaceStr(replaceStr.java:12)
at str.main(str.java:15)

das ist die fehlermeldung, wenn keine .tmp file da ist.
es wird die tmp file eerstellt, der in der ausgangsfile stehende text in sie hinein kopiert und dann kommt die fehlermeldung!!!!

 

[chriscool]

noch zu sagen, ich greife mit der klasse "str" auf die klasse replaceStr zu

 

[gustav]

Ich glaube das liegt daran, das Du ein wenig mit den Dateinamen durcheinander kommst. Warum darf die temporäre Datei nicht email.txt.tmp heissen ? Wenn Du in der Load Methode zusätzliche Endungen anfügst, dann musst Du aufpassen das die Datei auch exisitert. Konnte Deine Fehlermeldung leider nicht reproduzieren.

Was mir noch aufgefallen ist : Es gibt bereits eine Klasse File, vielleicht gibt es Nebeneffekte wenn Du zusätzlich noch eine solche Variable definierst. Um solche Probleme zu vermeiden, bietet sich folgende Notation an :
1. alle Variablen beginnen mit einem kleinen Buchstaben
2. Konstanten bestehen grundsätzlich aus Grossbuchstaben
3. Klassen beginnen mit einem Grossbuchstaben oder noch eine wenig weiter getrieben, beginnen immer mit einem speziellenGrosbuchstaben.....

 

[chriscool]

ok, ich habe den pfad zur dati nun wieder voll angegeben .......\email.txt.

nun bekomme ich, wenn beim ersten mal alles ausgeführt wird, die exception in thread main...., was ich oben gepostet habe. zum ersten mal besteht die datei ja gewöhnlicherweise nicht!!!!
die datei email.txt.tmp wird erstellt mit dem inhalt drin und dann bumm die exception.

besteht die file .tmp dann passiert genau das gleiche wie oben nur anstatt der fehlermeldung bekomme ich die exception "Fehler beim öffnen/Anlegen der temporören Datei"


Die Filevariablen habe ich auch nun umbenannt!!!

warum kommt er nicht mal zum ersetzten der strings????

 

[gustav]

Arbeitest Du zufällig mit Windows ?

Kannst ja mal versuchen existierende Dateien vorher zu löschen :

    if (tmpFile.exists())  // eventuell existierende Datei löschen
    {
      tmpFile.delete() ;
    }

Wer genau verursacht die Exception, replaceAll oder ein späterer Schreibzugriff auf die Datei.

Eventuell mußt Du noch prüfen, ob Du einen leeren Strings hast

  if (buf == null) { System.out.println("!!!! empty String detected") ; }

 

[chriscool]

nun funktioniert alles, habs hinbekommen!!!!!

vielen dank, super !!! :lol:

 

[chriscool]

Aber mal eine Frage, was macht die "back" variable???

ich versuche das ganze gerade so umzuprogrammieren, das ich so zusagen meine email.txt nie anrühr, nur um den standarttext herauszuholen!
will die datei auslesen, in einen string packen, entsrechende felder ersetzen und dann in einen nächsten string übergeben und auf dem bildschirm ausgeben.

Hast du eine idee, was ich an deinem code alles weglassen kann?????

 

[chriscool]

habs schon

 

[chriscool]

habs nicht
ich bekomme immer nur die letzte zeile ausgegeben!
hast du einen vorschlag??

 

[chriscool]

wie bekomme ich eigentlich einen string, also die taxtfile, in einen string und dann alles dort ersetzten. geht das mit dem stringbuffer???? übernimmt der auch absätze??

 

[chriscool]

jetzt hab ich aber !!!!!!!!!!!!!!!!

 

[gustav]

Die back Variable war ursprünglich dafür gedacht über Erfolg oder Misserfolg zu informieren, also true wenn alles glatt gegangen ist ansonsten false. Somit hast Du im Programm die Möglichkeit eventuelle Korrekturen vorzunehmen - z.B. dummy Datei für Load() erstellen. Wenn Du back auskommentierst ist es auch nicht weiter schlimm.

Generell könntest Du Dir überlegen mit StringBuffer zu arbeiten wenn Du die Daten weiter verarbeiten oder irgendwie zwischenspeichern willst.

 

[chriscool]

Gustv, ich stehe hier vor einem Problem.
Ich lese meine Datei nun Zeilenweise ein und ersetze Zeilenweise einen String. Wie kann ich denn mehrere Strings in einer Zeile ersetzen ohne den zuvor erstellten String mit der ersten ersetzten Variablen zu übetrschreiben???

Hast Du vielleicht eine Idee?

 

[chriscool]

habs schon