From 8094acc81d0231902938e45ca910351e109b59dd Mon Sep 17 00:00:00 2001
From: Muyuan Li <mli30@myune.edu.au>
Date: Mon, 25 Jul 2022 00:00:54 +0530
Subject: [PATCH] Bugs fixed in DotsAndBoxesGrid.java
---
src/main/java/dotsandboxes/DotsAndBoxesGrid.java | 12 +++++++++++-
1 file changed, 11 insertions(+), 1 deletion(-)
diff --git a/src/main/java/dotsandboxes/DotsAndBoxesGrid.java b/src/main/java/dotsandboxes/DotsAndBoxesGrid.java
index a9e7c5b..60d2119 100644
--- a/src/main/java/dotsandboxes/DotsAndBoxesGrid.java
+++ b/src/main/java/dotsandboxes/DotsAndBoxesGrid.java
@@ -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+1][y] && verticals[x][y] && verticals[x][y+1]){
+ return true;
+ }
+
+ return false;
}
/** Tries to claim a box for a player. If the box is complete, sets the ownership and returns true. */
@@ -140,6 +144,9 @@ public class DotsAndBoxesGrid {
}
// FIXME: You need to throw an exception if the line was already drawn.
+ if(horizontals[x][y] == true){
+ throw new IllegalStateException("Horizontal line already drawn");
+ }
this.horizontals[x][y] = true;
@@ -171,6 +178,9 @@ public class DotsAndBoxesGrid {
}
// You need to throw an exception if the line was already drawn.
+ if(verticals[x][y] == true){
+ throw new IllegalStateException("Vertical line already drawn");
+ }
this.verticals[x][y] = true;
// Try to claim the north or south boxes
--
GitLab