pnowak/Tic-Tac-Toe
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

develop #1

Merged
swarga merged 3 commits from develop into master 2020-12-23 23:44:42 +01:00
Conversation 0 Commits 3 Files Changed 1 +38 -22
pnowak commented 2020-12-23 22:29:34 +01:00
Owner
Copy Link
No description provided.
pnowak added 2 commits 2020-12-23 22:29:35 +01:00
swarga requested changes 2020-12-23 22:40:27 +01:00
swarga left a comment
Collaborator
Copy Link

Przeczytaj komentarze do odpowiednich linijek.

Przeczytaj komentarze do odpowiednich linijek.
main.py Outdated
@ -17,6 +15,23 @@ class TicTacToe:
"""
def run(self):
player1 = Player("X")
swarga commented 2020-12-23 22:36:07 +01:00
Collaborator
Copy Link

Takie rzeczy dobrze byłoby przenieść do konstruktora (pamiętaj o self!)

Takie rzeczy dobrze byłoby przenieść do konstruktora (pamiętaj o `self`!)
main.py Outdated
@ -20,0 +20,4 @@
board = Board()
for x in range(1,10):
swarga commented 2020-12-23 22:37:15 +01:00
Collaborator
Copy Link

Ten warunek jest już niepotrzebny, skoro masz isWin().
Gra powinna trwać do wygranej lub remisu.

Ten warunek jest już niepotrzebny, skoro masz `isWin()`. Gra powinna trwać do wygranej lub remisu.
main.py
@ -33,6 +48,18 @@ class Board:
else:
print("This field's taken.")
def isWin(self, token):
swarga commented 2020-12-23 22:38:22 +01:00
Collaborator
Copy Link

Ta funkcja byłaby dużo bardziej użyteczna, gdyby informowała nas który z graczy jest zwycięzcą (zostaw sobie ToDo w kodzie do przemyślenia na później).

Ta funkcja byłaby dużo bardziej użyteczna, gdyby informowała nas który z graczy jest zwycięzcą (zostaw sobie ToDo w kodzie do przemyślenia na później).
main.py
@ -36,1 +60,4 @@
return True
return False
def get_state():
swarga commented 2020-12-23 22:39:13 +01:00
Collaborator
Copy Link

Do zaktualizowania.

Do zaktualizowania.
main.py Outdated
@ -48,3 +76,3 @@
emptyField = True
while emptyField:
field = state[randint(0,8)]
randomField = randint(0,8)
swarga commented 2020-12-23 22:39:54 +01:00
Collaborator
Copy Link

Lepiej wybrać jedno z dostępnych pól, niż strzelać na ślepo aż na takie się trafi.

Lepiej wybrać jedno z dostępnych pól, niż strzelać na ślepo aż na takie się trafi.
pnowak added 1 commit 2020-12-23 23:37:12 +01:00
swarga merged commit 7d59652863 into master 2020-12-23 23:44:42 +01:00
swarga referenced this issue from a commit 2020-12-23 23:44:44 +01:00
Działający PoC
Sign in to join this conversation.
Reviewers
No reviewers
swarga
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: pnowak/Tic-Tac-Toe#1
No description provided.
Delete Branch "develop"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?