dots_v1 by alexthenugget · Pull Request #1 · alexthenugget/Dots_Game

alexthenugget · 2025-03-29T09:50:35Z

v1

12gerts

Нет README
Нет аннотаций типов
Не настроен black и mypy
Нет прокоммита
Нет указания о используемых библиотеках
Нет тестов

12gerts · 2025-03-31T10:06:04Z

+
+class DotsGame:
+    def __init__(self, root):
+        self.root = root


это все протектед

12gerts · 2025-03-31T10:06:52Z

+        self.setup_ui()
+        self.create_grid()


слишком тяжело для инита

12gerts · 2025-03-31T10:08:10Z

+            color = "darkred"
+            winning_score = red_score
+        else:
+            winner = "Ничья"


12gerts · 2025-03-31T10:10:56Z

+        if hasattr(self.game_logic, 'is_board_full') and self.game_logic.is_board_full():
+            message = f"Доска заполнена! Игра окончена! Победитель: {winner}" if winner != "Ничья" else "Доска заполнена! Игра окончена! Ничья!"
+        else:
+            message = f"Игра завершена досрочно! Победитель: {winner}" if winner != "Ничья" else "Игра завершена досрочно! Ничья!"


вот это бы упростить можно. А еще можно в enum сохранять сразу Победитель: Красный игрок и не надо будет условие проверять, вы все равно нигде это больше не используете

12gerts · 2025-03-31T10:11:15Z

+                entry["score"]
+            ))
+
+    def create_end_game_button(self):


огромный метод, декомпозиция нужна

12gerts · 2025-03-31T10:17:39Z

+
+
+def load_leaderboard():
+    if not os.path.exists("leaderboard.txt"):


используйте pathlib

12gerts · 2025-03-31T10:18:25Z

+    leaderboard = []
+    for line in lines:
+        if line.strip():
+            parts = line.strip().split(",")


так может лучше json вместо txt?

12gerts · 2025-03-31T10:18:39Z

+                "name": parts[1],
+                "score": int(parts[2])
+            })
+    while len(leaderboard) < 3:


опять магия

12gerts · 2025-03-31T10:20:26Z

+
+
+def update_leaderboard(new_name, new_score):
+    leaderboard = load_leaderboard()


Сделайте его моделькой какой-нибудь вместо словаря (BaseModel, dataclass, NamedTuple)

12gerts · 2025-03-31T10:20:55Z

+
+def update_leaderboard(new_name, new_score):
+    leaderboard = load_leaderboard()
+    if new_score > leaderboard[-1]["score"] or leaderboard[-1]["name"] == "None":


лучше конвертировать условие и сразу выйти

12gerts · 2025-04-29T07:47:25Z

.idea должна быть в gitignore

12gerts · 2025-04-29T07:47:52Z

@@ -1 +1,63 @@
-# Dots_Game
+# 🎮 Точки - классическая стратегическая игра


как устанавливать? как запускать?

12gerts · 2025-04-29T07:48:40Z

+    RED_WIN = ("Красный игрок", "darkred")
+    DRAW = ("Ничья", "black")
+
+    def __init__(self, winner_text: str, color: str) -> None:


откуда в enum инит взялся???

12gerts · 2025-04-29T07:50:02Z

+    BLUE_WIN = ("Синий игрок", "darkblue")
+    RED_WIN = ("Красный игрок", "darkred")
+    DRAW = ("Ничья", "black")


вот это что-то странное, если честно.

Сделайте что- то одно и mapping

12gerts · 2025-04-29T07:59:03Z

+        blue_score = self._game_logic.scores[1]
+        red_score = self._game_logic.scores[2]


что-то как-то не особо красиво это

12gerts · 2025-04-29T08:00:08Z

+        )
+
+        if result != GameResult.DRAW:
+            winning_score = blue_score if result == GameResult.BLUE_WIN else red_score


тут тоже непонятно зачем условие, можно сразу хранить нужное

12gerts · 2025-04-29T08:00:56Z

+        for i in range(start_x, end_x, CELL_SIZE):
+            self._canvas.create_line(i, start_y, i, end_y, tags="grid_line", fill="lightblue")
+        for i in range(start_y, end_y, CELL_SIZE):


i какие-то...

12gerts · 2025-04-29T08:02:41Z

+                else:
+                    pass


а в чем прикол?

12gerts · 2025-04-29T12:09:17Z

            f"{'Доска заполнена' if board_full else 'Игра завершена досрочно'}! " f"Игра окончена! {result.winner_text}"
        )

+        winning_score = blue_score if result == GameResult.BLUE_WIN else red_score


тут без условия, маппинг нужен

12gerts · 2025-04-29T12:10:12Z

 flake8 = "^7.2.0"
 isort = "^6.0.1"
 pre-commit = "^4.2.0"
+pydantic = "^2.11.3"


для чего он добавился, если он нигде не используется?

pydantic добавила, потому что при запуске с poetry обнаружила, что он используется в leaderboard и нигде не указан как внешняя библиотека

А он сам не подтягивается?

при запуске с poetry не пропускал, пока я не добавила явно в зависимости, поэтому решила оставить

Delete dots/leaderboard.txt v2 v3 v4 v5 v6 v7 v8 v8

12gerts reviewed Mar 31, 2025

View reviewed changes

alexthenugget force-pushed the dots_v1 branch from 8436265 to 2603d68 Compare April 25, 2025 10:31

12gerts reviewed Apr 29, 2025

View reviewed changes

Comment thread dots/game_logic.py

12gerts reviewed Apr 29, 2025

View reviewed changes

v1

6d7b488

Delete dots/leaderboard.txt v2 v3 v4 v5 v6 v7 v8 v8

alexthenugget force-pushed the dots_v1 branch from 245ee9a to 6d7b488 Compare May 2, 2025 06:06

alexthenugget added 2 commits May 2, 2025 13:45

v9

64a62aa

Update README.md

3ccd2c4



		def load_leaderboard():
		if not os.path.exists("leaderboard.txt"):



		def update_leaderboard(new_name, new_score):
		leaderboard = load_leaderboard()

		@@ -1 +1,63 @@
		# Dots_Game No newline at end of file
		# 🎮 Точки - классическая стратегическая игра

		blue_score = self._game_logic.scores[1]
		red_score = self._game_logic.scores[2]

Conversation

alexthenugget commented Mar 29, 2025

Uh oh!

12gerts left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants