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

added tic tac toe to repository #165

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

aidan-diaz
Copy link

I completed the challenge: 5
I feel good about my code: 4

I was able to complete the challenge, but I'm not sure that I was able to emphasize object oriented programming enough. I did not make a game object, but I did try to focus on encapsulation and abstractions.

…perties and methods for the tictactoe game inside of it. created an instance of TicTacToe and stored it in the game variable.
@aidan-diaz
Copy link
Author

was able to go back and add a TicTacToe class - created an instance of it and stored it in the variable game. Everything was easy to move into the TicTacToe class, but the conditions in the takePlayerTurn method were not running upon click. using the bind method that all Functions inherit seemed to resolve this problem, but I am not quite understanding what is going on underneath the hood for the bind method. Any insight would be helpful.

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.

1 participant