-
Notifications
You must be signed in to change notification settings - Fork 16
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
Add Tic-Tac-Toe Game code in python #38
Add Tic-Tac-Toe Game code in python #38
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good work! I have left some comments for changes, go through them :D
@@ -0,0 +1,282 @@ | |||
# Tic-Tac-Toe game for multiplayer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# Tic-Tac-Toe game for multiplayer | |
# Tic-Tac-Toe |
## Tic-Tac-Toe | ||
Tic-Tac-Toe is a multiplayer simple game. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is redundant. If you want to add stuff about the working of the code you can do so under the Description
sub-heading
|
||
## Code | ||
```python | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
```python | ||
|
||
from os import system, name | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
||
# import sleep to show output for some time period | ||
from time import sleep | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
||
# for mac and linux(here, os.name is 'posix') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# for mac and linux(here, os.name is 'posix') | |
# for mac and linux(here, os.name is 'posix') |
|
||
|
||
clear() | ||
str1 = "." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you come up with a better informative naming for these variables. For example, box1
, block1
, etc.
This goes for all the other variables too. To make it easier for other people to comprehend what's happening without wasting too much time!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM :-)
Add Tic-Tac-Toe Game code in python
Fixes: #14
Type of change
Checklist: