2005-12-14 Carl Worth <cworth@cworth.org>

[ttt] / src / ttt-board.c

diff --git a/src/ttt-board.c b/src/ttt-board.c
index 73d2741c4c446a3386a634824178e6b1d04bfb59..f8f62e045de587631fa10c5c1caceee3a3fdecd0 100644 (file)
--- a/src/ttt-board.c
+++ b/src/ttt-board.c
@@ -121,16 +121,40 @@ ttt_error_t ttt_board_make_move (ttt_board_t *board, int move)
 ttt_cell_t ttt_board_is_won (ttt_board_t *board)
 {
     int i;
+
+    /* Loop over rows and columns looking for a win. */
     for (i = 0; i <= 2; i++)
     {
-       if (board->cells[i] != TTT_CELL_EMPTY && board->cells[i] == board->cells[i + 3] && board->cells[i] == board->cells[i + 6])
-           return(board->cells[i]);
-       else if (board->cells[i * 3] != TTT_CELL_EMPTY && board->cells[i * 3] == board->cells[(i * 3) + 1] && board->cells[i * 3] == board->cells[(i * 3) + 2])
-           return(board->cells[i * 3]);
+       /* Check the current column */
+       if (board->cells[i] != TTT_CELL_EMPTY &&
+           board->cells[i] == board->cells[i + 3] &&
+           board->cells[i] == board->cells[i + 6])
+       {
+           return board->cells[i];
+       }
+       
+       /* Check the current row */
+       if (board->cells[i * 3] != TTT_CELL_EMPTY &&
+           board->cells[i * 3] == board->cells[(i * 3) + 1] &&
+           board->cells[i * 3] == board->cells[(i * 3) + 2])
+       {
+           return board->cells[i * 3];
+       }
+    }
+
+    if (board->cells[0] != TTT_CELL_EMPTY &&
+       board->cells[0] == board->cells[4] &&
+       board->cells[0]== board->cells[8])
+    {
+       return board->cells[0];
     }
-    if (board->cells[0] != TTT_CELL_EMPTY && board->cells[0] == board->cells[4] && board->cells[0]== board->cells[8])
-       return(board->cells[0]);
-    if (board->cells[2] != TTT_CELL_EMPTY && board->cells[2] == board->cells[4] && board->cells[2]== board->cells[6])
-       return(board->cells[2]);
-    return(TTT_CELL_EMPTY);
+
+    if (board->cells[2] != TTT_CELL_EMPTY &&
+       board->cells[2] == board->cells[4] &&
+       board->cells[2]== board->cells[6])
+    {
+       return board->cells[2];
+    }
+
+    return TTT_CELL_EMPTY;
 }