MineNet/Minigames
6
0 Fork 0
Code Issues Pull Requests 1 Packages Projects Releases 1 Wiki Activity

added Build Battle #15

Merged
jannis merged 6 commits from develop-jannis into develop 2026-10-16 18:50:20 +00:00
Conversation 3 Commits 6 Files Changed 7
+9 -7
1 changed files with 9 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 75cd304bda - Show all commits
src/main/java/eu/mhsl/minenet/minigames/instance/game/stateless/types/buildBattle/BuildBattle.java
+9 -7
View File
@@ -97,11 +97,11 @@ public class BuildBattle extends StatelessGame {
);
this.spawnPositions.put(player, plotCenter);
if(this.spawnPosx >= rowCount) {
if(this.spawnPosx >= rowCount - 1) {
this.spawnPosx = 0;
this.spawnPosz++;
}
this.spawnPosx++;
} else this.spawnPosx ++;
Pupsi marked this conversation as resolved Outdated
Pupsi commented 2026-10-16 15:22:38 +00:00
Outdated
Review
Copy link
Copy source

Im if wird x wieder auf 0 gesetzt, aber danach durch spawnPosx++ direkt wieder auf 1. x ist also nur beim ersten Spieler 0.

Im if wird x wieder auf 0 gesetzt, aber danach durch spawnPosx++ direkt wieder auf 1. x ist also nur beim ersten Spieler 0.
jannis commented 2026-10-16 17:52:18 +00:00
Outdated
Review
Copy link
Copy source

Im if wird x wieder auf 0 gesetzt, aber danach durch spawnPosx++ direkt wieder auf 1. x ist also nur beim ersten Spieler 0.

also das spawnPosx ++ vor das if statement?

> Im if wird x wieder auf 0 gesetzt, aber danach durch spawnPosx++ direkt wieder auf 1. x ist also nur beim ersten Spieler 0. also das spawnPosx ++ vor das if statement?
player.setGameMode(GameMode.ADVENTURE);
this.scheduler().scheduleNextTick(() -> player.teleport(plotCenter.add(0, 1.5, 0)));
return super.onPlayerJoin(player);
@@ -164,12 +164,14 @@ public class BuildBattle extends StatelessGame {
}
private void finishRound() {
int sum = this.currentVotes.values().stream().mapToInt(Integer::intValue).sum();
this.getPlayers().forEach(p -> p.getInventory().clear());
int voteSum = this.currentVotes.values().stream()
Pupsi marked this conversation as resolved Outdated
Pupsi commented 2026-10-16 15:38:47 +00:00
Outdated
Review
Copy link
Copy source

Vielleicht lieber einen Durchschnitt oder so anstatt einer Summe? Dann wird es nicht unfair, wenn bewertende Spieler leaven und insgesamt weniger Punkte vergeben werden können.

Vielleicht lieber einen Durchschnitt oder so anstatt einer Summe? Dann wird es nicht unfair, wenn bewertende Spieler leaven und insgesamt weniger Punkte vergeben werden können.
.mapToInt(Integer::intValue)
.sum();
int score = (int) Math.round((double) voteSum / this.getPlayers().size());
Player owner = this.currentOwner;
this.getPlayers().forEach(p -> p.getInventory().clear());
Pupsi marked this conversation as resolved Outdated
Pupsi commented 2026-10-16 15:31:49 +00:00
Outdated
Review
Copy link
Copy source

Wieso nicht erst insertResult(this.currentOwner, sum) und dann currentOwner = null ? Dann wäre Zeile 170 unnötig.

Wieso nicht erst insertResult(this.currentOwner, sum) und dann currentOwner = null ? Dann wäre Zeile 170 unnötig.
this.getScore().insertResult(this.currentOwner, score);
this.currentOwner = null;
this.getScore().insertResult(owner, sum);
}
private boolean isInArea(Point block, Player player) {