Separate agents from game logic #30

paul90317 · 2024-02-06T04:53:09Z

In this commit, I split the original ttt.c, making it easier to add new agents. To achieve this, I also made the following changes:

Introduced move_t to make negamax a pure function.
Moved qsort out from available_moves to make it a part of the game logic.

Additionally, I declared table and move_record as static arrays because they are unnecessary to be dynamically allocated.

I also noticed #29, and I made some modifications to it without changing its score; specifically, I changed
is_consecutive(score, -1) to score < 0, and vice versa.

agents/negamax.c

agents/negamax.h

game_logic.h

agents/negamax.c

jserv

Revise the git commit messages to reflect the reasons behind the proposed changes from a third-person perspective.

agents/negamax.c

game_logic.h

In this commit, the original `ttt.c` was split to facilitate the addition of new agents. Several changes were implemented to achieve this: 1. `move_t` was introduced to transform `negamax` into a pure function 2. `qsort` was relocated from `available_moves` to incorporate it into the game logic. The readability of the code also increases with the following changes: 1. `table` and `move_record` were declared as static arrays. 2. The condition `is_consecutive(score, -1)` was replaced with `score < 0`, and vice versa.

jserv · 2024-02-07T12:53:27Z

Thank @paul90317 for contributing. Next time, you should specify the meaningful name for git branch used by pull requests.

paul90317 force-pushed the temp branch 2 times, most recently from fcacab5 to 6ab9ffb Compare February 6, 2024 05:07