kommentare zu dem Plugin

Edit: Diese nachricht wurde als Spam markiert (WTF??) Habe also den Link zu etwas Code mit dem Code direkt ersetzt.
Edit 2: Auch die überarbeitete Version ist weg. Versuche jetzt das hier als pastebin zu schicken...



Hey!

Habe mir mal den Source-Code deines Plug-Ins angesehen, und mir sind ein paar Sachen noch aufgefallen (sortiert nach Wichtigkeit):

1) (wichtigstes)

PROBLEM:

Du benutzt aktuell für dein Scoreboard die Methode .getNewScoreboard(). Das wäre an sich kein Problem, aber es limitiert die Gesamtlänge des Namens ( Prefix + Spielername) auf 16 Zeichen! Das bedeutet: Wenn ein Spieler einen langen Namen hat (>10 Zeichen), könnte das Prefix nur noch 6 Zeichen inklusive Farbcodes sein! Das wäre dann z.B. "§cO | ", und mehr nicht.

Sollte die Länge zu groß sein, wird der Spieler einfach beim Join direkt wieder gekickt.

FIX:

Benutze einfach



.getMainScoreboard()

;)

2) Dein Plug-In ist wenig strukturiert. Und obwohl es jetzt bei so einem kleinen Plug-In noch nicht allzu wichtig ist, solltest du trotzdem insbesondere in der Zukunft darauf achten, ein paar "Clean Code"-Konventionen zu befolgen:

- Klassennamen: Großbuchstabe zu beginn

- Alles andere (Variablen, Methoden etc): klein

Außerdem, wenn ein Name mehrere Wörter beinhaltet:

- Eineneueklasse -> EineNeueKlasse (und auch mit Methoden, Variablen etc.)

Zudem: Du solltest wirklich unter-pakete anlegen. z.B. TreftCraft.<plugin-name>.main /.cmd /.events /.utils

gerade der 1. Teil (TreftCraft.<plugin-name>) ist hierbei wichtig, da sonst andere Plugins von dir mit dem selben Package einen Fehler auswerfen würden. Benenne also mal deine Packets um und erstelle ein paar neue.

3) (Du hast das bereits teilweise umgesetzt)

Erstelle für gleichbleibende Aufgaben Methoden. Bsp: Ein Team registrieren und sein Prefix setzen wäre dann:

public void createTeam(String name) {
  String prefix = cfg.getString("prefix."+name);

  sb.registerNewTeam(name);
  sb.getTeam(name).setPrefix(prefix);
}



Das kannst du btw. mit sehr vielen Sachen tun.

Sonst ein echt super Plug-In. Wenn du noch Fragen hast, oder z.B. Hilfe bei eigenen Rängen brauchst, schreib mich einfach auf Discord an: WeiiswurstDev#0016