develop #2

Merged
swarga merged 10 commits from develop into master 2020-12-25 04:59:51 +01:00
Owner
No description provided.
pnowak added 7 commits 2020-12-24 21:44:17 +01:00
swarga requested changes 2020-12-24 21:56:43 +01:00
@ -18,2 +14,4 @@
self.player1 = Player("X", "Player 1")
self.player2 = Player("O", "Player 2")
def run(self):
Collaborator

Ten cały blok kodu możnaby sprowadzić do:

  1. Gracz wykonuje ruch.
  2. Sprawdzamy, czy mamy wygranego (remis).
  3. Zmieniamy gracza.
  4. Wracamy do 1.
Ten cały blok kodu możnaby sprowadzić do: 1. Gracz wykonuje ruch. 2. Sprawdzamy, czy mamy wygranego (remis). 3. Zmieniamy gracza. 4. Wracamy do 1.
main.py Outdated
@ -22,3 +31,3 @@
def __init__(self):
self.state = list(" "*9)
self.empty_fields = [0, 1, 2, 3, 4, 5, 6, 7, 8]
Collaborator

To możemy zastąpić generacją listy.

To możemy zastąpić generacją listy.
main.py Outdated
@ -30,9 +39,26 @@ class Board:
def make_move(self, token, field):
if self.state[field] == " ":
self.state[field] = token
self.empty_fields.remove(field)
Collaborator

Good.

Good.
main.py Outdated
@ -33,3 +43,4 @@
else:
print("This field's taken.")
def is_win(self, token, name):
Collaborator

Poczytaj o enumie.
Plasznę możemy reprezentować jako listę intów: 0, -1 oraz +1.
Pomyśl w jaki sposób może to ułatwić znajdywanie wygranej (modulo is your friend).

Poczytaj o enumie. Plasznę możemy reprezentować jako listę intów: 0, -1 oraz +1. Pomyśl w jaki sposób może to ułatwić znajdywanie wygranej (modulo is your friend).
main.py Outdated
@ -36,0 +55,4 @@
print(name + " has won.")
return True
if len(self.empty_fields) == 0:
print("It's a draw.")
Collaborator

Board sam z siebie nie powinien nam niczego printować, póki go o to nie poprosimy. Myślę, że informację o wygranym można przecieść do Tic-Tac-Toe.

Board sam z siebie nie powinien nam niczego printować, póki go o to nie poprosimy. Myślę, że informację o wygranym można przecieść do Tic-Tac-Toe.
main.py Outdated
@ -36,1 +59,4 @@
return True
return False
def get_state():
Collaborator

No napisz to!

No napisz to!
main.py Outdated
@ -51,2 +73,2 @@
if field == " ":
return
def choose_random_field(self, empty_fields):
return random.choice(empty_fields)
Collaborator

Jeden tab za dużo?

Jeden tab za dużo?
main.py Outdated
@ -57,1 +79,4 @@
for opt, arg in opts:
if opt in ("-n", "--numberofrounds"):
number_of_rounds = arg
Collaborator

Jeśli nie podam żadnego argumentu, to powinien mi się wyświetlić help (lub gra powinna ruszyć raz).

Jeśli nie podam żadnego argumentu, to powinien mi się wyświetlić help (lub gra powinna ruszyć raz).
pnowak added 1 commit 2020-12-25 04:21:01 +01:00
pnowak added 1 commit 2020-12-25 04:45:20 +01:00
swarga approved these changes 2020-12-25 04:51:11 +01:00
swarga closed this pull request 2020-12-25 04:51:18 +01:00
swarga reopened this pull request 2020-12-25 04:51:30 +01:00
swarga approved these changes 2020-12-25 04:54:09 +01:00
swarga added 1 commit 2020-12-25 04:59:39 +01:00
swarga merged commit 0b973804e7 into master 2020-12-25 04:59:51 +01:00
swarga referenced this issue from a commit 2020-12-25 04:59:52 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
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#2
No description provided.