Exceptions benutzen

[Krabat]

Hallo

habe in diesem Artikel (link) folgendes Stück code gefunden. Meine Frage: Kann man den nicht zusammenfassen, indem man die letzte Zeile aus dem try Block raus nimmt und in den oberen try Block einfügt? Dann könnte man sich auch die IF Verzweigung sparen... oder hat das ganze einen Sinn so?

class ActorFactory {
    static public Actor createActor(String type) {
    	Actor actor = null;
        Class actorClass = null;

        try {
            actorClass = Class.forName(type);
        } catch (ClassNotFoundException e) {
            System.out.println("Error: class " + type + " not found.");
            e.printStackTrace();
        } catch (Exception e) {
            e.printStackTrace();
        }

        if (actorClass != null) {
            try {
                actor = (Actor) actorClass.newInstance();
            } catch (Exception e) {
                e.printStackTrace();
            }
        }
        return actor;
    }
}

 

[Roar]

ja, könnte man. weiß auch nich warum das newInstance() nochmal in nem trycatch drin steht, macht für mich keinen sinn. entweder es geht oder geht nich *g

 

[Bleiglanz]

die if Abfrage braucht man auf jeden Fall, weil sonst eine Nullpointer Exception fliegt

was mich eher stört ist das riesige catch(Exception), die Methode newInstance wirft ja nur InstantiationException und IllegalAccessException; und nur die sollten behandelt werden

 

[Roar]

die if Abfrage braucht man auf jeden Fall, weil sonst eine Nullpointer Exception fliegt

naja, nicht wenn man das newInstance() in den gleichen try block hinter das Class.forName() schriebt, denn wenn da ne exception fliegt wird ja gar nich mehr versucht was zu instantiieren.

 

[Krabat]

cool, thx, dann bin ich beruhigt