X-Git-Url: https://git.piment-noir.org/?p=Project_algorithmic_C.git;a=blobdiff_plain;f=lib%2Fothello.c;h=6c03f1dacaa7acbc57c3d55ad51df6e866257b6f;hp=4f1b83ec52cd1b8cda16f1696b85c56a9e499b4b;hb=aa739af2efbd062790fb7b4690a87e3774efa974;hpb=7aab9e03ce0779a461044f56bc3f478e1c8d2460

diff --git a/lib/othello.c b/lib/othello.c
index 4f1b83e..6c03f1d 100644
--- a/lib/othello.c
+++ b/lib/othello.c
@@ -23,8 +23,13 @@
 #include "othello.h"
 #include "debug.h"
 
+/**
+ * Get current round player integer
+ * @param  round_count current round integer
+ * @return             current round player
+ */
 unsigned int current_player(unsigned int round_count) {
-   
+
     if (round_count % 2 != 0) {
         return player_two;
     } else {
@@ -32,7 +37,12 @@ unsigned int current_player(unsigned int round_count) {
     }
 }
 
-unsigned int current_opponent(unsigned int current_player) {
+/**
+ * Get current round opponent integer
+ * @param  current_player current round player
+ * @return                current round opponent integer
+ */
+static unsigned int current_opponent(unsigned int current_player) {
 
     if (current_player == player_one) {
         return player_two;
@@ -46,11 +56,18 @@ unsigned int current_opponent(unsigned int current_player) {
  * |
  * y
  * |
- * v 
+ * v
  * The origin O has (1, 1) coordinates */
 
+/**
+ * Get pawn value at coordinates (y,x)
+ * @param  y          y coordinate
+ * @param  x          x coordinate
+ * @param  pawn_array array of played pawns
+ * @return            pawn integer type
+ */
 unsigned int get_box_value(int y, int x, unsigned int pawn_array[board_size][board_size]) {
-   
+
     return pawn_array[y-1][x-1];
 }
 
@@ -67,7 +84,7 @@ bool is_box_type(int y, int x, unsigned int pawn_array[board_size][board_size],
 }
 
 static bool is_valid_coordinates(int y, int x) {
-    
+
     if ((y > 0 && y < board_size + 1) && \
             (x > 0 && x < board_size + 1)) {
         return true;
@@ -76,18 +93,24 @@ static bool is_valid_coordinates(int y, int x) {
     }
 }
 
-/* helper function to set a correct value at the (y, x) coordinates in the pawns array */
+/**
+ * Helper function to set a correct value at the (y,x) coordinates in the pawns array
+ * @param y      [description]
+ * @param x      [description]
+ * @param type   [description]
+ * @param pawn_array [description]
+ */
 void set_pawn(int y, int x, unsigned int type, unsigned int pawn_array[board_size][board_size]) {
 
     if (type > 0 && type < 3 && \
             is_valid_coordinates(y, x)) {
         pawn_array[y-1][x-1] = type;
-    }
+    } //FIXME: else case should set invalid values to permit to catch errors
 }
 
 /* reverse the pawn at (y, x) coordinates if it exists */
 static void reverse_pawn(int y, int x, unsigned int pawn_array[board_size][board_size]) {
-     
+
     if (is_box_type(y, x, pawn_array, black)) {
         set_pawn(y, x, white, pawn_array);
     } else if (is_box_type(y, x, pawn_array, white)) {
@@ -104,7 +127,10 @@ void zero_pawns(unsigned int pawn_array[board_size][board_size]) {
     }
 }
 
-/* set the pawns in the start position */
+/**
+ * Set the pawns in the start position
+ * @param pawn_array array of played pawns
+ */
 void init_pawns(unsigned int pawn_array[board_size][board_size]) {
 
     /* the 2D array zeroing is not necessary if it is properly initialized to zero */
@@ -180,7 +206,7 @@ unsigned int eval_winner(unsigned int nb_white, unsigned int nb_black) {
     }
 }
 
-static unsigned int count_pawn_to_reverse_one_direction(int y, int x, int direction, unsigned int current_player, unsigned int pawn_array[board_size][board_size]) {
+static unsigned int count_pawn_to_reverse_one_direction(int y, int x, unsigned int direction, unsigned int current_player, unsigned int pawn_array[board_size][board_size]) {
     unsigned int nb_pawns_reversed = 0;
     int moving_y = y, moving_x = x;
 
@@ -199,8 +225,8 @@ static unsigned int count_pawn_to_reverse_one_direction(int y, int x, int direct
     return nb_pawns_reversed;
 }
 
-/* revert the pawns if needed in one direction */ 
-static unsigned int reverse_one_direction(int y, int x, int direction, unsigned int current_player, unsigned int pawn_array[board_size][board_size], bool dry_run) {
+/* revert the pawns if needed in one direction */
+static unsigned int reverse_one_direction(int y, int x, unsigned int direction, unsigned int current_player, unsigned int pawn_array[board_size][board_size], bool dry_run) {
     unsigned int nb_pawns_reversed = 0;
     int moving_y = y, moving_x = x;
 
@@ -218,20 +244,21 @@ static unsigned int reverse_one_direction(int y, int x, int direction, unsigned
     return nb_pawns_reversed;
 }
 
-/* loop optimized version of valid_shot changing nothing to the pawns 2D array */
-static bool is_legal_shot(int y, int x, unsigned int current_player, unsigned int pawn_array[board_size][board_size]) {
+/* loop optimized version of valid_shot function changing nothing to the pawns 2D array */
+bool is_legal_shot(int y, int x, unsigned int current_player, unsigned int pawn_array[board_size][board_size]) {
     unsigned int nb_pawns_reversed = 0;
-    
+
     if (!is_valid_coordinates(y, x) || !is_box_type(y, x, pawn_array, empty)) {
         return false;
     }
-    
+
     for (unsigned int direction = north; direction <= north_west; direction++) {
         nb_pawns_reversed += reverse_one_direction(y, x, direction, current_player, pawn_array, true);
         if (nb_pawns_reversed > 0) {
             return true;
         }
     }
+    return false;
 }
 
 /* play the shot if legal and flip or reverse the necessary pawns */
@@ -245,42 +272,50 @@ unsigned int valid_shot(int y, int x, unsigned int current_player, unsigned int
     for (unsigned int direction = north; direction <= north_west; direction++) {
         nb_pawns_reversed += reverse_one_direction(y, x, direction, current_player, pawn_array, false);
     }
-    
+
     if (nb_pawns_reversed == 0) {
-        return 0;
+        return nb_pawns_reversed;
     }
 
     set_pawn(y, x, current_player, pawn_array);
     return nb_pawns_reversed;
 }
 
-struct shots_list_s build_playable_shots_list(int y, int x, unsigned int current_player, unsigned int pawn_array[board_size][board_size]) {
-
-    for (unsigned int i = 0; i <= board_size; i++) {
-        for (unsigned int j = 0; j <= board_size; j++) {
-            if (is_legal_shot(y, x, current_player, pawn_array)) {
-
-            }
-        }
+static void add_shots_list_cell(int y, int x, unsigned int type, struct shots_list_s* shots_list) {
+    struct shots_list_s* list_cell = (struct shots_list_s*)malloc(sizeof(struct shots_list_s));
+    if (!list_cell) {
+        exit(EXIT_FAILURE);
     }
-}
-
-void create_shots_list_cell(struct shots_list_s* shots_list_cell) {
 
+    if (type > 0 && type < 5 && is_valid_coordinates(y, x)) {
+        list_cell->y = y;
+        list_cell->x = x;
+        list_cell->type = type;
+        list_add_tail(&(list_cell->list), &(shots_list->list));
+    }
 }
 
-static void set_shots_list_coordinates(int y, int x, struct shots_list_s* shots_list) {
+void free_shots_list(struct shots_list_s* shots_list) {
+    struct shots_list_s* list_counter;
 
-    if (is_valid_coordinates(y, x)) {
-        shots_list->y = y;
-        shots_list->x = x;
+    while (!list_empty(&shots_list->list)) {
+        list_counter = list_entry(shots_list->list.next, struct shots_list_s, list);
+        list_del(&(list_counter->list));
+        free(list_counter);
     }
 }
 
-void set_shots_lists_cell(int y, int x, unsigned int type, struct shots_list_s* shots_list) {
+void build_playable_shots_list(unsigned int current_player, struct shots_list_s* shots_list, unsigned int pawn_array[board_size][board_size]) {
 
-    if (type > 0 && type < 5) {
-        set_shots_list_coordinates(y, x, shots_list);
-        shots_list->type = type;
+    for (unsigned int i = 0; i <= board_size; i++) {
+        for (unsigned int j = 0; j <= board_size; j++) {
+            if (is_legal_shot(i, j, current_player, pawn_array)) {
+                add_shots_list_cell(i, j, hint_allowed, shots_list);
+            /* FIXME: a neighbourhood detection is needed
+            } else if (is_box_type(i, j, pawn_array, empty)){
+                add_shots_list_cell(i, j, hint_forbidden, shots_list);
+            */
+            }
+        }
     }
 }