home · contact · privacy
Refactor.
[plomrogue2-experiments] / server_ / game.py
index aafc7402017734031b287cf7a1c474123be4f19d..554bce6fb13f462e1e25878bbea0efaf4a02b66c 100644 (file)
@@ -3,13 +3,11 @@ sys.path.append('../')
 import game_common
 import server_.map_
 import server_.io
+import server_.tasks
+from server_.game_error import GameError
 from parser import ArgError
 
 
-class GameError(Exception):
-    pass
-
-
 class World(game_common.World):
 
     def __init__(self, game):
@@ -67,62 +65,29 @@ class World(game_common.World):
         self.things = [player, npc]
 
 
-class Task:
-
-    def __init__(self, thing, name, args=()):
-        self.name = name
-        self.thing = thing
-        self.args = args
-        self.todo = 3
-
-    def check(self):
-        if self.name == 'MOVE':
-            test_pos = self.thing.world.map_.move(self.thing.position, self.args[0])
-            if self.thing.world.map_[test_pos] != '.':
-                raise GameError(str(self.thing.id_) +
-                                ' would move into illegal terrain')
-            for t in self.thing.world.things:
-                if t.position == test_pos:
-                    raise GameError(str(self.thing.id_) +
-                                    ' would move into other thing')
+        return 'success'
 
-    def get_args_string(self):
-        stringed_args = []
-        for arg in self.args:
-            if type(arg) == 'string':
-                stringed_args += [server_.io.quote(arg)]
-            else:
-                raise GameError('stringifying arg type not implemented')
-        return ' '.join(stringed_args)
 
 
 class Thing(game_common.Thing):
 
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
-        self.task = Task(self, 'WAIT')
+        self.task = self.world.game.task_manager.get_task_class('WAIT')(self)
         self._last_task_result = None
         self._stencil = None
 
-    def task_WAIT(self):
-        return 'success'
-
-    def task_MOVE(self, direction):
-        self.position = self.world.map_.move(self.position, direction)
-        return 'success'
-    task_MOVE.argtypes = 'string:direction'
-
     def move_towards_target(self, target):
         dijkstra_map = type(self.world.map_)(self.world.map_.size)
         n_max = 256
         dijkstra_map.terrain = [n_max for i in range(dijkstra_map.size_i)]
         dijkstra_map[target] = 0
         shrunk = True
-        stencil = self.get_stencil()
+        visible_map = self.get_visible_map()
         while shrunk:
             shrunk = False
             for pos in dijkstra_map:
-                if stencil[pos] != '.':
+                if visible_map[pos] != '.':
                     continue
                 neighbors = dijkstra_map.get_neighbors(tuple(pos))
                 for direction in neighbors:
@@ -165,9 +130,9 @@ class Thing(game_common.Thing):
         #print('DEBUG result', direction)
         if target_direction:
             self.set_task('MOVE', (target_direction,))
-            #self.world.game.io.send('would move ' + direction)
 
     def decide_task(self):
+        # TODO: Check if monster can follow player too well (even when they should lose them)
         visible_things = self.get_visible_things()
         target = None
         for t in visible_things:
@@ -184,7 +149,8 @@ class Thing(game_common.Thing):
 
 
     def set_task(self, task_name, args=()):
-        self.task = Task(self, task_name, args)
+        task_class = self.world.game.task_manager.get_task_class(task_name)
+        self.task = task_class(self, args)
         self.task.check()  # will throw GameError if necessary
 
     def proceed(self, is_AI=True):
@@ -213,8 +179,7 @@ class Thing(game_common.Thing):
             return
         self.task.todo -= 1
         if self.task.todo <= 0:
-            task = getattr(self, 'task_' + self.task.name)
-            self._last_task_result = task(*self.task.args)
+            self._last_task_result = self.task.do()
             self.task = None
         if is_AI and self.task is None:
             try:
@@ -257,6 +222,7 @@ class Game(game_common.CommonCommandsMixin):
 
     def __init__(self, game_file_name):
         self.map_manager = server_.map_.map_manager
+        self.task_manager = server_.tasks.task_manager
         self.world = World(self)
         self.io = server_.io.GameIO(game_file_name, self)
         # self.pool and self.pool_result are currently only needed by the FIB
@@ -376,39 +342,39 @@ class Game(game_common.CommonCommandsMixin):
             t = self.world.get_thing(thing_id, False)
             if t is None:
                 raiseArgError('No such Thing.')
-            t.task = Task(t, task_name, args)
+            task_class = self.task_manager.get_task_class(task_name)
+            t.task = task_class(t, args)
             t.task.todo = todo
 
         def task_prefixed(command_name, task_prefix, task_command,
                           argtypes_prefix=''):
-            method = None
+            func = None
             argtypes = ''
             if command_name[:len(task_prefix)] == task_prefix:
                 task_name = command_name[len(task_prefix):]
-                task_method_candidate = 'task_' + task_name
-                if hasattr(Thing, task_method_candidate):
-                    method = partial(task_command, task_name)
-                    task_method = getattr(Thing, task_method_candidate)
-                    if hasattr(task_method, 'argtypes'):
-                        argtypes = task_method.argtypes
-            if method is not None:
-                return method, argtypes_prefix + argtypes
+                task_manager_reply = self.task_manager.get_task_class(task_name)
+                if task_manager_reply is not None:
+                    func = partial(task_command, task_name)
+                    task_class = task_manager_reply
+                    argtypes = task_class.argtypes
+            if func is not None:
+                return func, argtypes_prefix + argtypes
             return None, argtypes
 
-        method, argtypes = task_prefixed(command_name, 'TASK:', cmd_TASK_colon)
-        if method:
-            return method, argtypes
-        method, argtypes = task_prefixed(command_name, 'SET_TASK:',
+        func, argtypes = task_prefixed(command_name, 'TASK:', cmd_TASK_colon)
+        if func:
+            return func, argtypes
+        func, argtypes = task_prefixed(command_name, 'SET_TASK:',
                                          cmd_SET_TASK_colon,
-                                         'int:nonneg int:nonneg')
-        if method:
-            return method, argtypes
-        method_candidate = 'cmd_' + command_name
-        if hasattr(self, method_candidate):
-            method = getattr(self, method_candidate)
-            if hasattr(method, 'argtypes'):
-                argtypes = method.argtypes
-        return method, argtypes
+                                         'int:nonneg int:nonneg ')
+        if func:
+            return func, argtypes
+        func_candidate = 'cmd_' + command_name
+        if hasattr(self, func_candidate):
+            func = getattr(self, func_candidate)
+            if hasattr(func, 'argtypes'):
+                argtypes = func.argtypes
+        return func, argtypes
 
     def get_string_options(self, string_option_type):
         if string_option_type == 'geometry':
@@ -445,6 +411,6 @@ class Game(game_common.CommonCommandsMixin):
                 if task is not None:
                     task_args = task.get_args_string()
                     write(f, 'SET_TASK:%s %s %s %s' % (task.name, thing.id_,
-                                                    task.todo, task_args))
+                                                       task.todo, task_args))
             write(f, 'PLAYER_ID %s' % self.world.player_id)
     cmd_SAVE.dont_save = True