Objekt durch String instantiieren

[anp]

Hi,

kurz und knapp: wie kann man sauber anhand eines Strings auf die zu instantiierende Klasse schließen?

Bsp:

String string = "Klasse2";
if (string.equals("Klasse1") {
    new Klasse1();
} else if (string.equals("Klasse2") {
    new Klasse2();
}

So funktioniert's, ist aber keine schöne Lösung bei 20 oder mehr Klassen.

VG

 

[Ark]

Das ist garantiert Murks. Sag uns lieber, was du vorhast, sonst explodiert noch das Universum.

Ark

 

[hdi]

Das geht mit Reflections.

ich weiss leider nicht genau wie, kuck mal in die API.

Aber sowas wie getClass(String).newInstance() oder so.

 

[faetzminator]

hdi, du meinst wohl Class.forName(String).newInstance()

 

[anp]

bevor das Universum explodiert:

aus einer DB lese ich einen String aus, der die auszuführende Klasse beinhaltet, deshalb der blöde Umweg über Strings :/

diese Reflections sind eine gute Idee, wären die angebracht?

 

[Ark]

aus einer DB lese ich einen String aus, der die auszuführende Klasse beinhaltet

Was heißt denn "auszuführende Klasse"? Soll die berühmte main()-Methode dieser Klasse aufgerufen und so ein Java-Programm gestartet werden? Dann solltest du vielleicht nicht zu Reflections greifen, sondern zu eigenen Prozessen (Stichwort: java.lang.ProcessBuilder).

Ark

 

[SlaterB]

wieso denn ProcessBuilder und main? zu dieser Richtung gibts keinen Grund,

in der DB kann ganz normal AlgorthmusKlasseA, B, oder C stehen, für einfachste Dinge wie Sortieren oder sonstwas,
von keiner Basisklasse sondern von User-Einstellungen/ Auswahl abhängig,

nur aufgrund von Unschönheit würde ich an dieser Stelle nicht auf ein if/else verzichten, Codeplatz ist billig,
kann doch eine ganz separate Klasse ruhig zusätzliche 1000 Zeilen lang sein, die stört nicht,
dafür kann niemand unbekannten Code einschleusen,
Konstruktor-Aufrufe lassen sich in der IDE auffinden,

wenn es aber gerade das Ziel ist, dynamisch beliebige Klassen hinzuzufügen,
oder die bekannten Klassen aus einem anderen Projekt stammen, zur Compilierzeit nicht bekannt sind,
dann Class.forName() usw.

 

[anp]

in einer Tabelle stehen Aktionen, die durch ihren Namen unterschieden werden. Anhand des Namens soll nun in Java die entsprechende Aktions-Klasse instantiiert werden, mit der weitere Methoden aufgerufen werden.

Durch Eingabeüberprüfungen muss ich ja sowieso sicherstellen, dass nur valide Daten in die DB geschrieben werden, d.h. das Auslesen sollte keine Probleme durch den Nutzer darstellen, so dass ich an dieser Stelle vielleicht auf Class.forName() zurückgreife.

Bei der Übergabe der bestimmten Aktion muss diese natürlich validiert werden:
Wäre hier der Gebrauch von Enums sinnvoll, d.h. alle gültigen Aktionen sind in einem Enum zusammengefasst, mit denen ich dann die übergebene Aktion durch iteration überprüfen kann?!

 

[SlaterB]

ich empfehle dies, aber das bringt dir nur den Verzicht auf den ungenauen und zu validierenden String,
+ statt if/else kannst du mit Enum ein kürzeres switch bauen,

in Richtung Reflection ist das kein entscheidender Fortschritt

> Durch Eingabeüberprüfungen muss ich ja sowieso sicherstellen, dass nur valide Daten in die DB geschrieben werden

und wenn jemand auf anderen Wege die DB-Daten ändert?
ok, dann hat man eh ein Problem

 

[faetzminator]

Ich finde den Ansatz von anp nicht blöd

public MyAction getClass(String name) {
    try {
        Object instance = Class.forName(name).newInstance();
        if (instance instanceof MyAction) {
            return (MyAction) instance;
        }
    } catch (Exception e) {
        someLogger.log(e);
    }
    return null; 
}