Add Save-Photo Button,

Extend README.md with pictures,
reformat code, optimize imports

FieldObjects/Items/ItemBase.py

@@ -1,14 +1,14 @@
-
 from Utils.Const import *
 
 
 class ItemBase:
     """
     This class represents the base class of all items that may appear on the screen. It does not have any effect.
     """
+
     def __init__(self, controller, gamescreen):
         """
-        Creates an base Item.
+        Creates a base Item.
 
         :param controller: Controller instance of the game screen.
         :param gamescreen: The gamescreen instance where this item is placed in.
@@ -19,7 +19,7 @@ def __init__(self, controller, gamescreen):
 
         # This value should be manually adjusted depending on the respective Item that inherits this class.
         self.duration = DURATION_NORMAL
-        self.image_path = "Data/DefaultIcon.png"  # Relative path to item icon. (from __main__, 20 x 20 pixels, .png)
+        self.image_path = "Data/DefaultIcon.png"
 
         self.player = None  # Will be defined during method call activate.
 

FieldObjects/Items/ItemBigAll.py

@@ -1,4 +1,3 @@
-
 from FieldObjects.Items.ItemBase import ItemBase
 from Utils.Const import *
 
@@ -7,11 +6,11 @@ class ItemBigAll(ItemBase):
     """
     Increases the size of every player but the collecting player by one step.
     """
+
     # __init__ from base class.
     def __init__(self, controller, gamescreen):
         ItemBase.__init__(self, controller, gamescreen)
         self.duration = DURATION_NORMAL
-        # Relative path to item icon. (from __main__, 20 x 20 pixels, .png)
         self.image_path = "Data/IconBigAll.png"
         self.players_to_ignore = []  # Stores all players to ignore by this item
 
@@ -43,4 +42,3 @@ def deactivate(self):
             else:
                 # Choose next bigger value from the SPEEDS list
                 p.size = SIZES[max(SIZES.index(p.size) - 1, 0)]
-

FieldObjects/Items/ItemBlock.py

@@ -1,5 +1,5 @@
-
 from numpy.random import default_rng
+
 from FieldObjects.Items.ItemBase import ItemBase
 from Utils.Const import *
 
@@ -8,11 +8,11 @@ class ItemBlock(ItemBase):
     """
     Places a Block randomly on the field.
     """
+
     # __init__ from base class.
     def __init__(self, controller, gamescreen):
         ItemBase.__init__(self, controller, gamescreen)
         self.duration = DURATION_INFINITE  # No effective usage
-        # Relative path to item icon. (from __main__, 20 x 20 pixels, .png)
         self.image_path = "Data/IconBlock.png"
         self.rng = default_rng()
         self.blocksize = 100
@@ -32,7 +32,7 @@ def activate(self, player):
                                         to=(x_rand, y_rand, x_rand + self.blocksize, y_rand + self.blocksize)
                                         )
         # Effectively
-        self.gamescreen.walls[x_rand:x_rand+self.blocksize, y_rand:y_rand+self.blocksize] = -1
+        self.gamescreen.walls[x_rand:x_rand + self.blocksize, y_rand:y_rand + self.blocksize] = -1
 
     def deactivate(self):
         """

FieldObjects/Items/ItemClear.py

@@ -1,4 +1,3 @@
-
 from FieldObjects.Items.ItemBase import ItemBase
 from Utils.Const import *
 
@@ -7,11 +6,11 @@ class ItemClear(ItemBase):
     """
     Clears all walls within the game field.
     """
+
     # __init__ from base class.
     def __init__(self, controller, gamescreen):
         ItemBase.__init__(self, controller, gamescreen)
         self.duration = DURATION_INSTANT
-        # Relative path to item icon. (from __main__, 20 x 20 pixels, .png)
         self.image_path = "Data/IconClear.png"
 
     def activate(self, player):

FieldObjects/Items/ItemFastAll.py

@@ -1,4 +1,3 @@
-
 from FieldObjects.Items.ItemBase import ItemBase
 from Utils.Const import *
 
@@ -7,11 +6,11 @@ class ItemFastAll(ItemBase):
     """
     Increases the speed of every player but the collecting player by one step if possible.
     """
+
     # __init__ from base class.
     def __init__(self, controller, gamescreen):
         ItemBase.__init__(self, controller, gamescreen)
         self.duration = DURATION_NORMAL
-        # Relative path to item icon. (from __main__, 20 x 20 pixels, .png)
         self.image_path = "Data/IconFastAll.png"
         self.players_to_ignore = []  # Stores all players having max speed at the time of collecting this item.
 

FieldObjects/Items/ItemFastSelf.py

@@ -1,4 +1,3 @@
-
 from FieldObjects.Items.ItemBase import ItemBase
 from Utils.Const import *
 
@@ -7,11 +6,11 @@ class ItemFastSelf(ItemBase):
     """
     Increases the speed of the collecting player by one step if possible.
     """
+
     # __init__ from base class.
     def __init__(self, controller, gamescreen):
         ItemBase.__init__(self, controller, gamescreen)
         self.duration = DURATION_SHORT
-        # Relative path to item icon. (from __main__, 20 x 20 pixels, .png)
         self.image_path = "Data/IconFastSelf.png"
         self.was_at_max_speed = False
 

FieldObjects/Items/ItemFly.py

@@ -1,4 +1,3 @@
-
 from FieldObjects.Items.ItemBase import ItemBase
 from Utils.Const import *
 
@@ -8,11 +7,11 @@ class ItemFly(ItemBase):
     Makes the player flying and being able to cross any walls while not making a wall by himself during the time
     of this item being active.
     """
+
     # __init__ from base class.
     def __init__(self, controller, gamescreen):
         ItemBase.__init__(self, controller, gamescreen)
         self.duration = DURATION_SHORT
-        # Relative path to item icon. (from __main__, 20 x 20 pixels, .png)
         self.image_path = "Data/IconFly.png"
 
     def activate(self, player):

FieldObjects/Items/ItemGlueAll.py

@@ -1,4 +1,3 @@
-
 from FieldObjects.Items.ItemBase import ItemBase
 from Utils.Const import *
 
@@ -7,15 +6,13 @@ class ItemGlueAll(ItemBase):
     """
     Makes all players sbut the collecting player turn slower.
     """
+
     # __init__ from base class.
     def __init__(self, controller, gamescreen):
         ItemBase.__init__(self, controller, gamescreen)
         self.duration = DURATION_SHORT
-        # Relative path to item icon. (from __main__, 20 x 20 pixels, .png)
         self.image_path = "Data/IconGlueAll.png"
-
         self.player = None  # Will be defined during method call activate.
-
         self.players_to_ignore = []
 
     def activate(self, player):

FieldObjects/Items/ItemJump.py

@@ -1,4 +1,3 @@
-
 from FieldObjects.Items.ItemBase import ItemBase
 from Utils.Const import *
 
@@ -7,15 +6,13 @@ class ItemJump(ItemBase):
     """
     Moves the players 30 times the players speed pixels in forward direction.
     """
+
     # __init__ from base class.
     def __init__(self, controller, gamescreen):
         ItemBase.__init__(self, controller, gamescreen)
         self.duration = DURATION_ONETIME
-        # Relative path to item icon. (from __main__, 20 x 20 pixels, .png)
         self.image_path = "Data/IconJump.png"
-
         self.player = None  # Will be defined during method call activate.
-
         self.previous_speed = None
 
     def activate(self, player):

FieldObjects/Items/ItemPackage.py

@@ -1,5 +1,5 @@
-
 from numpy.random import default_rng
+
 from FieldObjects.Items.ItemBase import ItemBase
 from Utils.Const import *
 
@@ -8,11 +8,11 @@ class ItemPackage(ItemBase):
     """
     Places three random items on the game field.
     """
+
     # __init__ from base class.
     def __init__(self, controller, gamescreen):
         ItemBase.__init__(self, controller, gamescreen)
         self.duration = DURATION_INSTANT
-        # Relative path to item icon. (from __main__, 20 x 20 pixels, .png)
         self.image_path = "Data/IconPackage.png"
         self.rng = default_rng()
 

FieldObjects/Items/ItemRandom.py

@@ -1,23 +1,24 @@
+from importlib import import_module
 
 from numpy.random import default_rng
+
 from FieldObjects.Items.ItemBase import ItemBase
 from Utils.Const import *
-from importlib import import_module
 
 
 class ItemRandom(ItemBase):
     """
     Activates a random item effect.
     """
+
     # __init__ from base class.
     def __init__(self, controller, gamescreen):
         ItemBase.__init__(self, controller, gamescreen)
         self.duration = DURATION_INSTANT
-        # Relative path to item icon. (from __main__, 20 x 20 pixels, .png)
         self.image_path = "Data/IconRandom.png"
         self.rng = default_rng()
-
         self.available_item_names = self.controller.all_items
+
         if self.available_item_names.__contains__("ItemRandom"):
             self.available_item_names.remove("ItemRandom")
 
@@ -31,7 +32,7 @@ def activate(self, player):
         self.player = player
         # emulate a random item
         item_to_create = self.rng.choice(self.available_item_names)
-        item_module = import_module("FieldObjects.Items."+item_to_create)
+        item_module = import_module("FieldObjects.Items." + item_to_create)
         item_class = getattr(item_module, item_to_create)
         emulated_item = item_class(self.controller, self.gamescreen)
 

FieldObjects/Items/ItemRemoveBorder.py

@@ -1,4 +1,3 @@
-
 from FieldObjects.Items.ItemBase import ItemBase
 from Utils.Const import *
 
@@ -7,11 +6,12 @@ class ItemRemoveBorder(ItemBase):
     """
     Toggles the game screen borders off.
     """
+
     # __init__ from base class.
     def __init__(self, controller, gamescreen):
         ItemBase.__init__(self, controller, gamescreen)
         self.duration = DURATION_NORMAL
-        self.image_path = "Data/IconRemoveBorder.png"  # Relative path to item icon. (from __main__, 20 x 20 pixels, .png)
+        self.image_path = "Data/IconRemoveBorder.png"
 
     def activate(self, player):
         """

FieldObjects/Items/ItemSlickSelf.py

@@ -1,4 +1,3 @@
-
 from FieldObjects.Items.ItemBase import ItemBase
 from Utils.Const import *
 
@@ -7,16 +6,13 @@ class ItemSlickSelf:
     """
     Makes the collecting player turn faster.
     """
+
     # __init__ from base class.
     def __init__(self, controller, gamescreen):
         ItemBase.__init__(self, controller, gamescreen)
-
         self.duration = DURATION_LONG
-        # Relative path to item icon. (from __main__, 20 x 20 pixels, .png)
         self.image_path = "Data/IconSlickSelf.png"
-
         self.player = None  # Will be defined during method call activate.
-
         self.ignore = None
 
     def activate(self, player):

FieldObjects/Items/ItemSlowAll.py

@@ -1,4 +1,3 @@
-
 from FieldObjects.Items.ItemBase import ItemBase
 from Utils.Const import *
 
@@ -7,11 +6,11 @@ class ItemSlowAll(ItemBase):
     """
     Decrease the speed of every player but the collecting player by one step.
     """
+
     # __init__ from base class.
     def __init__(self, controller, gamescreen):
         ItemBase.__init__(self, controller, gamescreen)
         self.duration = DURATION_NORMAL
-        # Relative path to item icon. (from __main__, 20 x 20 pixels, .png)
         self.image_path = "Data/IconSlowAll.png"
         self.players_to_ignore = []  # Stores all players to ignore by this item
 
@@ -43,4 +42,3 @@ def deactivate(self):
             else:
                 # Choose next bigger value from the SPEEDS list
                 p.speed = SPEEDS[min(SPEEDS.index(p.speed) + 1, len(SPEEDS) - 1)]
-

FieldObjects/Items/ItemSlowSelf.py

@@ -1,4 +1,3 @@
-
 from FieldObjects.Items.ItemBase import ItemBase
 from Utils.Const import *
 
@@ -7,11 +6,11 @@ class ItemSlowSelf(ItemBase):
     """
     Decreases the speed of the collecting player by one step.
     """
+
     # __init__ from base class.
     def __init__(self, controller, gamescreen):
         ItemBase.__init__(self, controller, gamescreen)
         self.duration = DURATION_SHORT
-        # Relative path to item icon. (from __main__, 20 x 20 pixels, .png)
         self.image_path = "Data/IconSlowSelf.png"
         self.ignore = False  # Indicates if this Items effect should be ignored
 
@@ -40,4 +39,3 @@ def deactivate(self):
             pass
         else:
             self.player.speed = SPEEDS[min(SPEEDS.index(self.player.speed) + 1, len(SPEEDS) - 1)]
-

FieldObjects/Items/ItemSmallSelf.py

@@ -1,4 +1,3 @@
-
 from FieldObjects.Items.ItemBase import ItemBase
 from Utils.Const import *
 
@@ -7,11 +6,11 @@ class ItemSmallSelf(ItemBase):
     """
     Increases the size of every player but the collecting player by one step.
     """
+
     # __init__ from base class.
     def __init__(self, controller, gamescreen):
         ItemBase.__init__(self, controller, gamescreen)
         self.duration = DURATION_NORMAL
-        # Relative path to item icon. (from __main__, 20 x 20 pixels, .png)
         self.image_path = "Data/IconSmallSelf.png"
         self.was_at_min_size = False
 

FieldObjects/Items/ItemZiggZaggAll.py

@@ -1,4 +1,3 @@
-
 from FieldObjects.Items.ItemBase import ItemBase
 from Utils.Const import *
 
@@ -8,6 +7,7 @@ class ItemZiggZaggAll(ItemBase):
     sets the collecting everybody's but the collecting players turn rate to RATE_RIGHT_ANGLE.
     Every turn will then be a 90 degree turn.
     """
+
     # __init__ from base class.
     def __init__(self, controller, gamescreen):
         ItemBase.__init__(self, controller, gamescreen)

FieldObjects/Items/ItemZiggZaggSelf.py

@@ -1,4 +1,3 @@
-
 from FieldObjects.Items.ItemBase import ItemBase
 from Utils.Const import *
 
@@ -7,6 +6,7 @@ class ItemZiggZaggSelf(ItemBase):
     """
     Sets the collecting players turn rate to RATE_RIGHT_ANGLE. every turn will then be a 90 degree turn.
     """
+
     # __init__ from base class.
     def __init__(self, controller, gamescreen):
         ItemBase.__init__(self, controller, gamescreen)