13
0

Arena Store #18

Zusammengeführt
Lixfel hat 2 Commits von arena_store nach master 2021-12-22 12:44:43 +01:00 zusammengeführt

Datei anzeigen

@ -0,0 +1,27 @@
package de.steamwar.bungeecore;
public class Arenaserver extends Subserver {
private final String mode;
private final String map;
Review

Map bin ich mir nicht sicher, ob hier mit einbezogen werden sollte, da sie in allen Spielmodi keinen Einfluss auf das Spielgeschehen hat.

Map bin ich mir nicht sicher, ob hier mit einbezogen werden sollte, da sie in allen Spielmodi keinen Einfluss auf das Spielgeschehen hat.
Review

Zum anzeigen, was für eine Map der Server hat. Und bei MissleWars, wo die eine Map schon beliebter ist als die andere.

Ob das ganze in die Suche einbezogen werden soll, warte ich auf User Settings im Bungee wo man dann irgendwie sagen kann, ich möchte kein Merging, gleiche Map und gleicher Modus oder nur gleicher Modus.

Zum anzeigen, was für eine Map der Server hat. Und bei MissleWars, wo die eine Map schon beliebter ist als die andere. Ob das ganze in die Suche einbezogen werden soll, warte ich auf User Settings im Bungee wo man dann irgendwie sagen kann, ich möchte kein Merging, gleiche Map und gleicher Modus oder nur gleicher Modus.
private final boolean allowMerge;
Chaoscaot markierte diese Unterhaltung als gelöst
Review

eventFightID und ranked sind egal, was dich hier interessiert, ist, ob der Kampf mergable ist. Dafür reicht ein boolean.

eventFightID und ranked sind egal, was dich hier interessiert, ist, ob der Kampf mergable ist. Dafür reicht ein boolean.
public Arenaserver(String serverName, String mode, String map, boolean allowMerge, int port, ProcessBuilder processBuilder, Runnable shutdownCallback) {
super(Servertype.ARENA, serverName, port, processBuilder, shutdownCallback);
this.mode = mode;
this.map = map;
this.allowMerge = allowMerge;
}
public String getMode() {
return mode;
}
public String getMap() {
return map;
}
public boolean isAllowMerge() {
return allowMerge;
}
Chaoscaot markierte diese Unterhaltung als gelöst
Review

Nur dieser Konstruktor wird benötigt, die anderen sind nur Codeduplication und nicht Nodefähig. Bitte entfernen.

Nur dieser Konstruktor wird benötigt, die anderen sind nur Codeduplication und nicht Nodefähig. Bitte entfernen.
}