Code vereinfachen bei Reflection

[Guest]

Moin
Also, ich hab folgenden Code:

...
public ServerInterface serverinterface;
....
if(todo[i].getClassname().equals("accountmanagementhandler")){				
    boolean found=false;
    Class c=serverinterface.gui.handlercollection.accountmanagementhandler.getClass();
    Method[] m = c.getMethods();
    for (int j=0;j<m.length;j++){
        if(m[j].getName().equals(todo[i].getMethod())){
            m[j].invoke(serverinterface.gui.handlercollection.accountmanagementhandler, todo[i].getArgs());
            found=true;
            break;
        }
    }
if(!found)
    System.err.println("ServerInterfaceReceiver(): Klasse "+todo[i].getClassname()+"."+todo[i].getMethod()+" nicht gefunden!");
}else if(todo[i].getClassname().equals("gamemanagementhandler")){
    boolean found=false;
    Class c=serverinterface.gui.handlercollection.gamemanagementhandler.getClass();
    Method[] m = c.getMethods();
    for (int j=0;j<m.length;j++){
        if(m[j].getName().equals(todo[i].getMethod())){
            m[j].invoke(serverinterface.gui.handlercollection.gamemanagementhandler, todo[i].getArgs());
            found=true;
            break;
        }
    }
if(!found)
    System.err.println("ServerInterfaceReceiver(): Klasse "+todo[i].getClassname()+"."+todo[i].getMethod()+" nicht gefunden!");
}else if ...

Erstens:
Dies setzt sich nach dem gleichen Schema noch ein gutes Stück fort (insgesamt 12 solcher Aufrufe).
Wär das nicht auch dynamisch machbar? Das einzige was mir einfällt wäre alle in
serverinterface.gui.handlercollection vorhandenen Handler in ein Array zu packen und das dann bei jedem
Aufruf durchzulaufen und zu prüfen ob

serverinterface.gui.handlercollection.handlerarray[zahl].getClass().getName() .equals(todo[i].getClassname()),

aber ich weiss nicht, so elegant ist das auch nicht. Weiss da wer was besseres?

Zweitens:
Für :

    Method[] m = c.getMethods();
    for (int j=0;j<m.length;j++){
        if(m[j].getName().equals(todo[i].getMethod())){
            m[j].invoke(serverinterface.gui.handlercollection.gamemanagementhandler, todo[i].getArgs());
            found=true;
            break;
        }
    }

gibts doch bestimmt was besseres, oder? Weil in dem Fall würd die erste Methode mit passendem
Namen unabhängig vom von den Parametern benutzt werden, was ja nicht immer richtig sein muss.
Ausserdem weiss ich das es in c noch eine andere Methode gibt die genau das macht was ich will (nämlich nur
eine Methode zurückgeben und nicht ein ganzes Array davon), nur hab ichs nicht hinbekommen

Danke schonmal

 

[SlaterB]

wenn sich die 12 Blöcke nur durch
equals("x") -> Class c=y;
unterscheiden, dann musst du auch nur für dieses Mapping 12 ifs bauen,
die restliche Verarbeitung muss dann doch nur 1x im Code stehen?

-----------

die 12 einfachen ifs lassen sich auch noch verkürzen zu einer Map,
speichere darin irgendwann anfangs
"x" -> y
"z" -> w
usw. mit 12 einzeiligen put-Befehlen, dann hast du die nötigen Infos zusammen

edit: du brauchst wohl die jeweiligen Objekte,
die Klassen ergeben sich ja dann aus ihnen

-------------

in class gibts auch noch die Operation
public Method getMethod(String name, Class<?>... parameterTypes)
throws NoSuchMethodException, SecurityException

nur mit Namen scheint die aber nicht zu funktionieren, zumal es ja mehrere gleichnamige Operationen geben kann,
wenn du dir noch die Mühe machst, aus den Parametern die Parameter-Klassen zu bestimmen, dann könntest du die Operation in einem Schritt abfragen,

zum einen ist aber der Aufwand mit den Parameter-Klassen wohl größer als die Method-Schleife,
zum anderen wird intern in Class dann auch nur eine Schleife durchlaufen :roll:

 

[Guest]

Ohh jetzt qualt mein armes kleines Hirn.

wenn sich die 12 Blöcke nur durch
equals("x") -> Class c=y;
unterscheiden, dann musst du auch nur für dieses Mapping 12 ifs bauen,
die restliche Verarbeitung muss dann doch nur 1x im Code stehen?

Ah dann käme ich auf sowas:

if(todo[i].getClassname().equals("accountmanagementhandler")){             
    boolean found=false; 
    Class c=serverinterface.gui.handlercollection.accountmanagementhandler.getClass(); 
    doMethod(serverinterface.gui.handlercollection.accountmanagementhandler, todo[i].getArgs());
}

private void doMethod(Class c, Obect args[]){
    Method[] m = c.getMethods(); 
    for (int j=0;j<m.length;j++){ 
        if(m[j].getName().equals(todo[i].getMethod())){ 
            m[j].invoke(c, args); 
            found=true; 
            break; 
        } 
    } 
if(!found) 
    System.err.println("ServerInterfaceReceiver(): Klasse "+todo[i].getClassname()+"."+todo[i].getMethod()+" nicht gefunden!"); 
}

Würd ja schonmal 100 Zeilen Code sparen Und dann noch die ifs in eine Map rein, genau.
Guter Plan, Danke schön SlaterB

in class gibts auch noch die Operation
public Method getMethod(String name, Class<?>... parameterTypes)
throws NoSuchMethodException, SecurityException

nur mit Namen scheint die aber nicht zu funktionieren, zumal es ja mehrere gleichnamige Operationen geben kann,
wenn du dir noch die Mühe machst, aus den Parametern die Parameter-Klassen zu bestimmen, dann könntest du die Operation in einem Schritt abfragen,

zum einen ist aber der Aufwand mit den Parameter-Klassen wohl größer als die Method-Schleife,
zum anderen wird intern in Class dann auch nur eine Schleife durchlaufen :roll:

Naja dann... Am sinnigsten wärs dann ja konsequent keinen Methodennamen zweimal zu benutzen,
und ne eventuelle exeption ordentlich abzufangen.

Danke nochmals, der Tip (die Tips) war gut find ich

 

[SlaterB]

> Class c=serverinterface.gui.handlercollection.accountmanagementhandler.getClass();
> doMethod(serverinterface.gui.handlercollection.accountmanagementhandler, todo.getArgs());
>
> private void doMethod(Class c, Obect args[]){


was ist denn nun serverinterface.gui.handlercollection.accountmanagementhandler, eine Klasse oder ein Objekt?
wenn es eine Klasse ist, dann macht das .getClass() keinen Sinn

mache daraus

 Object o = serverinterface.gui.handlercollection.accountmanagementhandler; 
doMethod(o, todo[i].getArgs()); 
 

private void doMethod(Object o, Obect args[]){ 
  Class c = o.getClass()
  c.getMethods()...
  method.invoke(o, ..);

 

[Gast]

Stimmt, Denkfehler, in dem Fall wär mir sogar das wichtigste an dem ganzen Kostrukt flöten gegangen. danke dir