Skip to content
Snippets Groups Projects
Commit b24e4aa4 authored by Matt Ralston's avatar Matt Ralston
Browse files

Fixed two known bugs

parent 3a48c12e
Branches main
No related tags found
No related merge requests found
......@@ -111,8 +111,7 @@ public class DotsAndBoxesGrid {
}
// A box is complete if the north and south horizontals and the east and west verticals have all been drawn.
// FIXME: You'll need to fix this code (after writing a test first).
return true;
return horizontals[x][y] && horizontals[x][y+1] && verticals[x][y] && verticals[x+1][y];
}
/** Tries to claim a box for a player. If the box is complete, sets the ownership and returns true. */
......@@ -139,7 +138,9 @@ public class DotsAndBoxesGrid {
throw new IndexOutOfBoundsException(String.format("y was %d, which is out of range. Range is 0 to %d", y, height));
}
// FIXME: You need to throw an exception if the line was already drawn.
if (this.horizontals[x][y]) {
throw new IllegalStateException(String.format("line at %d, %d was already drawn and cannot be redrawn.", x, y));
}
this.horizontals[x][y] = true;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment