SteamWar/BungeeCore
Archiviert
13
2

Playtime Command #285

Zusammengeführt
Lixfel hat 24 Commits von Playtime_Command nach master 2022-01-01 21:04:49 +01:00 zusammengeführt
2 geänderte Dateien mit 1 neuen und 3 gelöschten Zeilen
Nur Änderungen aus Commit 858e657974 werden angezeigt - Alle Commits anzeigen

Datei anzeigen

@ -37,7 +37,7 @@ public class PlaytimeCommand extends BasicCommand{
if(!(sender instanceof ProxiedPlayer))
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Genauso hier ;)

Genauso hier ;)
Veraltet
Review

Hier nichts Leerzeile

Hier nichts Leerzeile
Veraltet
Review

Warum ist hier wieder eine Leerzeile

Warum ist hier wieder eine Leerzeile
Veraltet
Review

weils schön aussieht

weils schön aussieht
Veraltet
Review

NEIN!

NEIN!
return;
YoyoNow markierte diese Unterhaltung als gelöst
Review

Hier meinte ich ein Space nach if

Hier meinte ich ein Space nach `if`
String s = Message.parse("TIME_FORMAT", sender);
String s = Message.parse("DECIMAL_FORMAT", sender);
Veraltet
Review

Ich würde mir noch wünschen, dass dieses DecimalFormat vllt auch im deutschen dann mit ',' ist. Ich wüsste aber nicht wie der DecimalFormat genaut funktioniert.

Ich würde mir noch wünschen, dass dieses DecimalFormat vllt auch im deutschen dann mit ',' ist. Ich wüsste aber nicht wie der DecimalFormat genaut funktioniert.
Veraltet
Review
format = NumberFormat.getNumberInstance(locale);
format.setMaximumFractionDigits(2);
``` format = NumberFormat.getNumberInstance(locale); format.setMaximumFractionDigits(2); ```
Veraltet
Review

Ein TIME_FORMAT gibt es nicht, dass hast du umbenannt in DECIMAL_FORMAT.

Ein `TIME_FORMAT` gibt es nicht, dass hast du umbenannt in `DECIMAL_FORMAT`.
String format = new DecimalFormat(s).format((SteamwarUser.get((ProxiedPlayer) sender).getOnlinetime() / (double) 3600));
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Vllt sollte hier eine neue Nachricht gewählt werden, damit der User besser versteht was für ein Wert er erhält.

Vllt sollte hier eine neue Nachricht gewählt werden, damit der User besser versteht was für ein Wert er erhält.
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Diese Variable könnte man inlinen, da diese nur einmal verwendet wird.

Diese Variable könnte man inlinen, da diese nur einmal verwendet wird.
Message.send(Message.parse("HOURS_PLAYED", sender) + format,sender);
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Vllt sollte man auch eine Zeile mehr hier hinter schmeißen.

Vllt sollte man auch eine Zeile mehr hier hinter schmeißen.
Veraltet
Review

Hier bitte die Locale des Spielers nutzen. Du könntest es auch mit:

String s = Message.parse("TIME_FORMAT", sender);
new DecimalFormat(s).format(...)

lösen. Und dann müsste in der .properties datei ein key für 'TIME_FORMAT'.

Hier bitte die Locale des Spielers nutzen. Du könntest es auch mit: ```java String s = Message.parse("TIME_FORMAT", sender); new DecimalFormat(s).format(...) ``` lösen. Und dann müsste in der `.properties` datei ein key für 'TIME_FORMAT'.
Veraltet
Review

Hier wtf?

Hier wtf?
Veraltet
Review

HOURS_PLAYED:

Message.send("HOURS_PLAYED", sender, format);
`HOURS_PLAYED`: ```java Message.send("HOURS_PLAYED", sender, format); ```
Veraltet
Review

YoyoNow, sr, aber dein Lösungsanschlag hier ist schlicht falsch. Das mit NumberFormat ist der korrekte Weg (wenn auch mit player.getLocale()).

YoyoNow, sr, aber dein Lösungsanschlag hier ist schlicht falsch. Das mit NumberFormat ist der korrekte Weg (wenn auch mit player.getLocale()).
Veraltet
Review

Sr, das kann so nicht funktionieren und ist im allgemeinen nicht ganz so schön. Bitte zurück zu NumberFormat.
YoyoNow hat da einen ganz schlechten Tipp/Empfehlung gegeben, sr.

Sr, das kann so nicht funktionieren und ist im allgemeinen nicht ganz so schön. Bitte zurück zu NumberFormat. YoyoNow hat da einen ganz schlechten Tipp/Empfehlung gegeben, sr.
Veraltet
Review

Ich habe nur auch eine alternative gesagt, nicht welche man verwenden soll.

Ich habe nur auch eine alternative gesagt, nicht welche man verwenden soll.
Veraltet
Review

Was zum Henker machst du denn da schon wieder? Vergiss LocalUtils und DECIMAL_FORMAT. Nimm NumberFormat.getNumberInstance(((ProxiedPlayer)sender).getLocale()).

Dein Code kann nicht funktionieren, denn wie sollen die LocaleUtils denn aus deinem String "###,##" die korrekte Sprache herausfinden, um diese dann dem NumberFormat zu übergeben? NumberFormat weiß selbst, wie die Zahlen in einer Sprache korrekt formatiert werden.

Was zum Henker machst du denn da schon wieder? Vergiss LocalUtils und DECIMAL_FORMAT. Nimm `NumberFormat.getNumberInstance(((ProxiedPlayer)sender).getLocale())`. Dein Code kann nicht funktionieren, denn wie sollen die LocaleUtils denn aus deinem String "###,##" die korrekte Sprache herausfinden, um diese dann dem NumberFormat zu übergeben? NumberFormat weiß selbst, wie die Zahlen in einer Sprache korrekt formatiert werden.

Datei anzeigen

@ -561,6 +561,4 @@ GDPR_STATUS_FINISHED=§7Packen abgeschlossen
#Playtime Command
HOURS_PLAYED=§7Deine Spielzeit beträgt§8: §e{0}h
#Player
DECIMAL_FORMAT=###,##