String suchen und ersetzten, ohne neue Datei

[Laren]

Hi,

Ich habe den Code unten geschrieben, der Dateien(args) öffnen und einen String liest(args[0]) und diesen ersetzt (args[1]).
Aber wenn der Benutzter jetzt einen datei eingibt, die es nicht gibt, wird zwar eine Exception ausglöst, aber die Datei wird einfach erstellt, aber warum?

Grüße

package Ueb10;
import java.io.File;
import java.io.FileReader;
import java.io.FileWriter;
import java.io.IOException;
import java.io.Reader;
import java.io.Writer;
 
 
 
 

public class Replace {
 
    /**
     * @param args
     * args[0] übergibt den suchstring
     * args[1] übergibt den ersetzungsstring
     * args[2] übergibt die Datei 1
     * args[3] übergibt die Datei 2
     * .....
     * 
     */
    //**********Attribute************
    public static String ausgabe;
    public static String suchen="";
    public static String ersetzen="";
    public static void main(String[] args) 
    {
        try
        {
       if (args.length<3)
            {
                throw new ReneFalscheEingabeException("Alle Argumente müssen ausgefüllt sein! Es sind nur " +args.length +" ausgefüllt");
            }
      for (int i=2;i<=args.length-1;i++)
      {
      suchen = args[0];
      ersetzen = args[1];
      run(args[i]); //es wurde der Dialog durch args[i] ersetzt
      }
        }
         catch(ReneFalscheEingabeException e)
         {
             System.err.println( "Fehler bei der Benutzereingabe abgefangen" );
             e.printStackTrace();
         }
    }
    public static void run(String datei) 
    {
        getDatei(datei);
        setDatei(datei);
    }
    
    public static void getDatei(String datei)  
    {
        Reader in = null;
        ausgabe = "";
        try
        {
            File inputFile = new File(datei);
            if (inputFile.exists()==false)
            {
                throw new ReneDateiNichtGefundenException("Datei existiert nicht! " +datei);
            }
            if (inputFile.isDirectory()==true)
            {
                throw new ReneDateiNichtGefundenException("Datei ist ein Verzeichnis! "+datei);
            }
            if (inputFile.canRead()==false)
            {
                throw new ReneDateiNichtGefundenException("Datei kann nicht gelesen werden! "+datei);
            }
            if (inputFile.canWrite()==false)
            {
                throw new ReneDateiNichtGefundenException("Datei kann nicht geschrieben werden! "+datei);
            }
            in = new FileReader (inputFile);
            
            for(int c; (c=in.read()) !=-1;)
            {
                ausgabe = ausgabe +(char) c;
            }
               
        }
       
        catch(ReneDateiNichtGefundenException e)
        {
            System.err.println( "Falsche Datei eingabe abgefangen" );
            e.printStackTrace();
            try {
				in.close();
			} catch (IOException e1) {
				// Sollte sogar in.close nicht mehr funktionieren, wird hier abgefangen
				e1.printStackTrace();
			}

        }
        catch(ReneDateiLeserechteException e)
        {
            System.err.println( "Fehler bei den Leserechten abgefangen" );
            e.printStackTrace();
            try {
				in.close();
			} catch (IOException e1) {
				// Sollte sogar in.close nicht mehr funktionieren, wird hier abgefangen
				e1.printStackTrace();
			}
        }
        
        catch(IOException e)
        {
        
        }
        finally     
        {
            try 
            {
                in.close();
            }
            catch (Exception e) 
            {
        
            }
        }
 
    }
    
 
    
    public static void setDatei(String datei)
    {
        Writer fw = null;
        try
        {
            fw = new FileWriter(datei);
            fw.write(ausgabe.replace(suchen, ersetzen));
            }
        catch(IOException e)
        {
        System.err.println("Datei nicht geschrieben");
    }
    finally
    {
        if (fw != null)
        {
            try
            {
                fw.close();
            }
            catch (IOException e )
            {
                e.printStackTrace();
            }
            }
        }
    }
    
 
}

 

[nrg]

das kommt wegen deinem lokalen Exceptionhandling. Schmeiss mal alle, dann wirste sehn, dass nach dem Auslösen der Exception garnichts mehr passiert 😉

btw: etwas schönere Formatierung wäre net schlecht. Eclipse strg+a und dann strg+i

edit: und sowas wie

void get...

ist schonmal konzeptionell totaler quatsch. Ich würde dir einfach empfehlen mit get... den Dateiinhalt auch wirklich zu holen. Im Falle einer Exception einfach return null schreiben und dann vor setDatei auf null prüfen.

 

[FArt]

Ich würde dir einfach empfehlen mit get... den Dateiinhalt auch wirklich zu holen. Im Falle einer Exception einfach return null schreiben und dann vor setDatei auf null prüfen.

Im Fehlerfall null zurückgeben ist ein Antipattern. Wenn es einen Fehler gibt, dann sollte der Aufrufer damit umgehen, und zwar in Form einer Exception.
Unter umständen sollte man das gesamte Vorgehen hier passend überarbeiten...

 

[bygones]

bzgl formatieren... Strg+Shift+F formatiert allgemein....

 

[nrg]

Im Fehlerfall null zurückgeben ist ein Antipattern. Wenn es einen Fehler gibt, dann sollte der Aufrufer damit umgehen, und zwar in Form einer Exception.

dennoch besser als der aktuelle Zustand

Unter umständen sollte man das gesamte Vorgehen hier passend überarbeiten...

geb ich dir vollkommend Recht aber dann wird der Thread 3 Seiten lang. Es sei denn, jemand schreibt es ihm komplett neu - was bzgl. Lerneffekt allerdings genauso "Anti" wär.