diff options
author | Alan Aguiar <alanjas@hotmail.com> | 2013-12-06 20:03:25 (GMT) |
---|---|---|
committer | Walter Bender <walter@sugarlabs.org> | 2013-12-06 20:03:25 (GMT) |
commit | d93e65399eaaaaeec1f2d7e3615eab86ee292772 (patch) | |
tree | 086e07ec3a3beae3f7e1fcbd917941c73a7996c3 /TurtleArt/sprites.py | |
parent | cea68c2743ebfc239f9bc56de9d36ee3c9d4d50c (diff) |
Make sure spr is not None before accessing
Diffstat (limited to 'TurtleArt/sprites.py')
-rw-r--r-- | TurtleArt/sprites.py | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/TurtleArt/sprites.py b/TurtleArt/sprites.py index 7483c12..3f5d7df 100644 --- a/TurtleArt/sprites.py +++ b/TurtleArt/sprites.py @@ -106,7 +106,7 @@ class Sprites: def length_of_list(self): ''' How many sprites are there? ''' - return(len(self.list)) + return len(self.list) def append_to_list(self, spr): ''' Append a new sprite to the end of the list. ''' @@ -122,9 +122,7 @@ class Sprites: self.list.insert(i, spr) def find_in_list(self, spr): - if spr in self.list: - return True - return False + return (spr in self.list) def remove_from_list(self, spr): ''' Remove a sprite from the list. ''' @@ -258,7 +256,8 @@ class Sprite: if layer is not None: self.layer = layer for i in range(self._sprites.length_of_list()): - if self.layer < self._sprites.get_sprite(i).layer: + spr = self._sprites.get_sprite(i) + if spr is not None and self.layer < spr.layer: self._sprites.insert_in_list(self, i) self.inval() return |