Best Practice Objekt an alle Klassen verteilen ( Discord Bot )

[JvKbF]

Hallo zusammen,

ich stecke leider gerade etwas in der Klemme, bzw. weiß nicht weiter.
Ich habe jetzt seit knapp 2 Jahren kein Java mehr gemacht und versuche wieder rein zu kommen, indem ich einen Bot für Discord entwickle.

Wer schon mal einen Bot für Discord entwickelt hat, weiß dass dort alles über Listener und Events funktioniert. ( z. B. onGuildMessageReceived - wird aufgerufen wenn im Server eine Nachricht gesandt wurde ) Und da es sehr viele Events gibt, habe ich diese anhand von "Kategorien" in Klassen geordnet ( z. B. alle "onGuildMessageXXX" Methoden in eine Klasse, alle "onGuildMemberXXX" in eine Klasse usw. ).

Nun ist es so, dass ich auch verschiedene einstellungen habe, welche sich von Server zu Server unterscheiden ( können ). Dafür habe ich eine Settings-Klasse. Diese habe ich als Singleton konzipiert, sodass nicht jedes Mal, wenn die eine Klasse einstellungen braucht, diese neu aus config Files oder so gelesen werden müssen. ( zur Info am Rande, diese Klasse enthält eine HashMap in der die Objekte als Values und die ServerID als Key gespeichert sind - Wichtig ist hier also die ServerID, dass mir diese immer zur verfügung steht).

Mein Problem jetzt, ist es, dass ich im Konstruktor der jeweiligen Klassen die ServerID nicht zur verfügung habe, diese bekomme ich immer erst durch die Methoden ( eventhandler - also wie oben gesagt z. b. "onGuildMessageReceived( GuildMessageReceivedEvent event ) "). Nun möchte ich nicht in jeder Methode prüfen ob die Settings schon gelesen wurden und zur not dann das Objekt initialisieren. Ich würde sowas gerne nur einmal pro klasse machen - habe aber keine Ahnung wie.

Ist das irgendwie möglich? ( und ist mein Anliegen überhaupt verständlich? )

Edit:
Bzw. Kann ich dann das Settings Objekt irgendwie, wenn es ein Mal durch irgend eine der oben beschriebenen ListenerKlassen initialisiert wurde, an die anderen Listenerklassen verteilen? Ist das möglich?

Ich möchte halt vermeiden, dass in jeder Methode am Anfang steht

this.settings = settings.get_instance(event.getGuild().getID());

Eine Methode ist immer so aufgebaut:

    public void onGuildMessageUpdate(GuildMessageUpdateEvent event) {
    }

Um es Grundlegend mal aufzuzeigen, wie der Bot aufgebaut ist:

public class APP {

public static void main(String[] args){

JDABuilder builder = new JDABuilder(AccountType.BOT);
// some stuff
builder.addEventListener(new GuildMessageListener() );
// add other listeners
}
}

public class GuildMessageListener extends ListenerAdapter {
private Settings settings;

public void onGuildMessageReveived(GuildMessageReceivedEvent event) {
// do stuff
}

// other event methods

}

ich hoffe durch das Coding ist es etwas klarer geworden, was das problem sein könnte...

sonst müsste ich tatsächlich, wie oben beschrieben jede Methode mit dem Singleton ausstatten ... das ist jetzt nicht wirklich schön ..

 

[mihe7]

und ist mein Anliegen überhaupt verständlich?

Hm... nicht wirklich bzw. verstehe ich das Problem noch nicht.

Wenn Du ein Server-Objekt erzeugst, kennst Du dessen ID? Wenn nicht, warum nicht?

 

[JvKbF]

Hm... nicht wirklich bzw. verstehe ich das Problem noch nicht.

Wenn Du ein Server-Objekt erzeugst, kennst Du dessen ID? Wenn nicht, warum nicht?

Das liegt an dem Framework. Ich erstelle kein Server-Objekt, es ist so, dass der BOT ein Mitglied auf dem Server ist, heißt der Server existiert schon, egal ob mein Programm läuft oder nicht. Das heißt, mein Bot loggt sich ein und reagiert dann auf verschiedene Ereignisse ( z. B. wenn jemand in den Chat eine Nachricht schreibt ) - dann fängt der Bot die Nachricht ab bzw. er liest die Nachricht und kann damit dann weiter arbeiten. Diese Nachricht erhält er als Objekt, ( z. B. GuildMessageRecievedEvent ). Mit diesem Objekt, kann ich dann erst herausfinden, auf welchem Server ich bin.

Man muss es sich so vorstellen, dass ein User mitglied bei verschiedenen Servern sein kann ( vielleicht haben sie im englischen hier absichtlich das Wort "Guild" verwendet .. hmm .. anstatt server, weil es sonst zu verwirrungen kommt ) und immer wechseln kann. das gleiche ist auch bei Bots so, der ist mitglied bei verschiedenen servern und nimmt immer nur die verschiedenen ereignisse und verarbeitet diese.

Ich habe beim initialisieren leider keine Möglichkeit die Server zu identifizieren.

Edit:
Ist halt im prinzip, wenn man für eine GUI eine Klasse schreibt, die rein auf die EVENTS der Gui reagiert.

Ich werde jetzt dann erst mal in jede von mir implementierte Listener-Methode die "get_instance" methode aufrufen ... falls dennoch jemand eine Lösung weiß, bin ich sehr dankbar!

 

[mihe7]

OK, dann gib den Listenern halt einfach einen Settings-Proxy mit.

 

[Robat]

Gibt es nicht ein Event um abzufangen wann der Bot connected um dort die Server ID abzufangen? Dann könntest du dort deine settings ja initialisieren

 

[mihe7]

Dann könntest du dort deine settings ja initialisieren

Zusammen mit den Listenern...

Nun möchte ich nicht in jeder Methode prüfen ob die Settings schon gelesen wurden und zur not dann das Objekt initialisieren.

Manchmal denkt man (ich) einfach zu kompliziert... Prüfe halt einfach in Settings ab, ob sie schon gelesen wurden.

 

[JvKbF]

Gibt es nicht ein Event um abzufangen wann der Bot connected um dort die Server ID abzufangen? Dann könntest du dort deine settings ja initialisieren

Du hast recht! Dieses Event habe ich absolut übersehen ... danke!!!

 

[JvKbF]

God damn ... nein,auch das Event hat mir in diesem Fall nichts gebracht.
Ich habe hierbei ein anderes Problem, was sich vermutlich nicht lösen lässt. Und zwar hat die Anwendung von jedem Listener nur eine Instanz, heißt, dass nicht für jeden Server eine eigene Instanz vorhanden ist. Dementsprechend könnte man die einzelnen Listener alle als statische Methoden sehen und dementsprechend funktioniert das so, wie ich es mir ursprünglich gedacht habe gar nicht. - Schade.

dennoch Danke für die Vorschläge!