Skip to content

Kld/master#4

Open
kdefliese wants to merge 12 commits intoAda-C4:kld/masterfrom
kdefliese:kld/master
Open

Kld/master#4
kdefliese wants to merge 12 commits intoAda-C4:kld/masterfrom
kdefliese:kld/master

Conversation

@kdefliese
Copy link
Copy Markdown

No description provided.

@brittinator brittinator self-assigned this Feb 9, 2016
this.state = "game over";
winner = this.board[0];
}
else if (this.board[3] == this.board[4] && this.board[4] == this.board[5]) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a way to DRY this up with function calls and iteration?

@sallyamoore
Copy link
Copy Markdown

Your code looks great! Nice implementation! 👍

@kdefliese
Copy link
Copy Markdown
Author

Thanks for taking a look, @sallyamoore!

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.

3 participants