Java - ocena "projektu", co ulepszyć

pytanie zadane 6 maja 2018 w Java przez FroGiS Użytkownik (810 p.)

Siemka javy uczę się kilka tygodni i stworzyłem projekt ale wiem że pewnie dało by się go zrobić lepiej (dlatego nie wiedziałem do jakiej kategorii go dać projekty czy Programowanie>Java) chce abyście powiedzieli co zrobić aby :

- było czytelniejsze

- było bardziej zoptymalizowane

Link do GitHub

komentarz 7 maja 2018 przez Wiciorny Ekspert (269,790 p.)

4 klasy? Projekt? :D No nie jest to projekt

komentarz 7 maja 2018 przez FroGiS Użytkownik (810 p.)

Fakt może nie jest to wielki projekt ale wątpię że jest założone ile musi być klas aby nazwać coś projektem przynajmniej nigdzie nie widzialem.

komentarz 7 maja 2018 przez Wiciorny Ekspert (269,790 p.)

no nie musi ale projekt jest czymś nad czym pracuje się no dosyć sporo czasu mówiąc o projekcie wartym uwagi na gita, a w 4 klasach nie zamkniesz niczego funkcjonalnego - lub bedzie to z perspektywy clean code niewlasciwe ... napisanie czegos co wymaga dnia jest dosyć słabe

komentarz 7 maja 2018 przez FroGiS Użytkownik (810 p.)

dla ciebie wymaga dnia a ja nie umiem na tyle javy żeby robić PROJEKTY na setki klas w kilka dni. Wiem że nie jest to takie funkcjonalne dlatego nie dałem to na kategorie projekty a java, żeby ludzie z forum podpowiedzieli jak lepiej pisać na przyszłość...a projekt nie musi być w cale ogromny na kilkadziesiąt klas żeby go nazwać Projektem.

komentarz 8 maja 2018 przez Wiciorny Ekspert (269,790 p.)

dobrze, ale napisałeś wyraźnie "OCENA PROJEKTU" co poprawić co zamienić. Tu nie ma czego oceniać... i w tym rzecz, lepiej by brzmiało co poprawić w kodzie np.

1 odpowiedź

odpowiedź 6 maja 2018 przez Aisekai Nałogowiec (42,190 p.)

Sam się uczę Javy, dużych projektów za sobą nie mam ale parę zastrzeżeń mam:

-Długość funkcji, imo możnaby było podzielić je jeszcze bardziej.

-W każdej funkcji masz: File file = new File(fileName);. Zastanowilbym się, czy nie lepiej zamiast pola String fileName, dac już po prostu pole typu File.

-Nazewnictwo zmiennych: widziałem parę razy zmienne o nazwie hasło, zmień je na angielskie. Tak samo zmienna o nazwie "a" czy "b" niewiele mówi.

-Jak porownujesz zmienną String z jakimś napisem, to lepiej zamiast zmienna.equals("test") dac "test".equals(zmienna) - zadziała również wtedy, gdy zmienna = null. W twoim przypadku, rzuci wyjątek NullPointerException.

-Nazwa funkcji: information() to niezbyt dobra nazwa, lepsza byłaby printInformation(). Tak samo account().

-Nazwa klasy: Config. Dużo też nie mówi, a raczej wprowadza w błąd. Lepsza byłaby już np CreditCard, albo samo Card.

-W funkcji main, powinieneś tylko uruchomić odpowiednią metodę/utworzyć odpowiedni obiekt odpowiedzialny za całą logikę twojej aplikacji. Pozbylbym się tego co masz w mainie i zrobił z tego osobną klasę.

-Mógłbyś wyodrębnić kod sprawdzający hasło, czy kod odpowiedzialny za wpisanie hasła, bo z tego co widziałem w paru miejscach tam się to powtarza.

Na razie tyle.