Changed Fight Scoreboard to SWScoreboard #194
@ -28,48 +28,50 @@ import de.steamwar.fightsystem.winconditions.*;
|
||||
import de.steamwar.scoreboard.SWScoreboard;
|
||||
import de.steamwar.scoreboard.ScoreboardCallback;
|
||||
import org.bukkit.Bukkit;
|
||||
import org.bukkit.event.EventHandler;
|
||||
import org.bukkit.event.Listener;
|
||||
import org.bukkit.event.player.PlayerJoinEvent;
|
||||
import org.bukkit.event.player.PlayerQuitEvent;
|
||||
import org.bukkit.scoreboard.Scoreboard;
|
||||
|
||||
import java.util.EnumSet;
|
||||
import java.util.HashMap;
|
||||
import java.util.Set;
|
||||
|
||||
public class FightScoreboard {
|
||||
|
||||
private FightScoreboard(){}
|
||||
public class FightScoreboard implements Listener {
|
||||
|
||||
private static final Set<FightState> fullScoreboard = EnumSet.of(FightState.RUNNING, FightState.SPECTATE);
|
||||
private static int index = 0;
|
||||
private static ScoreBoardObjective objective;
|
||||
|
||||
|
|
||||
public static void init(){
|
||||
Lixfel
hat
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(), () -> {
|
||||
Bukkit.getOnlinePlayers().forEach(player -> SWScoreboard.removeScoreboard(player));
|
||||
|
||||
FightTeam fightTeam = getIndexDisplay();
|
||||
ScoreBoardObjective objective;
|
||||
if(fightTeam != null)
|
||||
objective = teamScoreboard(fightTeam);
|
||||
else
|
||||
objective = generalScoreboard();
|
||||
|
||||
Bukkit.getOnlinePlayers().forEach(player -> SWScoreboard.createScoreboard(player, new ScoreboardCallback() {
|
||||
@Override
|
||||
public HashMap<String, Integer> getData() {
|
||||
return objective.getScores();
|
||||
}
|
||||
|
||||
@Override
|
||||
public String getTitle() {
|
||||
return objective.getTitle();
|
||||
}
|
||||
}));
|
||||
}, 0, 200);
|
||||
Bukkit.getScheduler().scheduleSyncRepeatingTask(FightSystem.getPlugin(), () -> nextIndexDisplay(), 0, 200);
|
||||
}
|
||||
|
||||
|
Lixfel
hat
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() {
|
||||
return Bukkit.getScoreboardManager().getMainScoreboard();
|
||||
}
|
||||
|
||||
@EventHandler()
|
||||
|
Lixfel
hat
Hast du darauf geachtet, dass die Teammitglieder immer noch keinen Nametag über sich haben & in der passenden Farbe rechts angezeigt werden? Aufgrund der derzeitigen SWScoreboard-Implementierung (welche ich überarbeiten möchte) habe ich da nämlich so meine Befürchtungen. Hast du darauf geachtet, dass die Teammitglieder immer noch keinen Nametag über sich haben & in der passenden Farbe rechts angezeigt werden? Aufgrund der derzeitigen SWScoreboard-Implementierung (welche ich überarbeiten möchte) habe ich da nämlich so meine Befürchtungen.
Chaoscaot
hat
Die Nametags werden gehided und der Name hat auch die richtige Farbe in der Sidebar. Die Nametags werden gehided und der Name hat auch die richtige Farbe in der Sidebar.
|
||||
public void onPlayerJoin(PlayerJoinEvent event) {
|
||||
SWScoreboard.createScoreboard(event.getPlayer(), new ScoreboardCallback() {
|
||||
@Override
|
||||
public HashMap<String, Integer> getData() {
|
||||
return objective.getScores();
|
||||
}
|
||||
|
||||
@Override
|
||||
public String getTitle() {
|
||||
return objective.getTitle();
|
||||
}
|
||||
});
|
||||
}
|
||||
|
||||
@EventHandler
|
||||
public void onPlayerQuit(PlayerQuitEvent event) {
|
||||
SWScoreboard.removeScoreboard(event.getPlayer());
|
||||
}
|
||||
|
||||
private static ScoreBoardObjective teamScoreboard(FightTeam fightTeam){
|
||||
ScoreBoardObjective objective = new ScoreBoardObjective(fightTeam.getColoredName());
|
||||
fightTeam.getPlayers().forEach(fp -> {
|
||||
@ -123,6 +125,11 @@ public class FightScoreboard {
|
||||
if(index > 2)
|
||||
return;
|
||||
index = 0;
|
||||
FightTeam team = getIndexDisplay();
|
||||
|
Lixfel
hat
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
Es wird wohl immer einmal kein Scoreboard angezeigt werden weil 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
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
|
||||
if(team != null)
|
||||
objective = teamScoreboard(team);
|
||||
else
|
||||
objective = generalScoreboard();
|
||||
}
|
||||
|
||||
|
Lixfel
hat
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 static class ScoreBoardObjective {
|
||||
|
||||
Man sollte das Scoreboard nicht regelmäßig bei jedem Spieler neu setzen, das macht das SWScoreboard schon von selbst.