SteamWar/FightSystem
SteamWar/FightSystem
Archiviert
13
1
Fork 0
Code Issues 6 Pull-Requests Releases Wiki Aktivität

Changed Fight Scoreboard to SWScoreboard #194

Manuell gemergt
Lixfel hat 7 Commits von scoreboard-reimplement nach master 2020-11-17 09:29:41 +01:00 zusammengeführt
Diskussion 6 Commits 7 Geänderte Dateien 3 +7 -8
1 geänderte Dateien mit 7 neuen und 8 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen
Nur Änderungen aus Commit 2bc9bded5c werden angezeigt - Alle Commits anzeigen

11
FightSystem_Main/src/de/steamwar/fightsystem/utils/FightScoreboard.java
Datei anzeigen

@ -45,7 +45,7 @@ public class FightScoreboard implements Listener {
private static ScoreBoardObjective objective; private static ScoreBoardObjective objective;
Lixfel hat 2020-10-10 20:39:25 +02:00 kommentiert
Veraltet
Review
Link kopieren

Man sollte das Scoreboard nicht regelmäßig bei jedem Spieler neu setzen, das macht das SWScoreboard schon von selbst.

Man sollte das Scoreboard nicht regelmäßig bei jedem Spieler neu setzen, das macht das SWScoreboard schon von selbst.
public static void init(){ public static void init(){
Lixfel hat 2020-11-17 07:23:02 +01:00 kommentiert
Review
Link kopieren

Fände es besser, direkt titel und scores zu speichern (siehe unten)

Fände es besser, direkt titel und scores zu speichern (siehe unten)
Bukkit.getScheduler().scheduleSyncRepeatingTask(FightSystem.getPlugin(), () -> nextIndexDisplay(), 0, 200); Bukkit.getScheduler().scheduleSyncRepeatingTask(FightSystem.getPlugin(), FightScoreboard::nextIndexDisplay, 0, 200);
} }
Lixfel hat 2020-11-17 07:24:08 +01:00 kommentiert
Review
Link kopieren

Listener sollen im Ordner Listener sein, in diesem Fall solltest du wsl. einfach im ConnectionListener im JonListener einen Aufruf hierher machen.

Listener sollen im Ordner Listener sein, in diesem Fall solltest du wsl. einfach im ConnectionListener im JonListener einen Aufruf hierher machen.
public static Scoreboard getScoreboard() { public static Scoreboard getScoreboard() {
@ -73,7 +73,7 @@ public class FightScoreboard implements Listener {
} }
private static ScoreBoardObjective teamScoreboard(FightTeam fightTeam){ private static ScoreBoardObjective teamScoreboard(FightTeam fightTeam){
ScoreBoardObjective objective = new ScoreBoardObjective(fightTeam.getColoredName()); final ScoreBoardObjective objective = new ScoreBoardObjective(fightTeam.getColoredName());
fightTeam.getPlayers().forEach(fp -> { fightTeam.getPlayers().forEach(fp -> {
if(fp.isLiving()) if(fp.isLiving())
objective.addScore(fightTeam.getPrefix() + fp.getPlayer().getName(), (int) Math.ceil(fp.getPlayer().getHealth())); objective.addScore(fightTeam.getPrefix() + fp.getPlayer().getName(), (int) Math.ceil(fp.getPlayer().getHealth()));
@ -82,7 +82,7 @@ public class FightScoreboard implements Listener {
} }
private static ScoreBoardObjective generalScoreboard(){ private static ScoreBoardObjective generalScoreboard(){
ScoreBoardObjective objective = new ScoreBoardObjective("§6Kampf"); final ScoreBoardObjective objective = new ScoreBoardObjective("§6Kampf");
if (Config.Timeout || Config.Points || Config.HeartRatioTimeout) { if (Config.Timeout || Config.Points || Config.HeartRatioTimeout) {
int fightTime = FightSystem.getFightTime(); int fightTime = FightSystem.getFightTime();
if (fightTime >= 60) if (fightTime >= 60)
@ -123,9 +123,8 @@ public class FightScoreboard implements Listener {
private static void nextIndexDisplay() { private static void nextIndexDisplay() {
index++; index++;
if(index > 2) if(index > 2)
return;
index = 0; index = 0;
FightTeam team = getIndexDisplay(); final FightTeam team = getIndexDisplay();
if(team != null) if(team != null)
Lixfel hat 2020-10-15 09:24:40 +02:00 kommentiert
Veraltet
Review
Link kopieren

Das schaut mir katastrophal verbuggt aus, da dürfte nie das korrekte Scoreboard angezeigt werden!

Das schaut mir katastrophal verbuggt aus, da dürfte nie das korrekte Scoreboard angezeigt werden!
YoyoNow hat 2020-10-24 20:59:08 +02:00 kommentiert
Veraltet
Review
Link kopieren

Es wird wohl immer einmal kein Scoreboard angezeigt werden weil Index = 0 bedeutet die getIndexDisplay() Methode gibt null zurück. Dies sollte behoben werden.

Es wird wohl immer einmal kein Scoreboard angezeigt werden weil `Index = 0` bedeutet die `getIndexDisplay()` Methode gibt `null` zurück. Dies sollte behoben werden.
YoyoNow hat 2020-10-25 10:15:50 +01:00 kommentiert
Veraltet
Review
Link kopieren

Der Fehler ist nun behoben, bzw um 13:00 am gleichen Tag wurde dieser Fehler noch behoben, deswegen habe ich den Pull-Request jetzt nach einer guten Zeit drauf gucken genehmigt

Der Fehler ist nun behoben, bzw um 13:00 am gleichen Tag wurde dieser Fehler noch behoben, deswegen habe ich den Pull-Request jetzt nach einer guten Zeit drauf gucken genehmigt
objective = teamScoreboard(team); objective = teamScoreboard(team);
else else
@ -135,7 +134,7 @@ public class FightScoreboard implements Listener {
private static class ScoreBoardObjective { private static class ScoreBoardObjective {
Lixfel hat 2020-11-17 07:29:09 +01:00 kommentiert
Review
Link kopieren

Die Funktion nextIndexDisplay sollte sich mit der Funktion getIndexDisplay zusammenführen lassen.

Die Funktion nextIndexDisplay sollte sich mit der Funktion getIndexDisplay zusammenführen lassen.
private final String title; private final String title;
private HashMap<String, Integer> scores = new HashMap<>(); private final HashMap<String, Integer> scores = new HashMap<>();
public ScoreBoardObjective(String title) { public ScoreBoardObjective(String title) {
this.title = title; this.title = title;