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

state: isPhaseOver sped up code

Browse Source
This commit is contained in:
Nathan Woodburn 2023-05-08 13:07:03 +10:00
parent 73293edb43
commit f6f3f4a6a1
Signed by: nathanwoodburn
GPG Key ID: 203B000478AD0EF1
1 changed files with 8 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
src/comp1110/ass2/State.java
View File

@ -409,28 +409,20 @@ public class State {
* Defaults to simple mode
*/
public boolean isPhaseOver() {
return isPhaseOver(true);
}
/**
* is the phase over?
* @param simple boolean don't check all player moves
*/
public boolean isPhaseOver(boolean simple){
boolean resourcesLeft = false;
for (Resource r : resources) {
if (r.isAvailable() && r.getType() != 'S') resourcesLeft = true;
}
boolean moveLeft = false;
if (simple) {
if (getCurrentPlayer().canPlay(this)) moveLeft = true;
}
else
{
for (Player player : players) {
if (player.canPlay(this)) moveLeft = true;
int numSettlers = 30 - ((numPlayers - 2) * 5);
for (Player p: players) {
boolean canPlay = false;
if (p.getVillages().length < 5) canPlay = true;
if (p.getSettlers().length < numSettlers) canPlay = true;
if (canPlay) {
if (p.canPlay(this)) moveLeft = true;
}
}
return !resourcesLeft || !moveLeft;
}