Rework SWScoreboard to ProtocolLib and remove unneccassary VersionDependents #72

Lixfel · 2020-11-28T21:59:53+01:00

Lixfel hat

2020-11-28 21:59:53 +01:00

kommentiert

Besitzer

Closes #71

PS: So schaut ein guter PullRequest aus: 10:1 gelöschte zu erstellten Zeilen, und kein bisschen Funktionalität weniger & mehr Überblick! ;)

Closes #71 PS: So schaut ein guter PullRequest aus: 10:1 gelöschte zu erstellten Zeilen, und kein bisschen Funktionalität weniger & mehr Überblick! ;)

AdmiralSeekrank hat die Änderungen 2020-11-28 22:55:21 +01:00 genehmigt

Chaoscaot hat die Änderungen 2020-11-29 01:45:42 +01:00 genehmigt

Chaoscaot hat einen Kommentar hinterlassen

Besitzer

Sind zwar nur kleinigkeiten, würde mir dennoch eine erklärung des toggles wünchen und die vereinheitlichung der static final Variablennamen.

SpigotCore_Main/src/de/steamwar/scoreboard/SWScoreboard.java Veraltet

						
				@ -32,2 +40,3 @@

				    private static int toggle = 0;

				    private static HashMap<Player, Object> playerBoards = new HashMap<>(); //Object -> Scoreboard | Alle Versionen in der Map!

				    private static final ProtocolManager manager = ProtocolLibrary.getProtocolManager();

Chaoscaot hat

2020-11-29 01:36:15 +01:00

kommentiert

Besitzer

Also entweder kleingeschrieben oder komplett groß bei den static final Variablen, und nicht das eine so und das andere so.

Lixfel hat

2020-11-29 20:12:19 +01:00

kommentiert

Autor

Besitzer

Konstanten GROß, anderes wie gewohnt camelCase.

SpigotCore_Main/src/de/steamwar/scoreboard/SWScoreboard.java

						
				@ -34,0 +48,4 @@

				        setScoreboardConstants(0);

				        setScoreboardConstants(1);

				        Bukkit.getScheduler().runTaskTimer(Core.getInstance(), () -> {

				            toggle ^= 1; // Toggle between 0 and 1