patsaf / taskzero Goto Github PK

1. Czy w repo na pewno potrzebne są wszystkie foldery które są? Np. katalog .idea to katalog z ustawieniami IntellIJ, a share'ując go nie powinieneś nakazywać komuś ze swoich ustawień. W repo powinny się znajdować tylko pliki istotne. Zapoznaj się czym jest plik .gitignore

Trochę skomplikowany ten build.gradle - nie duplikuj w nim kodu, dependencje trzymaj też w jednym bloku.

2. ++ za użycie regexów.

3. Wyjątek, którym rzucasz powinien dziedziczyć po IllegalArgumentException

4. Czy instancje generowane przez Factory to rzeczywiście odpowiednie osoby? Kurde, generowanie losowych danych osobowych sadsasa fdsgscv nie jest dobrym pomysłem.
   4a. Przebuduj tę klasę na czytelniejszą, bo póki co jest masakra.
   4b. Klasa nie powinna zawierać w sobie listy instancji. Ma ona służyc do ich produkcji, więc powinna zwracać je na zewnątrz. Jeżeli chcesz zwracać je na zewnątrz posortowane, to w odpowiedniej metodzie powinnaś dać znać, że zwracana struktura danych jest już posortowana

5. Istnieją różne konwencje tworzenia metod toString, raczej polecam wybrać jedną z nich, np. JSON'a i z niej korzystać.