SteamWar/BungeeCore
SteamWar/BungeeCore
Archiviert
13
2
Fork 1
Code Issues 14 Pull-Requests 4 Releases Wiki Aktivität

Ranked #306

Zusammengeführt
Lixfel hat 25 Commits von Ranked nach master 2022-03-13 20:26:16 +01:00 zusammengeführt
Diskussion 3 Commits 25 Geänderte Dateien 9 +8 -6
2 geänderte Dateien mit 8 neuen und 6 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen
Nur Änderungen aus Commit 530171c2e3 werden angezeigt - Alle Commits anzeigen

4
src/de/steamwar/bungeecore/comms/handlers/FightEndsHandler.java
Datei anzeigen

@ -73,11 +73,11 @@ public class FightEndsHandler implements SpigotHandler {
if (blueSchemOwner != 0 && redSchemOwner != 0) {
for (int bluePlayer : fightEndsPacket.getBluePlayers()) {
int playerElo = UserElo.getElo(bluePlayer, fightEndsPacket.getGameMode());
int playerElo = UserElo.getElo(bluePlayer, fightEndsPacket.getGameMode()).orElse(1000);
UserElo.setElo(bluePlayer, fightEndsPacket.getGameMode(), (int) Math.round(playerElo + K * (blueResult - blueWinExpectation)));
}
for (int redPlayer : fightEndsPacket.getRedPlayers()) {
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2022-03-11 12:15:29 +01:00 kommentiert
Veraltet
Review
Link kopieren

Du hast hier sehr viel Codedopplung drin, würde mir eine teamunabhängige Abstrahierung wünschen.

Du hast hier sehr viel Codedopplung drin, würde mir eine teamunabhängige Abstrahierung wünschen.
int playerElo = UserElo.getElo(redPlayer, fightEndsPacket.getGameMode());
int playerElo = UserElo.getElo(redPlayer, fightEndsPacket.getGameMode()).orElse(1000);
UserElo.setElo(redPlayer, fightEndsPacket.getGameMode(), (int) Math.round(playerElo + K * (1 - blueResult - redWinExpectation)));
}
}

10
src/de/steamwar/bungeecore/sql/UserElo.java
Datei anzeigen

@ -19,6 +19,8 @@
package de.steamwar.bungeecore.sql;
import java.util.Optional;
public class UserElo {
private UserElo() {
}
@ -27,15 +29,15 @@ public class UserElo {
private static final Statement setElo = new Statement("UPDATE UserELo SET Elo = ? WHERE Season = ? AND UserID = ? AND GameMode = ?");
private static final Statement place = new Statement("SELECT COUNT(*) AS Place FROM UserElo WHERE GameMode = ? AND Elo > ? AND Season = ?");
public static int getElo(int userID, String gameMode) {
public static Optional<Integer> getElo(int userID, String gameMode) {
return getElo(Season.getSeason(), userID, gameMode);
}
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2022-03-11 12:22:04 +01:00 kommentiert
Veraltet
Review
Link kopieren

Da lieber eine Map<Map<>>, das bläst sich dann weniger auf.

Da lieber eine Map<Map<>>, das bläst sich dann weniger auf.
YoyoNow hat 2022-03-11 13:14:34 +01:00 kommentiert
Veraltet
Review
Link kopieren

Wie meinst du das genau?

Wie meinst du das genau?
Lixfel hat 2022-03-11 13:16:08 +01:00 kommentiert
Veraltet
Review
Link kopieren

HashMaps brauchen zur Kollisionsvermeidung immer Freiraum. Je kleiner die HashMaps bleiben, desto besser.

HashMaps brauchen zur Kollisionsvermeidung immer Freiraum. Je kleiner die HashMaps bleiben, desto besser.
YoyoNow hat 2022-03-11 13:20:21 +01:00 kommentiert
Veraltet
Review
Link kopieren

Würdest du dann GameMode UserId oder UserId GameMode als reihenfolge wählen?

Würdest du dann GameMode UserId oder UserId GameMode als reihenfolge wählen?
Lixfel hat 2022-03-11 13:22:49 +01:00 kommentiert
Veraltet
Review
Link kopieren

Definitiv Map<GameMode, Map<User, >>

Definitiv Map<GameMode, Map<User, >>
public static int getElo(int season, int userID, String gameMode) {
public static Optional<Integer> getElo(int season, int userID, String gameMode) {
return elo.select(rs -> {
if (rs.next())
return rs.getInt("Elo");
return 1000;
return Optional.of(rs.getInt("Elo"));
return Optional.empty();
}, userID, gameMode, season);
}