piorus / python-mastermind-game-opengl Goto Github PK

1. Chyba nie są potrzebne naraz requirements.txt i Pipfile z przyległościami. Tutaj mam Pythona 3.6.9, więc według Pańskiego Pipfile nie mam szans na uruchomienie programu — czy to prawda?
2. Jeśli Pański pylintrc czymś się różni od domyślnego, warto o tym napisać w komentarzu na samej górze. Jeśli się nie różni, napisać, że się nie różni.
3. To jest nieładne. Czy from src import app nie działa? Jeśli działa, to powyrzucać hack ze wszystkich plików.
4. To się pisze from OpenGL import GL.
5. To wyrzucić, a na końcu dopisać

if __name__ == '__main__':
    pygame.init()
    unittest.main()

6. Drobiazg: lepiej TestState.
7. Zamiast żmudnej listy zakupów z modułu lepiej importować moduł (2. rozdział).
8. from game import gui.
9. To do main() (3. i 4. rozdział). Podobnie z oknem, które jako self.window jest do niczego nie potrzebne, prawda?
10. Bez skrótów proszę (2. rozdział). Nazwać stałe DEFAULT_POSITION, DEFAULT_UP, DEFAULT_FRONT.
11. To mi się nie podoba. Czy nie lepiej mieć stałe np. ALL_DIRECTIONS = FORWARD, ..., RIGHT = range(4) i np. 4-elementową listę self.moving?
12. Tutaj użyć ww. stałej ALL_DIRECTIONS.
13. Czy to sprawdzenie jest konieczne (4. rozdział)? Niektóre obiekty wychodzą ułomne.
14. Szukanie igły w .keys() ma czas liniowy. if igła in stóg_dict: — czas stały.
15. Może tutaj spodoba się Panu collections.defaultdict(list).
16. Tutaj lepiej bez obciachu (koniec 2. rozdziału).
17. Zamiast dodawania, przy którym można się pomylić, użyłbym np. range(pygame.USEREVENT, pygame.USEREVENT + 8).
18. Takie puste punkty lepiej pomijać. :return: None pomijać zawsze.
19. Dlaczego x i y są liczone na odwrót? Jeśli kryje się za tym głębsza myśl, dodać komentarz.
20. Nie rozumiem: wg opisu color[3] powoduje IndexError. Jeśli chodzi o kanał alfa, to poprawić opis i skomentować magiczne 3 lub nawet zrobić z niego stałą.
21. Zrobić z tego koloru stałą. Nadto przeczytać moje narzekania na kolory w 3. rozdziale.
22. Nie rozumiem tych nazw.
23. Niezbyt mi się podoba tutaj robienie ciężkiej pracy w konstruktorze. Czy nie można tego odłożyć na potem?
24. Korzystanie z self.text do komunikacji z self.load() też mi się nie podoba. Czy można by go przekazywać jako parametr?
25. Zamiast komentarzy, czy zadziała tutaj target=..., level=0,...?
26. Jaki super()?
27. Rozwinąć INV (2. rozdział).
28. Ponazywać, poczytać o kolorach w 3. rozdziale.
29. Wystarczy i. Argumenty test.Text ponazywać. To samo niżej.
30. Z 4 i 12 zrobić stałe.
31. Tak jest nieładnie. Wyciągnąć treść __init__() do osobnej metody i wywoływać ją z dwóch miejsc.
32. Zadanie z gwiazdką: trzecia klasa do grania. Sama nie wie, jaki dokładnie ma kod — zamiast tego ma listę kodów niesprzecznych ze swoimi dotychczasowymi odpowiedziami. Na początku na liście są wszystkie możliwe kody. Potem w każdej turze jedzie po tej liście, dla każdego elementu znajduje odpowiedź na bieżącą próbę człowieka i powiększa w collections.Counter liczniki odpowiadające tym odpowiedziom. Po przejechaniu listy wybiera odpowiedź z maksymalnym licznikiem, znowu przejeżdża listę, żeby ją zawęzić zgodnie ze swoją odpowiedzią i zastępuje starą listę przez nową.

Stworzenie projektu w OpenGL przy użyciu pyOpenGL