Kalkulator parametrów geotechnicznych - codereview

0

Witam, stworzyłem swój pierwszy "poważny" program. Chciałem prosić o ocenienie kodu, oraz konstruktywną krytykę.

Uprzedzam że część kodu zawiera polskie nazewnictwo z powodu specyfiki metod obliczeniowych opartych na polskiej normie (np. żwir w Polsce jest inaczej znormalizowany niż w Anglii, a słowo "pospółka" nie ma odpowiednika).

Interfejs tworzyłem, jednocześnie ucząc się.
Chociaż w przyszłości nie mam zamiaru wgłębiać się w swinga.

https://github.com/Feranodor/Geotechnical-Parameters-Application

0

Nie wgłębiałem się ale wygląda to dość patologicznie. Pisane chyba w stylu prof. Zelenta, ctrl+c i ctrl+v. Proponuję Ci się zapoznać z kolekcjami i poczytać o dziedziczeniu, bo niby używasz ale chyba korzyści z tego niewiele. Poszukaj informacji jak wyeliminować kod który się powtarza oraz o konwencji nazewnictwa w Javie.

0

W zasadzie niewiele jest dobrze. Nie ma tu mowy o żadnym MVC/MVP, wszystko upchane na jana do klas Widoku, poza tym wszędzie ifologia i copypaste. Zrobiłeś tam same enumy a potem całą logikę upakowałeś 3 klas ;]

0

No właśnie długo zastanawiałem się jak zrobić by pozbyć się ifologii. Czy jest inny sposób na implementację takiej tabeli jak w załączniku?

1 użytkowników online, w tym zalogowanych: 0, gości: 1