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
+281 -3
jannis commented 2026-10-15 09:06:57 +00:00
Collaborator
Copy link
Copy source
No description provided.
jannis added 5 commits 2026-10-15 09:06:57 +00:00
fixed issues after ending in colorjump b175433701
fixed issues after ending in colorjump 103bd23bf4
fixed one issue in block battle 096b035503
added BuildBattle 7e0c126a0d
"Refactored BuildBattle game logic and chunk generator: optimized code structure, improved plot assignment, and enhanced rating system." 53b13b6560
Pupsi requested changes 2026-10-16 15:32:05 +00:00
Dismissed
src/main/java/eu/mhsl/minenet/minigames/instance/game/stateless/types/buildBattle/BuildBattle.java Outdated
@@ -0,0 +101,4 @@
this.spawnPosx = 0;
this.spawnPosz++;
}
this.spawnPosx++;
Pupsi commented 2026-10-16 15:22:38 +00:00
Collaborator
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
Author
Collaborator
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?
Pupsi marked this conversation as resolved
src/main/java/eu/mhsl/minenet/minigames/instance/game/stateless/types/buildBattle/BuildBattle.java Outdated
@@ -0,0 +169,4 @@
Player owner = this.currentOwner;
this.currentOwner = null;
this.getScore().insertResult(owner, sum);
Pupsi commented 2026-10-16 15:31:49 +00:00
Collaborator
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.
Pupsi marked this conversation as resolved
src/main/java/eu/mhsl/minenet/minigames/instance/game/stateless/types/buildBattle/BuildBattle.java Outdated
@@ -0,0 +164,4 @@
}
private void finishRound() {
int sum = this.currentVotes.values().stream().mapToInt(Integer::intValue).sum();
Pupsi commented 2026-10-16 15:38:47 +00:00
Collaborator
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.
Pupsi marked this conversation as resolved
jannis added 1 commit 2026-10-16 18:43:19 +00:00
fixed issues in BuildBattle 75cd304bda
Pupsi approved these changes 2026-10-16 18:48:49 +00:00
jannis merged commit 4bd03c2603 into develop 2026-10-16 18:50:20 +00:00
jannis deleted branch develop-jannis 2026-10-16 18:50:20 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Pupsi
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
besiece bytedream (Ruben Otto) goldbaja (Jannik Goldbach) Hannes (Hannes Frommann) jannis MineTec (Elias Müller) olischma (Martin Olischläger) Pupsi (Lars Neuhaus)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: MineNet/Minigames#15