[Java] Problem mit Exceptions...

@mersa

Eclipse ist schon ein tolles Programm und sofern die Fehler rot markiert werden, sollte man sie korrigieren, sonst wird eine Exception geworfen.
Gelb markierte Fehler müssen nicht unbedingt korrigiert werden. Es kommt auf die Situation an, wie bestimmte Objekte genutzt werden.

Du kannst eine Exception im Konstruktor werfen:

Code:

public CodeCleaner() throws IOException
{
...
}

Allerdings muss die Exception auch einen Grund haben, dort zu sein. Bei einem leeren Konstruktor würde die Exception wenig Sinn machen.
Sollte Eclipse jammern, dann könnte es daran liegen, dass der an einigen anderen Stellen z.B. Methoden ebenfalls eine Exception haben möchte.

Du willst doch eine Datei auslesen, die doch bereits vorhanden ist, oder?

Dann brauchste das hier nicht unbedingt (kannst es aber drin lassen):

Code:

static File codefile = new File("code.txt");

Hier musste dann anstatt "codefile" den Dateinamen eingeben, sofern sich die Datei im Wurzelordner befindet ansonsten den Pfad dorthin, falls du codefile nicht verwenden möchtest:

Code:

static FileReader reader = new FileReader(codefile);

Entferne außerdem vor File, FileReader und BufferedReader "static".

 

Falls es dich interessiert ist hier noch eine etwas ausführlichere Erklärung:

Das was du da implizit verwendet hast ist ein so genannter Static Initializer. Initializer sind z.B. die Statements mit denen einer Variable ein initialer Wert zugewiesen wird. Und da es sich hier um static Variablen handelt, landet das dann eben im Static Initializer.

PHP:

      static File codefile = new File("code.txt");
      static FileReader reader = new FileReader(codefile);
      static BufferedReader buf = new BufferedReader(reader);

Diese 3 Zuweisungen werden vom Compiler intern im Prinzip in folgenden Code umgewandelt und dann compiliert (die beiden Codeblöcke sind also vollkommen identisch und erzeugen auch den selben Bytecode):

PHP:

      static File codefile;
      static FileReader reader;
      static BufferedReader buf;

      static {
            codefile = new File("code.txt");
            reader = new FileReader(codefile);
            buf = new BufferedReader(reader);
      }

Diese eigenartige static Methode ohne Namen und Parameter ist eben jener static Initializer, der hier jetzt explizit verwendet wird. Das Konstrukt compiliert tatsächlich zu einer static Methode, die intern den Namen <clinit> hat und als allererstes nach bzw. beim Laden einer Klasse aufgerufen wird.

In dieser Methode kannst du auch per try/catch Exceptions behandeln:

PHP:

      static File codefile;
      static FileReader reader;
      static BufferedReader buf;

      static {
            codefile = new File("code.txt");
            try {
                  reader = new FileReader(codefile);
                  buf = new BufferedReader(reader);
            } catch( IOException ioex ) {
                  ioex.printStackTrace();
            }
      }

Allerdings wäre das in deinem Fall nicht der sinnvollste Weg, da die Variablen reader und buf dann ggf. null sind und du nicht unmittelbar an der Stelle auf die Exceptions reagieren könntest, an der der Reader verwendet wird, da diese Methode ja vom Classloader aufgerufen wird, also noch bevor deine main Methode aufgerufen wird. Im Prinzip könntest du die Exception auch weiterleiten, wenn du sie in einen Error oder eine RuntimeException kapselst, z.B. so:

PHP:

            } catch( IOException ioex ) {
                  throw new ExceptionInInitializerError( ioex );
            }
      }

Dann würde die Klasse aber gar nicht erst vollständig geladen werden falls eine Exception auftritt. Sowas kann in bestimmten Fällen sinnvoll sein, aber für dein Programm wäre es besser es ungefähr so zu machen:

PHP:

import java.io.BufferedReader;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileNotFoundException;
import java.io.FileReader;
import java.io.IOException;
    
public class CodeCleaner
{
    protected String filename;
    protected BufferedReader buf;
    
    public CodeCleaner( String name  )
    {
        filename = name;
        // [....]
    }

    public boolean loadFile()
    {
        try {
            buf = new BufferedReader( new FileReader( filename ) );
        } catch( IOException ioex ) {
            ioex.printStackTrace();
            return false;
        }
        return true;
    }
        
    public void closeFile()
    {
        if( buf == null ) // File ist nicht geöffnet
            return;
        try {
            buf.close();
        } catch( IOException ioex ) {
            // ioex.printStackTrace();
            // Die Ausgabe kann man bei close() weglassen. Der Fall tritt sowieso nie auf.
        }
        buf = null;
    }

    public void parseFile() throws IOException
    {
        // [...]
    }

    public void ausgabe() throws IOException
    {
        // [...]
    }

    public static void main( String[] args ) throws Throwable
    {
        String name = "code.txt";
        if( args.length >= 1 ) // Name kann auch per Kommandozeile übergeben werden
            name = args[ 0 ];

        CodeCleaner cc = new CodeCleaner( name );
        if( cc.loadFile() )
        {
            System.out.println( "File loaded" );
            cc.ausgabe();
            cc.parseFile();
            cc.closeFile();
            cc.ausgabe();
        }
        else
        {
            System.out.println( "File not loaded." );
        }
    }
}


 

@mersa

Du kannst doch die main()-Methode von der Klasse CodeCleaner löschen und diese in eine andere Klasse z.B. namens Application.java, die nur die main()-Methode enthält, einfügen.

In der Klasse kannste dann einen neuen CodeCleaner erzeugen und dort jeweils auf die Methoden von CodeCleaner zugreifen und in der Klasse CodeCleaner.java müssen dann File, FileReader, BufferedReader nicht mehr static sein.

 

@mersa

Habe mir deinen Code genauer angeschaut. Du hast sogar die Methoden "static" gemacht. Das musste aber nicht. Lass das "static" weg und dein Code sollte dann auch funktionieren.

Außerdem habe ich dein Code bissel umgeschrieben und eine zweite Klasse erzeugt, um dann dort die main()-Methode zu benutzen.

Die parse_file()-Methode habe ich nicht bearbeitet, schließlich weiß ich nicht, was du mit ihr machen möchtest.

Code:

package testpackage;

import java.io.*;
    
public class CodeCleaner
{
    File codefile; //= new File("code.txt");
    FileReader reader; //= new FileReader(codefile); 
    BufferedReader buf; //= new BufferedReader(reader);
    
    public CodeCleaner(String s) throws FileNotFoundException
    {
        this.codefile = new File(s);
        this.reader = new FileReader(codefile);
        this.buf = new BufferedReader(reader);
    }

    public boolean load_file() throws FileNotFoundException
    {
        return codefile.exists();
    }

    public void parse_file() throws IOException, FileNotFoundException
    {
        //int c = 0;
    }

    public void ausgabe() throws IOException
    {
        String zeile;

        while((zeile = buf.readLine()) != null)
        {
            System.out.println(zeile);
        }
    }
}

Code:

package testpackage;

import java.io.*;

public class Application
{
    public static void main(String... args) throws FileNotFoundException, IOException, Exception
    {
        CodeCleaner c = new CodeCleaner("readme.txt");
        if (c.load_file() == true)
        {
                System.out.println("File loaded:");
                c.ausgabe();
                //c.parse_file();
                //c.ausgabe();
        }
        else
        {
            System.out.println("File not found.");
        }
    }
}

Einfach in Eclipse diese Klassen und das Package (ich arbeite lieber mit Packages, ist aber bei mehreren JAVA-Dateien ebenfalls sinnvoll, um die Übersicht nicht zu verlieren ) erzeugen und den obigen Code einfügen.
Bei Eclipse muss außerdem die Datei, die du lesen möchtest im gleichen Ordner liegen wie der src-Ordner.

Wenn du die Konsole zum Kompilieren und Ausgeben nutzt, dann muss sich die Datei außerhalb des Package liegen. Im obigen Code also im gleichen Ordner wo sich der testpackage-Ordner befindet.

Fragen? Fragen!