SteamWar/BungeeCore
Archiviert
13
2

Add PollResult Command #153

Manuell gemergt
Lixfel hat 4 Commits von pollresult nach master 2020-11-27 17:11:54 +01:00 zusammengeführt
3 geänderte Dateien mit 4 neuen und 6 gelöschten Zeilen
Nur Änderungen aus Commit a3179483fd werden angezeigt - Alle Commits anzeigen

Datei anzeigen

@ -41,10 +41,8 @@ public class PollresultCommand extends BasicCommand {
Map<String, Integer> voted = PollAnswer.getCurrentResults();
Message.send("POLLRESULT_HEADER", player, PollAnswer.getAllAnswered(), PollSystem.getQuestion());
int i = 1;
for (Map.Entry<String, Integer> e: voted.entrySet()) {
Message.send("POLLRESULT_LIST", sender, i, e.getKey(), e.getValue());
i++;
Message.send("POLLRESULT_LIST", sender, e.getKey(), e.getValue());
}
}
Review

i ist nicht wirklich sinnvoll, kann man weglassen.

i ist nicht wirklich sinnvoll, kann man weglassen.
}

Datei anzeigen

@ -70,7 +70,7 @@ public class PollAnswer {
}
public static Integer getAllAnswered() {
ResultSet set = SQL.select("SELECT Count(UserID) AS Times FROM PollAnswer");
ResultSet set = SQL.select("SELECT Count(UserID) AS Times FROM PollAnswer WHERE Question = ?", PollSystem.getQuestion());
Veraltet
Review

Da sollte man evtl. noch nach WHERE Question = ? überprüfen.

Da sollte man evtl. noch nach WHERE Question = ? überprüfen.
try {
if(!set.next())
throw new SecurityException("Could not get PollAnswers");

Datei anzeigen

@ -55,5 +55,5 @@ HELP_BAU_BAU=§8/§ehelp bau §8- §7Hilfe zu nützlichen Werkzeugen
HELP_BAU_BAU_HOVER=§eNützliche Zusatzfunktionen
#PollresultCommand
POLLRESULT_HEADER=§e§lEs haben {0} abgestimmt auf die Frage: §f{1}
POLLRESULT_LIST=§e{0}. §7{1} mit {2} stimmen
POLLRESULT_HEADER=§eEs haben {0} abgestimmt auf die Frage: §7{1}
Veraltet
Review

Das ganze würde ich nicht §e§l machen, das wäre etwas penetrant, eher §7§l oder nur §e. Dazu bitte nicht §f (Nicht im Farbschema!), wenn dann §7

Das ganze würde ich nicht §e§l machen, das wäre etwas penetrant, eher §7§l oder nur §e. Dazu bitte nicht §f (Nicht im Farbschema!), wenn dann §7
POLLRESULT_LIST=§e{0}§8: §7{1}
Veraltet
Review

Bitte eher §e{2}§8: §7{1}

Bitte eher §e{2}§8: §7{1}