Recenzja programu

pytanie zadane 10 sierpnia 2017 w Java przez xmigox Obywatel (1,090 p.)

Witam wszystkich,

Ostatnio zacząłem pisać w javie, zamierzam tworzyć apki mobilne ale na razie trenuje jave 8. Ostatnio napisałem swój pierwszy "większy" projekt - sklep w konsoli. Starałem się stworzyć czysty i zrozumiały kod. Także tak jak w tytule, proszę o rady etc.

Link do repozytorium: https://github.com/theprogrammer11011/Shop

1 odpowiedź

odpowiedź 10 sierpnia 2017 przez mbabane Szeryf (79,280 p.)

Długa metoda main, która ma komentarze do kodu, a wiec pewnie da się z niej wydzielić przynajmniej kilka metod jak nie klas.

//Prizes of different bottles sizes(prize0 = 0.5 liter bottle,   prize1 = 1 liter bottle,   prize2 = 1.5 liter bottle,   prize3 = 2 liter bottle)
	double prize0 = PrizeRound(prize/2);
	double prize1 = prize;
	double prize2 = PrizeRound(prize*1.5);
	double prize3 = PrizeRound(prize*2);

Potrzebny komentarz zatem nazwy zmiennych są nieprecyzyjnie dobrane.

W tej samej klasie jest tez metoda PrizeRound, czy nie powinna być ona prywatna? Używana jest tylko wewnątrz klasy (przynajmniej tak na szybko sprawdziłem i nie znalazłem użycia po za nią).

Jeśli chcesz być bardziej Javowy to metody nazywaj rozpoczynając z małych liter. Jeśli klasa nie jest typową struktura danych, w której są tylko zmienne i nic konkretnego, to staraj się nie używać publicznych zmiennych klasowych, a stosować tzw. settery i gettery (jeśli są potrzebne, bo czasami zmienne prywatne są używane tylko wewnątrz klasy i nie udostępnia się ich na zewnątrz).

Jesli chcesz doskonalić sztukę pisania dobrego kodu to zaglądnij tu: Robert C. Martin "Czysty kod. Podręcznik dobrego programisty"