SteamWar/MissileWars
Archiviert
13
0

Balance ItemCountdown for uneven Teams #12

Manuell gemergt
Lixfel hat 6 Commits von BalanceItems nach master 2020-09-26 20:46:30 +02:00 zusammengeführt
Besitzer
Keine Beschreibung angegeben.
Lixfel hat 2020-09-26 11:49:03 +02:00 Änderungen angefragt
@ -47,2 +49,2 @@
itemList.add(SpecialItem.getRandomItem());
}
List<Player> blue = new ArrayList<>(MissileWars.getBlueTeam().getPlayers());
List<Player> red = new ArrayList<>(MissileWars.getBlueTeam().getPlayers());
Besitzer

Das nochmal in eine ArrayList einzufügen, ist nicht wirklich elegant, evtl. kann man die Funktion weiterhin pro Team verwenden?

Das nochmal in eine ArrayList einzufügen, ist nicht wirklich elegant, evtl. kann man die Funktion weiterhin pro Team verwenden?
Autor
Besitzer

Ich wollte es dadurch eleganter machen ich überlege nochmal wie man das vllt ändern kann

Ich wollte es dadurch eleganter machen ich überlege nochmal wie man das vllt ändern kann
@ -57,2 +59,2 @@
ItemStack itemStack = itemStackList.remove(random.nextInt(itemStackList.size()));
givePlayerItem(p, itemStack);
givePlayerItem(blue.get((i + randomOffsetBlue) % blue.size()), itemStack);
givePlayerItem(red.get((i + randomOffsetRed) % red.size()), itemStack);
Besitzer

Das ist nicht ganz so zufällig, dass es immer derselbe sein wird (solange niemand joint/leavt), der mehrfach Items bekommt.

Das ist nicht ganz so zufällig, dass es immer derselbe sein wird (solange niemand joint/leavt), der mehrfach Items bekommt.
Autor
Besitzer

Warum das denn, bei welcher Konstellation von Teams glaubst du das denn?

Warum das denn, bei welcher Konstellation von Teams glaubst du das denn?
Lixfel hat 2020-09-26 20:33:44 +02:00 Änderungen angefragt
@ -50,3 +55,1 @@
giveTeamItem(MissileWars.getBlueTeam(), itemList);
giveTeamItem(MissileWars.getRedTeam(), itemList);
}
Player blue = MissileWars.getBlueTeam().getPlayers().removeFirst();
Besitzer

Bitte das Ganze zu einer Funktion im Team machen um nicht das Ganze doppelt zu coden! Dann muss auch nicht getPlayers() public sein.

Bitte das Ganze zu einer Funktion im Team machen um nicht das Ganze doppelt zu coden! Dann muss auch nicht getPlayers() public sein.
Lixfel hat die Änderungen 2020-09-26 20:46:26 +02:00 genehmigt
Lixfel hat diesen Pull-Request 2020-09-26 20:46:30 +02:00 geschlossen
Lixfel löschte die Branch BalanceItems 2020-09-26 20:46:37 +02:00
Dieses Repo ist archiviert. Du kannst Pull-Requests nicht kommentieren.
Keine Beschreibung angegeben.