Add max_number limitation (with tests, nogal)

[naja.git] / naja / gameboard.py

diff --git a/naja/gameboard.py b/naja/gameboard.py
index 5e12568ded530dc8386a5a90542338f0771746b1..79e565e9a67bbbd380ef6e7f8cc41d6475fbfbe3 100644 (file)
--- a/naja/gameboard.py
+++ b/naja/gameboard.py
@@ -110,8 +110,8 @@ class GameBoard(object):
         board_locations = []
         for x in range(5):
             for y in range(5):
-                board_location = LocationCard.new_location(
-                    choice(deck['cards']).copy())
+                new_choice = cls.choose_card(deck['cards'], board_locations)
+                board_location = LocationCard.new_location(new_choice.copy())
                 board_locations.append([(x, y), board_location.export()])
         return board_locations
 
@@ -134,9 +134,35 @@ class GameBoard(object):
             self.replace_card(position)
 
     def replace_card(self, position):
-        location = LocationCard.new_location(choice(self.locations).copy())
+        new_choice = self.choose_card(self.locations,
+                                      self.board_locations.items(),
+                                      position)
+        location = LocationCard.new_location(new_choice.copy())
         self.board_locations[position] = location
 
+    @classmethod
+    def choose_card(cls, cards, board_locations, position=None):
+        # Find which cards are at their maximum and exclude them from
+        # the choice list
+        counts = {}
+        choices = {card['card_name']: card for card in cards}
+        for pos, card in board_locations:
+            if pos == position:
+                # skip the card we're replacing if appropriate
+                continue
+            if isinstance(card, LocationCard):
+                key = card.card_name
+                max_num = card.max_number
+            else:
+                key = card['card_name']
+                max_num = card.get('max_number', 25)
+            counts.setdefault(key, 0)
+            counts[key] += 1
+            if counts[key] >= max_num:
+                if key in choices:
+                    del choices[key]
+        return choice(choices.values())
+
     def shift_location_row(self, change, is_vertical):
         px, py = self.player.position
         shifted_locations = {}
@@ -181,13 +207,9 @@ class GameBoard(object):
         if px > 0:
             locations_to_rotate.append((px - 1, py))
 
-        print "rotating", direction
-
         if ROTATION[direction] == ROTATION.CLOCKWISE:
-            print "CLOCKWISE"
             new_positions = locations_to_rotate[1:] + [locations_to_rotate[0]]
         elif ROTATION[direction] == ROTATION.ANTICLOCKWISE:
-            print "ANTICLOCKWISE"
             new_positions = ([locations_to_rotate[-1]] + locations_to_rotate[:-1])
 
         for old, new in zip(locations_to_rotate, new_positions):
@@ -225,16 +247,20 @@ class LocationCard(object):
     A particular set of options available on a location.
     """
 
-    def __init__(self, bitwise_operand, location_actions):
+    def __init__(self, card_name, bitwise_operand, location_actions,
+                 max_number=25):
+        self.card_name = card_name
         self.bitwise_operand = bitwise_operand
         self.actions = location_actions
+        self.max_number = max_number
         self.check_actions()
 
     @classmethod
     def import_location(cls, state):
         location_actions = [
             cls.build_action(definition) for definition in state['actions']]
-        return cls(state['bitwise_operand'], location_actions)
+        return cls(state['card_name'], state['bitwise_operand'],
+                   location_actions, state['max_number'])
 
     @classmethod
     def build_action(cls, definition):
@@ -249,9 +275,13 @@ class LocationCard(object):
             bits = cls.parse_bits(definition['bits'])
         else:
             bits = cls.generate_bitwise_operand()
+        max_number = definition.get('max_number', 25)
+        card_name = definition['card_name']
         return cls.import_location({
             'bitwise_operand': bits,
             'actions': definition['actions'],
+            'max_number': max_number,
+            'card_name': card_name,
         })
 
     @classmethod
@@ -263,6 +293,8 @@ class LocationCard(object):
         return {
             'bitwise_operand': sorted(self.bitwise_operand),
             'actions': [action.export() for action in self.actions],
+            'max_number': self.max_number,
+            'card_name': self.card_name,
         }
 
     def check_actions(self):