Skip to content
Snippets Groups Projects
Commit 9cef2445 authored by hmccrist's avatar hmccrist
Browse files

Fixed bugs

parent 9231f72f
No related branches found
No related tags found
No related merge requests found
......@@ -112,7 +112,11 @@ 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;
if (horizontals[x][y] && horizontals[x][y+1] && verticals[x][y] && verticals[x+1][y]) {
return true;
} else {
return false;
}
}
/** Tries to claim a box for a player. If the box is complete, sets the ownership and returns true. */
......@@ -141,6 +145,10 @@ public class DotsAndBoxesGrid {
// FIXME: You need to throw an exception if the line was already drawn.
if (horizontals[x][y]) {
throw new IllegalArgumentException("Horizontal line already drawn");
}
this.horizontals[x][y] = true;
// Try to claim the north or south boxes
......@@ -172,6 +180,10 @@ public class DotsAndBoxesGrid {
// You need to throw an exception if the line was already drawn.
if (verticals[x][y]) {
throw new IllegalArgumentException("Vertical line already drawn");
}
this.verticals[x][y] = true;
// Try to claim the north or south boxes
boolean claimE = claimBox(x, y, player);
......
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