fixed isValidSquare to comply with row/col standard. refactored curly…

… braces.

src/model/Board.java

@@ -4,9 +4,8 @@
 
 /**
  * The representation is a 8x8 grid where
- * A[y][x] denotes the (x, y) indexed piece.
- * It is swapped because we dereference by "row" first,
- * which is "y".
+ * A[row][col] marks the position of the checker piece.
+ * Smoke starts at lower row and fire starts at higher row.
  * @author Ayamin
  *
  */
@@ -28,15 +27,18 @@ public Board() {
 		movesSinceCapture = 0;
 		init();
 	}
-
+
+	public boolean isValidSquare(int row, int col) {
+		return 	0 <= row && row < BOARD_SIZE && 
+				0 <= col && col < BOARD_SIZE;
+	}
+
 	/**
 	 * Initialize the board putting checker pieces in their starting locations
 	 */
-	private void init()
-	{
+	private void init() {
 		for(int row = 0; row < 3; row++){
-			for (int col = 0; col < 4; col++)
-			{
+			for (int col = 0; col < 4; col++) {
 				Piece red_piece = new Piece(Color.RED, 2*col + (row % 2), row);
 				Piece black_piece = new Piece(Color.BLACK, 2*col + (BOARD_SIZE - 1 - row) %2, BOARD_SIZE - 1 - row);
 				representation[row][2*col+ (row % 2)] = red_piece;
@@ -48,29 +50,23 @@ private void init()
 	/**
 	 * Print the current board representation
 	 */
-	public void print()
-	{
-		for(int row = 0; row < BOARD_SIZE; row++)
-		{
-			for (int col = 0; col < BOARD_SIZE; col++)
-			{
+	public void print() {
+		for(int row = 0; row < BOARD_SIZE; row++) {
+			for (int col = 0; col < BOARD_SIZE; col++) {
 				if (!isOccupied(row, col))
 					System.out.print("| ");
-				else if (representation[row][col].getColor() == Color.RED)
-				{
+				else if (representation[row][col].getColor() == Color.RED) {
 					if (representation[row][col].getType() == Type.NORMAL)
 						System.out.print("|r");
 					else	
 						System.out.print("|R");
 				}
-				else
-				{
+				else {
 					if(representation[row][col].getType() == Type.NORMAL)
 						System.out.print("|b");
 					else
 						System.out.print("|B");
 				}
-
 			}
 			System.out.println("|");
 		}
@@ -80,8 +76,7 @@ else if (representation[row][col].getColor() == Color.RED)
 	 * return true if square contains a piece
 	 * return false otherwise
 	 */
-	public boolean isOccupied(int row, int col)
-	{
+	public boolean isOccupied(int row, int col) {
 		return representation[row][col] != null;
 	}
 
@@ -99,7 +94,6 @@ public ArrayList<Piece> getBlackPieces() {
 
 	public void setBlackPieces(ArrayList<Piece> black_pieces) {
 		this.black_pieces = black_pieces;
-	}
-
+	}
 }