C
Chrisliebaer
Guest
Naja, das Plugin ist relativ Sinnfrei, da wie schon erwähnt, viele jeden Tag eine neue IP haben werden oder zusammen über den selben Anschluss spielen. Außerdem stellt sich mir die Frage, was für ein Sinn es hat, den Spielernamen in der MotD anzuzeigen, wobei natürlich das jeder machen kann, wie er möchte.
Außerdem möchte ich mal davor warnen, dass das Plugin wirklich schlecht entwickelt ist. Das dürfte für die eigentliche Aufgabe mal locker <90% unnötige Rechenzeit fressen.
Vor allem das mehrfache aufrufen der Stringmethoden zusammen mit dem Festplattenzugriff (sofern Bukkit das nicht cached; ungeachtet der Tatsache, dass es an der Stelle nichtmal notwendig ist die Config zu speichern). Ansonsten gibts noch ein paar kleinere Ungereimtheiten, aber DAS ist wirklich alles andere als performant. Für ~100 Zeilen ganz schon viel komisches Zeugs drin. Als kleines Spaßprojekt sicherlich lustig, aber auf nem Produktivserver würde ICH das nicht ausführen, zumal das Ping Event ziemlich häufig gefeuert wird (jedes mal, wenn jemand seine Serverliste aktualisiert) da sollte man an der Stelle schon darauf achten, dass das fix geht.
Außerdem möchte ich mal davor warnen, dass das Plugin wirklich schlecht entwickelt ist. Das dürfte für die eigentliche Aufgabe mal locker <90% unnötige Rechenzeit fressen.
Code:
for (int in = 0; in < motd.length(); in++)
{
nachricht = motd;
nachricht = nachricht.replace("[Player]", name);
e.setMotd(nachricht);
saveConfig();
}
Vor allem das mehrfache aufrufen der Stringmethoden zusammen mit dem Festplattenzugriff (sofern Bukkit das nicht cached; ungeachtet der Tatsache, dass es an der Stelle nichtmal notwendig ist die Config zu speichern). Ansonsten gibts noch ein paar kleinere Ungereimtheiten, aber DAS ist wirklich alles andere als performant. Für ~100 Zeilen ganz schon viel komisches Zeugs drin. Als kleines Spaßprojekt sicherlich lustig, aber auf nem Produktivserver würde ICH das nicht ausführen, zumal das Ping Event ziemlich häufig gefeuert wird (jedes mal, wenn jemand seine Serverliste aktualisiert) da sollte man an der Stelle schon darauf achten, dass das fix geht.