feat(ui): работа с новыми классами состояний приложения

src/mind_reader/core/__init__.py

@@ -1,5 +1,7 @@
-from mind_reader.core.game_state import GameState
+from .state import AppState, LobbyState, SessionState
 
 __all__ = [
-    "GameState",
+    "AppState",
+    "LobbyState",
+    "SessionState",
 ]

src/mind_reader/core/game_state.py

@@ -1,33 +0,0 @@
-class GameState:
-    def __init__(self, local_player_name: str):
-        self.local_player_name: str = local_player_name
-        self.remote_player_name: str | None = None
-        self.lobby_players: dict[str, str] = {self.local_player_name: "NONE"}
-
-    def add_remote_player(self, player_name: str):
-        self.remote_player_name = player_name
-        self.lobby_players[self.remote_player_name] = "NONE"
-
-    def delete_remote_player(self):
-        if self.remote_player_name in self.lobby_players:
-            del self.lobby_players[self.remote_player_name]
-        self.remote_player_name = None
-
-    def set_player_role(self, player_name: str, role: str) -> bool:
-        current_role = self.lobby_players.get(player_name)
-        if current_role == role:
-            return False
-
-        if role in ["NONE", "SETTER", "GUESSER"]:
-            self.lobby_players[player_name] = role
-            return True
-
-        return False
-
-    @property
-    def local_player_role(self) -> str:
-        return self.lobby_players[self.local_player_name]
-
-    def is_session_ready(self) -> bool:
-        roles = list(self.lobby_players.values())
-        return ("SETTER" in roles) and ("GUESSER" in roles)

src/mind_reader/core/state.py

@@ -0,0 +1,172 @@
+from typing import Callable, Optional
+
+from wx.lib.pubsub import pub
+
+from ..network import NetworkController, NetworkMesssage
+
+OnNetworkEventCallback = Callable[[dict[str, str]], None]
+
+
+class BaseState:
+    def __init__(self, app_state: "AppState"):
+        self.app_state = app_state
+
+    def handle_network_message(self, message: NetworkMesssage) -> None:
+        pass
+
+
+class LobbyState(BaseState):
+    def __init__(self, app_state: "AppState", local_player_name: str):
+        super().__init__(app_state)
+        self.local_player_name: str = local_player_name
+        self.remote_player_name: Optional[str] = None
+        self.players: dict[str, str] = {self.local_player_name: "NONE"}
+
+    def handle_network_message(self, message: NetworkMesssage) -> None:
+        callbacks: dict[str, OnNetworkEventCallback] = {
+            "add_player": self.handle_add_player,
+            "get_player": self.handle_get_player,
+            "set_role": self.handle_set_player_role,
+            "disconnect": self.handle_disconnect,
+            "server_stop": self.handle_server_stop,
+        }
+
+        callback = callbacks.get(message.event, None)
+        if callback:
+            callback(message.payload)
+            pub.sendMessage("lobby.refresh", state=self)
+
+    def send_local_player(self) -> None:
+        player_name = self.local_player_name
+        player_role = self.players.get(player_name, "NONE")
+
+        self.app_state.network.send(
+            "add_player",
+            {
+                "player_name": player_name,
+                "player_role": player_role,
+            },
+        )
+
+    def get_remote_player(self) -> None:
+        self.app_state.network.send("get_player")
+
+    def set_player_role(self, role: str) -> bool:
+        current_role = self.players[self.local_player_name]
+        result = False
+
+        if current_role == role:
+            result = False
+
+        if role in ["NONE", "SETTER", "GUESSER"]:
+            self.players[self.local_player_name] = role
+            result = True
+
+        if result:
+            if self.app_state.network.conn:
+                self.app_state.network.send(
+                    "set_role",
+                    {
+                        "player_name": self.local_player_name,
+                        "player_role": role,
+                    },
+                )
+
+        return result
+
+    def send_client_disconnect(self) -> None:
+        self.app_state.network.send(
+            "disconnect", {"player_name": self.local_player_name}
+        )
+
+    def send_server_stop(self) -> None:
+        self.app_state.network.send("server_stop")
+
+    def handle_add_player(self, payload: dict[str, str]) -> None:
+        player_name = payload["player_name"]
+        player_role = payload["player_role"]
+        self.remote_player_name = player_name
+        self.players[self.remote_player_name] = player_role
+
+    def handle_get_player(self, payload: dict[str, str]) -> None:
+        self.send_local_player()
+
+    def handle_set_player_role(self, payload: dict[str, str]) -> None:
+        player_name = payload["player_name"]
+        player_role = payload["player_role"]
+
+        self.players[player_name] = player_role
+
+    def handle_disconnect(self, payload: dict[str, str]) -> None:
+        player_name = payload["player_name"]
+        self.players.pop(player_name)
+        self.remote_player_name = None
+
+    def handle_server_stop(self, payload: dict[str, str]) -> None:
+        pub.sendMessage("mainframe.remote_server_stop", state=self.app_state)
+
+    @property
+    def local_player_role(self) -> str:
+        return self.players[self.local_player_name]
+
+    def is_ready(self) -> bool:
+        roles = list(self.players.values())
+        return ("SETTER" in roles) and ("GUESSER" in roles)
+
+
+class SessionState(BaseState):
+    def __init__(self, app_state: "AppState"):
+        super().__init__(app_state)
+
+
+class AppState:
+    def __init__(self):
+        self.network = NetworkController(
+            self.handle_network_message,
+            self.handle_network_notice,
+        )
+
+        self.lobby: Optional[LobbyState] = None
+        self.session: Optional[SessionState] = None
+
+    def handle_network_message(self, message: NetworkMesssage) -> None:
+        if self.lobby:
+            self.lobby.handle_network_message(message)
+
+        if self.session:
+            self.session.handle_network_message(message)
+
+    def handle_network_notice(self, notice: str) -> None:
+        pub.sendMessage("mainframe.statusbar", notice=notice)
+
+    def recreate_network_controller(self) -> None:
+        self.network.stop()
+        self.network = NetworkController(
+            self.handle_network_message,
+            self.handle_network_notice,
+        )
+
+    def connect(self, player_name: str, host: str, port: int = 8994) -> bool:
+        if self.network.connect(host, port):
+            self.lobby = LobbyState(self, player_name)
+            self.lobby.send_local_player()
+            self.lobby.get_remote_player()
+            return True
+        else:
+            self.recreate_network_controller()
+            return False
+
+    def serve(self, player_name: str, host: str, port: int = 8994) -> None:
+        self.network.serve(host, port)
+        self.lobby = LobbyState(self, player_name)
+
+    def disconnect(self) -> None:
+        if self.lobby:
+            if self.network.is_server:
+                self.lobby.send_server_stop()
+            else:
+                self.lobby.send_client_disconnect()
+
+            self.lobby = None
+
+        self.recreate_network_controller()

src/mind_reader/network/protocol.py

@@ -9,7 +9,7 @@ _id_generator = itertools.count(0)
 @dataclass
 class NetworkMesssage:
     event: str
-    payload: dict = field(default_factory=dict)
+    payload: dict[str, str] = field(default_factory=dict)
     id: int = field(default_factory=lambda: next(_id_generator))
     version: str = "1.0"
 

src/mind_reader/ui/main_frame.py

@@ -1,10 +1,12 @@
-import wx
+from typing import Optional
 
-from mind_reader.core import GameState
-from mind_reader.network import NetworkController, NetworkMesssage
-from mind_reader.ui.dialogs import ClientConnectDialog, ServerStartDialog
-from mind_reader.ui.menu_bar import AppMenuBar
-from mind_reader.ui.views import LobbyPanel
+import wx
+from wx.lib.pubsub import pub
+
+from ..core import AppState
+from ..ui.dialogs import ClientConnectDialog, ServerStartDialog
+from ..ui.menu_bar import AppMenuBar
+from ..ui.views import LobbyPanel
 
 
 class MainFrame(wx.Frame):
@@ -16,12 +18,9 @@ class MainFrame(wx.Frame):
             size=wx.Size(700, 500),
             style=wx.DEFAULT_FRAME_STYLE,
         )
-        self.state: GameState | None = None
-        self.network = NetworkController(
-            self._handle_network_message,
-            self._handle_network_notice,
-        )
-        self.lobby_panel: LobbyPanel | None = None
+        self.state: AppState = AppState()
+
+        self.lobby_panel: Optional[LobbyPanel] = None
 
         self.Center()
 
@@ -32,7 +31,8 @@ class MainFrame(wx.Frame):
         self.SetMenuBar(self.menu_bar)
 
         self.CreateStatusBar(1)
-        # self.SetStatusText("Подключение отсутствует")
+        pub.subscribe(self.set_statusbar_text, "mainframe.statusbar")
+        pub.subscribe(self.remote_server_stop, "mainframe.remote_server_stop")
 
         self._bind_menu_events()
 
@@ -45,11 +45,17 @@ class MainFrame(wx.Frame):
             wx.EVT_MENU, self.on_disconnect, self.menu_bar.item_disconnect
         )
 
+    def set_statusbar_text(self, notice: str):
+        self.SetStatusText(notice)
+
+    def remote_server_stop(self, state: AppState):
+        self.on_disconnect(None)
+
     def _show_lobby(self):
         self._destroy_lobby()
 
-        if self.state:
-            self.lobby_panel = LobbyPanel(self, self.state)
+        if self.state.lobby:
+            self.lobby_panel = LobbyPanel(self, self.state.lobby)
             self.main_sizer.Add(self.lobby_panel, 1, wx.EXPAND)
             self.Layout()
 
@@ -60,51 +66,29 @@ class MainFrame(wx.Frame):
             self.lobby_panel = None
             self.Layout()
 
-    def _handle_network_message(self, m: NetworkMesssage):
-        pass
-
-    def _handle_network_notice(self, notice: str):
-        self.SetStatusText(notice)
-
-    def _recreate_network_controller(self):
-        self.network.stop()
-        self.network = NetworkController(
-            self._handle_network_message,
-            self._handle_network_notice,
-        )
-
     def on_client_connect(self, event):
         d = ClientConnectDialog(self)
+
         if d.ShowModal() == wx.ID_OK:
-            name, ip = d.get_data()
-            if self.network.connect(ip):
-                self.state = GameState(name)
-                self._show_lobby()
+            player_name, ip = d.get_data()
+            if self.state.connect(player_name, ip):
                 self.menu_bar.set_connected_state(True)
-            else:
-                self._recreate_network_controller()
-            # self.SetStatusText(f"Подключено к {ip} | Ваше имя {name}")
+                self._show_lobby()
+
         d.Destroy()
 
     def on_server_start(self, event):
         d = ServerStartDialog(self)
-        if d.ShowModal() == wx.ID_OK:
-            name, ip = d.get_data()
-            self.network.serve(ip)
-            self.state = GameState(name)
 
+        if d.ShowModal() == wx.ID_OK:
+            player_name, ip = d.get_data()
+            self.state.serve(player_name, ip)
+            self.menu_bar.set_connected_state(True)
             self._show_lobby()
 
-            self.menu_bar.set_connected_state(True)
-            # self.SetStatusText(f"Сервер запущен на {ip} | Ваше имя {name}")
         d.Destroy()
 
     def on_disconnect(self, event):
-        self._recreate_network_controller()
-
         self._destroy_lobby()
-
-        self.state = None
-
+        self.state.disconnect()
         self.menu_bar.set_connected_state(False)
-        # self.SetStatusText("Подключение отсутствует")

src/mind_reader/ui/views/lobby_view.py

@@ -1,8 +1,9 @@
-from typing import Callable, Optional, cast
+from typing import Callable, Optional
 
 import wx
+from wx.lib.pubsub import pub
 
-from mind_reader.core import GameState
+from ...core import LobbyState
 
 
 class RoleColumn(wx.Panel):
@@ -48,12 +49,13 @@ class RoleColumn(wx.Panel):
 
 
 class LobbyPanel(wx.Panel):
-    def __init__(self, parent, state: GameState):
+    def __init__(self, parent, state: LobbyState):
         super().__init__(parent)
         self.state = state
-
         self._init_ui()
 
+        self.Bind(wx.EVT_WINDOW_DESTROY, self._on_destroy)
+
     def _init_ui(self):
         main_sizer = wx.BoxSizer(wx.VERTICAL)
 
@@ -83,12 +85,12 @@ class LobbyPanel(wx.Panel):
 
         self.SetSizer(main_sizer)
 
-        self.refresh_from_state()
+        pub.subscribe(self.refresh, "lobby.refresh")
+
+        self.refresh(self.state)
 
     def _on_role_click(self, target_role: str):
-        success = self.state.set_player_role(
-            self.state.local_player_name, target_role
-        )
+        success = self.state.set_player_role(target_role)
 
         if not success:
             wx.MessageBox(
@@ -98,15 +100,19 @@ class LobbyPanel(wx.Panel):
             )
             return
 
-        self.refresh_from_state()
+        self.refresh(self.state)
 
     def _on_start_click(self, event):
         wx.MessageBox("Сессия запущена!", "Старт", wx.OK | wx.ICON_INFORMATION)
 
-    def refresh_from_state(self):
+    def _on_destroy(self, event):
+        pub.unsubscribe(self.refresh, "lobby.refresh")
+        event.Skip()
+
+    def refresh(self, state: LobbyState):
         players_by_role = {"NONE": [], "SETTER": [], "GUESSER": []}
 
-        for player, role in self.state.lobby_players.items():
+        for player, role in state.players.items():
             if role in players_by_role:
                 players_by_role[role].append(player)
 
@@ -114,9 +120,9 @@ class LobbyPanel(wx.Panel):
         self.column_setter.set_players(players_by_role["SETTER"])
         self.column_guesser.set_players(players_by_role["GUESSER"])
 
-        local_player_role = self.state.local_player_role
+        local_player_role = state.local_player_role
         self.column_none.set_role_state(local_player_role)
         self.column_setter.set_role_state(local_player_role)
         self.column_guesser.set_role_state(local_player_role)
 
-        self.button_start.Enable(self.state.is_session_ready())
+        self.button_start.Enable(state.is_ready())