Event ähnliches System

[H3llGhost]

Hallo Leute,

ich habe mal eine Frage:
Ich schreibe eine Applikation die auf übertragene "Eventcodes" reagieren soll.
Eventcodes wie zum Beispiel Login, Logout, etc.
Dafür habe ich mir eine Klasse ServerEvent angelegt mit einer Funktion doEvent():

import java.net.Socket;

public class ServerEvent {
	private Server s;
	public ServerEvent(Server s) {
		this.s = s;
	}
	
	void doEvent(String event, String msg, Socket s) {
		
	}
}

Also ich habe mir gedacht, ich lege eine ArrayList an mit allen möglichen EventCodes um ein möglichen ServerHack oder ähnliches zu umgehen.
Und dann soll der String zu einem Funktionsnamen werden und ausgeführt werden.
Mein Problem ich weiß nicht wie ich das umsetzen kann.

Alternativ habe ich mir schon EventListener angeschaut und ähnliches, aber ich weiß nicht, ob ich sowas für meine Idee nutzen kann.

Vielen Dank im Vorraus.

 

[SlaterB]

die Liste wird wohl kaum verhindern, dass jemand anders auch "Login" senden kann,

allgemein ist wenig von einer Frage erkennbar,
wie Protokolle ungefähr funktionieren weißt du anscheinend, jemand sendet ein Kommando,
der andere versteht es und reagiert darauf,

zu deinem Code ist pauschal zu sagen, dass ein Event selber selten eine Methode doEvent hat,
Objekte erstellt man mit Konstruktoren,
das Event kannt vielleicht an einen Handler übergeben werden: handle.doSomethingWithEvent(event);

 

[H3llGhost]

Na klar verhindern kann ich es nie, aber ich kann einen Absturz der Serverapplikation verhindern.

Hmm ...
ich versuche es mal anders zu formulieren.
Ich möchte ungerne mit Hilfe von switch oder if entscheiden welche Funktion ausgeführt werden soll aus diesem Grund die Idee, dass der übertragene Eventcode als Methodenname interpretiert wird und so auch ausgeführt wird.

Deinen Hinweis mit dem Handler verstehe ich nicht so ganz ...
Aber ich hoffe meins hat es etwas verdeutlicht.

Als Beispiel mit Variablen:

String eventcode = "Login";
runEvent("do"+eventcode);

public void doLogin(String[] args) {
  //Methodeninhalt
}

 

[SlaterB]

per Reflection könnte man Methoden anhand ihres Names aufrufen, falls das die Frage ist,
schön ist das aber nicht und auch nicht sicher, damit hat der Aufrufer quasi den Gesamtzugriff

dann lieber ein switch/ if, so kannst du genau einschränken, welches Kommando erlaubt ist

 

[H3llGhost]

Ja so könnte das heißen ...
Ich werde mich gleich mal schlau lesen ...

Also ich werde nie die Reflection dann alleine nutzen.
Ich würde vorher überprüfen ob das 1. ein zugelassener eventcode ist und 2. ob die Berechtigungen stimmen.
So dürfte nichts passieren oder?

 

[SlaterB]

müsste klappen

 

[H3llGhost]

Hast du eventuell einen guten Link, wo das Thema Reflection erklärt wird?
Ich finde irgendwie nur Sachen, die mich nicht weiterbringen.

Danke im Vorraus für deine Mühe.

 

[SlaterB]

Galileo Computing :: Java ist auch eine Insel (8. Auflage) – 24 Reflection und Annotationen
Galileo Computing :: Java ist auch eine Insel (8. Auflage) – 24.3 Attribute, Methoden und Konstruktoren

 

[tuxedo]

Oder sich den ganzen Kram sparen und gleich ein fertiges RPC System nehmen wie RMI, SIMON, Spring RPC, ... ;-)

Außer man will es "zu Fuß" machen um auch auf low-level Netzwerkkommunikation Erfahrung zu bekommen.

- Alex

 

[H3llGhost]

@tuxedo:
Es ist meine Aufgabe kein fertiges System zu nehmen.

@SlaterB:
Ich habe mir die Links mal angeschaut und mal ne kleine Testklasse geschrieben.

package Server;
import java.net.Socket;
import java.util.ArrayList;
import java.lang.reflect.*; 


public class ServerEvent {
	private Server s;
	private Method[] methods;
	
	public ServerEvent(Server s) {
		this.s = s;
		this.methods = this.getClass().getMethods();
		this.doEvent("blabla", new ArrayList<String>(), new Socket());
	}
	
	void doEvent(String event, ArrayList<String> msgs, Socket s) {
		for (int i=0; i < this.methods.length; i++) {
			System.out.println(this.methods[i].toString());
		}
	}
	
	void blabla() {
		
	}
}

Meiner Meinung nach sollte doEvent nur die Methoden doEvent und blabla auflisten.
Aber diese sind garnicht in der Auflistung.
Mir wird folgendes ausgegeben:

public final void java.lang.Object.wait() throws java.lang.InterruptedException
public final void java.lang.Object.wait(long,int) throws java.lang.InterruptedException
public final native void java.lang.Object.wait(long) throws java.lang.InterruptedException
public native int java.lang.Object.hashCode()
public final native java.lang.Class java.lang.Object.getClass()
public boolean java.lang.Object.equals(java.lang.Object)
public java.lang.String java.lang.Object.toString()
public final native void java.lang.Object.notify()
public final native void java.lang.Object.notifyAll()

Wo ist da mein Denkfehler?

 

[SlaterB]

API von getMethods() nicht gelesen bzw. nicht alles, was zu Class geschrieben wird, z.B. andere Methoden die ähnliches zurückgeben,
im Lehrbuch stehts recht undeutlich

 

[H3llGhost]

Warum müssen die Funktionen in der eigenen Klasse public sein? *grml*

 

[SlaterB]

gibt wie gesagt noch eine andere Methode, die das nicht erfordert

 

[H3llGhost]

Danke!

 

[H3llGhost]

So ich bins wieder ...
Ich kann das invoke nicht aufrufen und kann es nicht verstehen warum.
Bitte um eine Gedankenhilfe.

Im Konstruktor rufe ich folgendes in der Klasse auf:

this.methods = this.getClass().getDeclaredMethods();

Und meine Funktion doEvent sieht wie folgt aus:

	void doEvent(String event, ArrayList<String> msgs, Socket s) {
		this.msgs = msgs;
		this.socket = s;
		int j = 0;
		for (int i = 0; i < this.methods.length; i++) {
			if (this.methods[i].getName().equals("do"+event))
				j = i;
		}
		try {
			this.methods[j].invoke(this);
		} catch (IllegalArgumentException e) {
			System.out.println(e);
		} catch (IllegalAccessException e) {
			System.out.println(e);
		} catch (InvocationTargetException e) {
			System.out.println(e);
		}
		this.msgs.clear();
		this.socket = null;
	}

Er gibt mir jedes Mal eine InvocationTargetException aus und ich verstehe nicht warum.

 

[SlaterB]

ohne den Inhalt von event und die Methodendeklaration zu verraten, und in der Exception steht auch nix weiteres?

edit:
j sollte am Anfang -1 sein, sonst wird die 0te Methode genommen falls keine gefunden wird..

 

[H3llGhost]

Ich habe nun den Fehler lokalisieren können.
Er liegt in der Funktion mysql.checkString(2, "blabal").

Die Funktion sieht wie folgt aus:

	public boolean checkString(int type, String s) {
		String pattern = new String();
		switch (type) {
			case 0: //Nums
				
			break;
			case 1: //Alpha
				
			break;
			case 2: //NumsAlpha
				pattern = "[a-zA-Z0-9]+";
			break;
		}
		return s.matches(pattern);
	}

Zur Info sie befindet sich in einer anderen Klasse und in einer anderen Datei.

Also generell funktioniert die Funktion in einer anderen Klasse aber nicht in der Funktion.
Ich bin am Verzweifeln ...

 

[Gastredner]

Wäre es nicht vielleicht einfacher, die Methoden in eigene "ActionObjects" auszulagern statt sie über Reflection aufzurufen? In etwa folgendermaßen:

public interface ActionObject {

	public void doAction();
}

public class LoginAction implements ActionObject {

	public void doAction() {
		/* 
		 * Hier die Handlung - also den Login - ausführen.
		 * Falls nötig, könnte man auch noch Argumente übergeben, z. B.
		 * in Form einer Map.
		 */
	}
}

public class SomeServer() {

	public SomeServer {
		// Nötige Schritte zur Erzeugung des Servers ausführen und dann irgendwann...
		initializeActions();
		// ...aufrufen, um die vorhandenen Actions zu laden.
	}

	private Map<String, ActionObject> availableActions;
	
	private void initializeActions() {
		availableActions = new HashMap<String, ActionObject>();
		availableActions.put("login", new LoginAction());
		// Und so weiter...alternativ: Vorhandene Actions z. B. aus einer Datei auslesen.
	}

	public void processRequest(String action) {
		ActionObject a = availableActions.get(action);
		if (a != null) {
			// Hier den Fehler - unbekannte Aktion - behandeln.
		} else {
			a.doAction();
			/* 
			 * Alternativ: Map<String, Class> und mit Class.newInstance() eine neue 
			 * Instanz der benötigten Klasse für die Aktion erzeugen.
			 */
		}
	}
}

Oder gäbe es gute Gründe, die dagegen sprächen?

 

[H3llGhost]

Ich bin wie schon gesagt immer offen für alle neuen Dinge ...
Nur ich habe halt ein paar Actions mehr.
Deswegen wüsste ich jetzt nicht wie ich das erweitern könnte.
Kannst du mir eventuell nen Tipp geben?

Danke im Vorraus ...

 

[SlaterB]

ich bin ja eher noch für

if (event.equals("Login") {
doLogin();
} else if
..
} else {
Exception, unknown
}

usw., oder aus dem String eine Enum parsen und dann per switch,
wenn es nur die eine Stelle ist und sonst keine Dynamik, wäre Action vielleicht übertrieben

 

[H3llGhost]

Danke für eure Hilfe.
Es klappt nun ...

Ich hatte die Klasseninitialisierung in der falschen Reihenfolge ... *grml*