Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

updated TIC TAC TOE #33

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

shreyaa2904
Copy link

The provided code implements a simple Tic-Tac-Toe game in C, featuring two players who take turns placing their markers on a 3x3 grid. It includes functions to draw the game board, update cell states, and determine the winner. Key corrections were made to ensure the winner-checking logic correctly identifies winning conditions by using equality checks instead of bitwise operations. The game responds to keyboard inputs for navigation and selection, providing visual feedback and error handling when cells are already occupied. Overall, this code serves as a foundation for an interactive and reliable Tic-Tac-Toe experience.

The provided code implements a simple Tic-Tac-Toe game in C, featuring two players who take turns placing their markers on a 3x3 grid. It includes functions to draw the game board, update cell states, and determine the winner. Key corrections were made to ensure the winner-checking logic correctly identifies winning conditions by using equality checks instead of bitwise operations. The game responds to keyboard inputs for navigation and selection, providing visual feedback and error handling when cells are already occupied. Overall, this code serves as a foundation for an interactive and reliable Tic-Tac-Toe experience.
@sasdallas
Copy link

?? This looks AI generated and useless to codebase

@sasdallas
Copy link

Taking a closer look it seems like the AI overwrote most existing code, replacing it with a comment of "kept for readability"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants