Mercurial > parpg-source

comparison components/container.py @ 24:eab8af30dfc7

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
get_free_slot no raises an exception instead of appending a slot to the container.
author KarstenBock@gmx.net
date Fri, 02 Sep 2011 12:54:43 +0200
parents d8c17766895b
children ff3e395abf91
comparison
equal deleted inserted replaced
23:979fffc5af49 24:eab8af30dfc7
30 self.bulk = bulk 30 self.bulk = bulk
31 self.max_bulk = max_bulk 31 self.max_bulk = max_bulk
32 32
33 def __str__(self): 33 def __str__(self):
34 return "Item would exceed the bulk limit of the container." 34 return "Item would exceed the bulk limit of the container."
35 35
36 class NoFreeSlotError(Exception):
37 """Error that gets raised when the container has no free slots."""
38
39 def __str__(self):
40 return "Container can't hold any more items."
41
36 def get_free_slot(container): 42 def get_free_slot(container):
37 """Returns the first slot of the container that is not occupied.""" 43 """Returns the first slot of the container that is not occupied."""
38 index = 0 44 index = 0
39 for child in container.children: 45 for child in container.children:
40 if not child: 46 if not child:
41 return index 47 return index
42 index += 1 48 index += 1
43 container.children.append(None) 49 raise NoFreeSlotError
44 return index
45 50
46 def get_total_bulk(container): 51 def get_total_bulk(container):
47 """Returns the bulk of all items in the container.""" 52 """Returns the bulk of all items in the container."""
48 total_bulk = 0 53 total_bulk = 0
49 for child in container.children: 54 for child in container.children: