X-Git-Url: https://git.ctpug.org.za/?a=blobdiff_plain;f=naja%2Fgameboard.py;fp=naja%2Fgameboard.py;h=79e565e9a67bbbd380ef6e7f8cc41d6475fbfbe3;hb=fda6095a8a402c06ca995001800de75e3de36c8c;hp=944f21fadac1ba732de90e4b797b1c5fadab8860;hpb=79dcb11340635fa1f620909106c0d9a0788e3b89;p=naja.git

diff --git a/naja/gameboard.py b/naja/gameboard.py
index 944f21f..79e565e 100644
--- 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 = {}
@@ -221,10 +247,12 @@ class LocationCard(object):
     A particular set of options available on a location.
     """
 
-    def __init__(self, card_name, 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
@@ -232,7 +260,7 @@ class LocationCard(object):
         location_actions = [
             cls.build_action(definition) for definition in state['actions']]
         return cls(state['card_name'], state['bitwise_operand'],
-                   location_actions)
+                   location_actions, state['max_number'])
 
     @classmethod
     def build_action(cls, definition):
@@ -247,10 +275,12 @@ 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,
         })
 
@@ -263,6 +293,7 @@ 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,
         }