home · contact · privacy
Refactor.
[plomrogue2-experiments] / client.py
index 2e30fe08452dc90fa51723e212a57fb80e315c43..b10ba90d22c6f7931a75ce430e16eb54d81bee58 100755 (executable)
--- a/client.py
+++ b/client.py
 #!/usr/bin/env python3
-
 import urwid
 import plom_socket_io
 import socket
 import threading
+from parser import ArgError, Parser
+from game import World
+
+
+class Thing:
+    def __init__(self, id_, position, symbol):
+        self.id_ = id_
+        self.symbol = symbol
+        self.position = position
+
+class Game:
+    world = World()
+    log_text = ''
+
+    def log(self, msg):
+        """Prefix msg plus newline to self.log_text."""
+        self.log_text = msg + '\n' + self.log_text
+
+    def cmd_THING_TYPE(self, i, type_):
+        t = self.world.get_thing(i)
+        symbol = '?'
+        if type_ == 'human':
+            symbol = '@'
+        elif type_ == 'monster':
+            symbol = 'm'
+        t.symbol = symbol
+    cmd_THING_TYPE.argtypes = 'int:nonneg string'
+
+    def cmd_THING_POS(self, i, yx):
+        t = self.world.get_thing(i)
+        t.position = list(yx)
+    cmd_THING_POS.argtypes = 'int:nonneg yx_tuple:nonneg'
+
+    def cmd_THING_POS(self, i, yx):
+        t = self.world.get_thing(i)
+        t.position = list(yx)
+    cmd_THING_POS.argtypes = 'int:nonneg yx_tuple:nonneg'
+
+    def cmd_MAP_SIZE(self, yx):
+        """Set self.map_size to yx, redraw self.terrain_map as '?' cells."""
+        self.world.set_map_size(yx)
+    cmd_MAP_SIZE.argtypes = 'yx_tuple:nonneg'
+
+    def cmd_TURN_FINISHED(self, n):
+        """Do nothing. (This may be extended later.)"""
+        pass
+    cmd_TURN_FINISHED.argtypes = 'int:nonneg'
+
+    def cmd_NEW_TURN(self, n):
+        """Set self.turn to n, empty self.things."""
+        self.world.turn = n
+        self.world.things = []
+    cmd_NEW_TURN.argtypes = 'int:nonneg'
+
+    def cmd_TERRAIN_LINE(self, y, terrain_line):
+        self.world.set_map_line(y, terrain_line)
+    cmd_TERRAIN_LINE.argtypes = 'int:nonneg string'
+
+
+class WidgetManager:
+
+    def __init__(self, socket, game):
+        """Set up all urwid widgets we want on the screen."""
+        self.game = game
+        edit_widget = self.EditToSocketWidget(socket, 'SEND: ')
+        self.map_widget = urwid.Text('', wrap='clip')
+        self.turn_widget = urwid.Text('')
+        self.log_widget = urwid.Text('')
+        map_box = urwid.Padding(self.map_widget, width=50)
+        widget_pile = urwid.Pile([edit_widget, map_box, self.turn_widget,
+                                  self.log_widget])
+        self.top = urwid.Filler(widget_pile, valign='top')
+
+    def draw_map(self):
+        """Draw map view from .game.terrain_map, .game.things."""
+        map_lines = []
+        map_size = len(self.game.world.terrain_map)
+        start_cut = 0
+        while start_cut < map_size:
+            limit = start_cut + self.game.world.map_size[1]
+            map_lines += [self.game.world.terrain_map[start_cut:limit]]
+            start_cut = limit
+        for t in self.game.world.things:
+            line_as_list = list(map_lines[t.position[0]])
+            line_as_list[t.position[1]] = t.symbol
+            map_lines[t.position[0]] = ''.join(line_as_list)
+        return "\n".join(map_lines)
+
+    def update(self):
+        """Redraw all non-edit widgets."""
+        self.turn_widget.set_text('TURN: ' + str(self.game.world.turn))
+        self.log_widget.set_text(self.game.log_text)
+        self.map_widget.set_text(self.draw_map())
 
+    class EditToSocketWidget(urwid.Edit):
+        """Extends urwid.Edit with socket to send input on 'enter' to."""
 
-class UrwidSetup:
+        def __init__(self, socket, *args, **kwargs):
+            super().__init__(*args, **kwargs)
+            self.socket = socket
 
-    def __init__(self, socket):
+        def keypress(self, size, key):
+            """Extend super(): on Enter, send .edit_text, and empty it."""
+            if key != 'enter':
+                return super().keypress(size, key)
+            plom_socket_io.send(self.socket, self.edit_text)
+            self.edit_text = ''
+
+
+class PlomRogueClient:
+
+    def __init__(self, game, socket):
         """Build client urwid interface around socket communication.
 
         Sets up all widgets for writing to the socket and representing data
-        from it. Sending via a self.EditToSocket widget is straightforward;
-        polling the socket for input from the server in parallel to the urwid
-        main loop not so much:
+        from it. Sending via a WidgetManager.EditToSocket widget is
+        straightforward; polling the socket for input from the server in
+        parallel to the urwid main loop not so much:
 
         The urwid developers warn against sharing urwid resources among
         threads, so having a socket polling thread for writing to an urwid
@@ -25,137 +131,51 @@ class UrwidSetup:
         write socket.recv output to an object that is then linked to by
         self.server_output (which is known to the urwid thread), then use the
         pipe to urwid to trigger it pulling new data from self.server_output to
-        handle via self.InputHandler. (We *could* pipe socket.recv output
+        handle via self.handle_input. (We *could* pipe socket.recv output
         directly, but then we get complicated buffering situations here as well
         as in the urwid code that receives the pipe output. It's easier to just
         tell the urwid code where it finds full new server messages to handle.)
         """
+        self.game = game
+        self.parser = Parser(self.game)
         self.socket = socket
-        self.main_loop = urwid.MainLoop(self.setup_widgets())
+        self.widget_manager = WidgetManager(self.socket, self.game)
         self.server_output = []
-        input_handler = getattr(self.InputHandler(self.reply_widget,
-                                                  self.map_widget,
-                                                  self.server_output),
-                                'handle_input')
-        self.urwid_pipe_write_fd = self.main_loop.watch_pipe(input_handler)
+        self.urwid_loop = urwid.MainLoop(self.widget_manager.top)
+        self.urwid_pipe_write_fd = self.urwid_loop.watch_pipe(self.
+                                                              handle_input)
         self.recv_loop_thread = threading.Thread(target=self.recv_loop)
 
-    def setup_widgets(self):
-        """Return container widget with all widgets we want on our screen.
+    def handle_input(self, trigger):
+        """On input from recv_loop thread, parse and enact commands.
 
-        Sets up an urwid.Pile inside a returned urwid.Filler; top to bottom:
-        - an EditToSocketWidget, prefixing self.socket input with 'SEND: '
-        - a 50-col wide urwid.Padding container for self.map_widget, which is
-          to print clipped map representations
-        - self.reply_widget, a urwid.Text widget printing self.socket replies
-        """
-        edit_widget = self.EditToSocketWidget(self.socket, 'SEND: ')
-        self.reply_widget = urwid.Text('')
-        self.map_widget = self.MapWidget('', wrap='clip')
-        map_box = urwid.Padding(self.map_widget, width=50)
-        widget_pile = urwid.Pile([edit_widget, map_box, self.reply_widget])
-        return urwid.Filler(widget_pile, valign='top')
-
-    class EditToSocketWidget(urwid.Edit):
-        """Extends urwid.Edit with socket to send input on 'enter' to."""
-
-        def __init__(self, socket, *args, **kwargs):
-            super().__init__(*args, **kwargs)
-            self.socket = socket
+        Serves as a receiver to urwid's watch_pipe mechanism, with trigger the
+        data that a pipe defined by watch_pipe delivers. To avoid buffering
+        trouble, we don't care for that data beyond the fact that its receival
+        triggers this function: The sender is to write the data it wants to
+        deliver into the container referenced by self.server_output, and just
+        pipe the trigger to inform us about this.
 
-        def keypress(self, size, key):
-            """Extend super(): on Enter, send .edit_text, and empty it."""
-            if key != 'enter':
-                return super().keypress(size, key)
-            plom_socket_io.send(self.socket, self.edit_text)
-            self.edit_text = ''
-
-    class MapWidget(urwid.Text):
-        """Stores/updates/draws game map."""
-        terrain_map = ' ' * 25
-        position = [0, 0]
-
-        def draw_map(self):
-            """Draw map view from .terrain_map, .position."""
-            whole_map = []
-            for c in self.terrain_map:
-                whole_map += [c]
-            pos_i = self.position[0] * (5 + 1) + self.position[1]
-            whole_map[pos_i] = '@'
-            self.set_text(''.join(whole_map))
-
-        def update_terrain(self, terrain_map):
-            """Update self.terrain_map."""
-            self.terrain_map = terrain_map
-            self.draw_map()
-
-        def update_position_y(self, position_y_string):
-            """Update self.position[0]."""
-            self.position[0] = int(position_y_string)
-            self.draw_map()
-
-        def update_position_x(self, position_x_string):
-            """Update self.position[1]."""
-            self.position[1] = int(position_x_string)
-            self.draw_map()
-
-    class InputHandler:
-        """Delivers data from other thread to widget via message_container.
-
-        The class only exists to provide handle_input as a bound method, with
-        widget and message_container pre-set, as (bound) handle_input is used
-        as a callback in urwid's watch_pipe – which merely provides its
-        callback target with one parameter for a pipe to read data from an
-        urwid-external thread.
+        If the message delivered is 'BYE', quits Urwid. Otherwise tries to
+        parse it as a command, and enact it. In all cases but the 'BYE', calls
+        self.widget_manager.update.
         """
-
-        def __init__(self, widget1, widget2, message_container):
-            self.widget1 = widget1
-            self.widget2 = widget2
-            self.message_container = message_container
-
-        def handle_input(self, trigger):
-            """On input from other thread, either quit or write to widget text.
-
-            Serves as a receiver to urwid's watch_pipe mechanism, with trigger
-            the data that a pipe defined by watch_pipe delivers. To avoid
-            buffering trouble, we don't care for that data beyond the fact that
-            its receival triggers this function: The sender is to write the
-            data it wants to deliver into the container referenced by
-            self.message_container, and just pipe the trigger to inform us
-            about this.
-
-            If the message delivered is 'BYE', quits Urwid.
-            """
-
-            def mapdraw_command(prefix, func):
-                n = len(prefix)
-                if len(msg) > n and msg[:n] == prefix:
-                    m = getattr(self.widget2, func)
-                    m(msg[n:])
-                    return True
-                return False
-
-            msg = self.message_container[0]
-            if msg == 'BYE':
-                raise urwid.ExitMainLoop()
-                return
-            found_command = False
-            try:
-                found_command = (
-                    mapdraw_command('TERRAIN\n', 'update_terrain') or
-                    mapdraw_command('POSITION_Y ', 'update_position_y') or
-                    mapdraw_command('POSITION_X ', 'update_position_x'))
-            except Exception as e:
-                self.widget1.set_text('TROUBLESOME ARGUMENT: ' + msg + '\n' +
-                                      str(e))
+        msg = self.server_output[0]
+        if msg == 'BYE':
+            raise urwid.ExitMainLoop()
+        try:
+            command = self.parser.parse(msg)
+            if command is None:
+                self.game.log('UNHANDLED INPUT: ' + msg)
             else:
-                if not found_command:
-                    self.widget1.set_text('UNKNOWN COMMAND: ' + msg)
-            del self.message_container[0]
+                command()
+        except ArgError as e:
+            self.game.log('ARGUMENT ERROR: ' + msg + '\n' + str(e))
+        self.widget_manager.update()
+        del self.server_output[0]
 
     def recv_loop(self):
-        """Loop to receive messages from socket and deliver them to urwid.
+        """Loop to receive messages from socket, deliver them to urwid thread.
 
         Waits for self.server_output to become empty (this signals that the
         input handler is finished / ready to receive new input), then writes
@@ -164,19 +184,21 @@ class UrwidSetup:
         """
         import os
         for msg in plom_socket_io.recv(self.socket):
-            while len(self.server_output) > 0:
-                pass
+            while len(self.server_output) > 0:  # Wait until self.server_output
+                pass                            # is emptied by input handler.
             self.server_output += [msg]
             os.write(self.urwid_pipe_write_fd, b' ')
 
     def run(self):
-        """Run in parallel main and recv_loop thread."""
+        """Run in parallel urwid_loop and recv_loop threads."""
         self.recv_loop_thread.start()
-        self.main_loop.run()
+        self.urwid_loop.run()
         self.recv_loop_thread.join()
 
 
-s = socket.create_connection(('127.0.0.1', 5000))
-u = UrwidSetup(s)
-u.run()
-s.close()
+if __name__ == '__main__':
+    game = Game()
+    s = socket.create_connection(('127.0.0.1', 5000))
+    p = PlomRogueClient(game, s)
+    p.run()
+    s.close()