Skip to content
Snippets Groups Projects
Commit 9158fa25 authored by skaur40's avatar skaur40
Browse files

Fixed the bugs

parent 2154a9b6
No related branches found
No related tags found
No related merge requests found
...@@ -112,6 +112,10 @@ public class DotsAndBoxesGrid { ...@@ -112,6 +112,10 @@ public class DotsAndBoxesGrid {
// A box is complete if the north and south horizontals and the east and west verticals have all been drawn. // 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). // FIXME: You'll need to fix this code (after writing a test first).
if(!horizontals[x][y] || !horizontals[x+1][y] || !verticals[x][y] || !verticals[x][y+1]){
return false;
}
return true; return true;
} }
...@@ -140,6 +144,9 @@ public class DotsAndBoxesGrid { ...@@ -140,6 +144,9 @@ public class DotsAndBoxesGrid {
} }
// FIXME: You need to throw an exception if the line was already drawn. // FIXME: You need to throw an exception if the line was already drawn.
if(horizontals[x][y]){
throw new IllegalStateException("Line already drawn");
}
this.horizontals[x][y] = true; this.horizontals[x][y] = true;
...@@ -171,6 +178,9 @@ public class DotsAndBoxesGrid { ...@@ -171,6 +178,9 @@ public class DotsAndBoxesGrid {
} }
// You need to throw an exception if the line was already drawn. // You need to throw an exception if the line was already drawn.
if(verticals[x][y]){
throw new IllegalStateException("Line already drawn");
}
this.verticals[x][y] = true; this.verticals[x][y] = true;
// Try to claim the north or south boxes // Try to claim the north or south boxes
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment