nathanwoodburn/comp1110-ass2
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 1 Wiki Activity

player: Fixed canPlay bug

Browse Source
This commit is contained in:
Nathan Woodburn 2023-05-09 10:46:51 +10:00
parent 77aaa07984
commit 519e9e840e
Signed by: nathanwoodburn
GPG Key ID: 203B000478AD0EF1
1 changed files with 7 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/comp1110/ass2/Player.java
View File

@ -243,10 +243,11 @@ public class Player {
// Check if the player has placed all their settlers or villages // Check if the player has placed all their settlers or villages
int numSettlers = 30 - ((state.getNumPlayers() - 2) * 5); int numSettlers = 30 - ((state.getNumPlayers() - 2) * 5);
boolean hasSettler = (settlers.length < numSettlers); boolean hasSettler = (settlers.length < numSettlers);
boolean hasVillage = (settlers.length < 5); boolean hasVillage = (villages.length < 5);
if (state.getCurrentPhase() != 'E'){ if (!hasSettler && !hasVillage) return false;
if (!hasSettler && !hasVillage) return false; // if (state.getCurrentPhase() != 'E'){
} // if (!hasSettler && !hasVillage) return false;
// }
// Add used coords // Add used coords
@ -302,13 +303,13 @@ public class Player {
else if(y > state.boardHeight - 1) continue; else if(y > state.boardHeight - 1) continue;
switch (state.getCurrentPhase()) { switch (state.getCurrentPhase()) {
case 'E' -> { case 'E' -> {
if (!islandCoords.contains(cord)) return true; if (!islandCoords.contains(cord) && hasSettler) return true;
if ((isAdjacent(cord, playerVillageCoords) || isAdjacent(cord, playerSettlerCoords))) return true; if ((isAdjacent(cord, playerVillageCoords) || isAdjacent(cord, playerSettlerCoords))) return true;
} }
// Settlement Phase // Settlement Phase
case 'S' -> { case 'S' -> {
// if the settler is adjacent with any of the pieces return true // if the settler is adjacent with any of the pieces return true
if ((isAdjacent(cord, playerVillageCoords) || isAdjacent(cord, playerSettlerCoords))) return true; if ((isAdjacent(cord, playerVillageCoords) || isAdjacent(cord, playerSettlerCoords)) && hasSettler) return true;
} }
} }
} }