Czołem Mircy,
Powoli zaczynam znowu programować. Ponad dwa lata temu skończyłem technikum, ale raczej unikałem przez całe cztery lata jakiegokolwiek samokształcenia, przez co sporo zapomniałem, o ile w ogóle chciało mi się coś nauczyć.
Ale do rzeczy: tworzę mały projekcik przelicznika miar w Javie. W sumie to w takiej formie chyba go już zostawię, jednak byłbym wdzięczny za code review. Wszelkie uwagi bardzo mile widziane, co do nazw zmiennych, logiki całego programu
Powoli zaczynam znowu programować. Ponad dwa lata temu skończyłem technikum, ale raczej unikałem przez całe cztery lata jakiegokolwiek samokształcenia, przez co sporo zapomniałem, o ile w ogóle chciało mi się coś nauczyć.
Ale do rzeczy: tworzę mały projekcik przelicznika miar w Javie. W sumie to w takiej formie chyba go już zostawię, jednak byłbym wdzięczny za code review. Wszelkie uwagi bardzo mile widziane, co do nazw zmiennych, logiki całego programu






























Poprawiłem chyba większość z tego co mi wytknęliście. Ale przez to pojawił mi się problem, jak podzielić program na package.
Nie bardzo też mogę wymyślić w jaki sposób opuścić wybieranie jednostki i powrócić do menu głównego (ostatnia metoda w klasie UnitReader).
I znowu jeżeli macie jeszcze jakieś inne uwagi, to piszcie, proszę. Chcę zostawić
Widzę, że masz zdublowaną metodę writeProblemMessage -> stwórz nową klasę, której zadaniem będzie wypisywanie errorów.
W klasie UnitReader modyfikatory dostępów mają być defaultowe? Jak wsadzisz to do jakiegoś pakietu to nie będziesz miał dostępu do tych metod.
Staraj się nie zostawiać zakomentowanego kodu, zwłaszcza jeśli idzie to na mastera.
int index = 1; ?????? co oznacza ta zmienna? może powinno to być
nie rób takich rzeczy: