Uwagi

[MarcinCiura]

1. Proszę wymienić biblioteki zewnętrzne w pliku requirements.txt (składnię sobie Pan znajdzie). Dzięki temu użytkownik będzie mógł je zainstalować przez pip install -r requirements.txt

2. Radzę zainstalować pylint i przepuścić przez niego kod. Dzięki temu odpadnie mi konieczność wypisywania uwag typu: ustawić importy w innej kolejności, powstawiać puste wiersze tu i ówdzie itp.

3. Stałe m i n nie mogą się tak nazywać. Raz, że nie wiadomo, co znaczą, dwa że powinny być dużymi literami, jak inne Pana stałe.

4. Drobiazg: w Pythonie 3 nie trzeba dziedziczyć z object. Wystarczy class Game itp.

5. Konstruktor Game.__init__() nie jest zgodny z regułami sztuki. Pierwsze 9 wierszy (oprócz pygame.init()) jest OK, ale Pana pętla while powinna należeć do metody np. Game.run(). Pod koniec pliku run.py dodać:

   
   def main():
   pygame.init()
   game = Game()
   game.run()

if name == 'main': main()

6. Z magicznych stałych `(450, 285)` itp. zrobić nazwane stałe.

7. Python to nie Java. Nie ma potrzeby robić osobnych plików na klasy tej wielkości, co `InfoScreen` itp.

8. Przydałyby się docstringi do modułów, klas i metod.

9. Drobiazg: w `GameScreen.block_on_click()` zamiast używać `\` do kontynuacji wierszy, wystarczy cały warunek ująć w dodatkowe nawiasy.

[MarcinCiura]

1. Ze (100, 100, 100) i 285 zrobić nazwane stałe.

2. W run.py dodać pusty wiersz nad stałymi i nad klasą InfoScreen. To samo w game_screen.py.

3. Kolory w block.py. :) Proszę o nich przeczytać w 3. rozdziale Poradnika.