Mam tutaj konsolowy program do obsługi wypożyczeń rowerów.
Proszę o krytykę i wskazówki, co poprawić, żeby zwiększyć czytelność i poprawność kodu. Dla ludzi o mocnych nerwach:)
https://4programmers.net/Pastebin/7424 - python
https://4programmers.net/Pastebin/7425 - sql
0

- Rejestracja:prawie 10 lat
- Ostatnio:12 miesięcy
- Postów:4700
0
- Zastanawiam się po co ta klasa?! Jeżeli chciałeś się popisać znajomością paradygmatu, to sorry ale nie wyszło. Jeżeli już korzystasz z klas to rób to z głową, a nie wszystko pakujesz do jednej. To nie Delphi.
- Ciekawi mnie również te "rekurencyjne" wywołanie metody logowanie(). Czy nie można tego menu jak człowiek walnąć w pętli while tylko 100 razy pisać self.logowanie()?
- Nie podobają mi się te zapytania SQL. Czy nie fajniej byłoby wykorzystać jakiś ORM?
- Nie podoba mi się mieszanie zapytań do bazy z interfejsem użytkownika. Ładniej byłoby gdyby kod był podzielony na warstwy. Przemyśl implementację wzorców MVP, Repository itd.
- Gdyby ta wypożyczalnia wypaliła i zostałbyś rowerowym gigantem, który wypożycza rowery w Polsce i na świecie, a w konsekwencji zamiast odwoływać się do lokalnej bazy musiałbyś korzystać z REST. Czy ten kod jest gotowy na taką sytuację? Czy wystarczy, że podmienisz jeden obiekt, czy może będziesz musiał go pisać od początku?

- Rejestracja:ponad 17 lat
- Ostatnio:3 dni
- Lokalizacja:Warszawa
- Postów:2251
0
- Za dużo kodu w funkcjach (wszelkie if'y itp itd) - metoda zwroc jest totalnie przegięta.
- Polskie nazwy funkcji
- Komentarze - przy obecnym kodzie troszkę pomagają, jak napiszesz ten kod lepiej (nazwy funkcji jasno będa mówiły co robią) - komentarze będą zbędne.
- Niektóre funkcje zwracają niepotrzebnie całe stringi - zamiast danych (same stringi to już raczej w warstwie widoku powinny latać)

- Rejestracja:około 8 lat
- Ostatnio:około 7 lat
- Postów:60
0
Obczaj sobie ten pythonowy Tool do liczenia "złożoności kodu" :). Pomaga pisać proste, TESTOWANLE funkcje. Czym mniej punktów tym lepiej :)
https://pypi.python.org/pypi/radon
Zarejestruj się i dołącz do największej społeczności programistów w Polsce.
Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.