[borrow]

#java #javapomoc <-- obserwuj proszę #codereview

https://gist.github.com/enseitankad0/ce1471b67069f91ece848b6a97d10222

Mogę prosić o code review? W sumie programik generuje losowe auta, wkłada je do pliku .dat i .txt a następnie czyta. To wszystko. Zastanawiam się czy coś dało się prościej porobić albo czy coś nie wygląda jak januszerka.

Pozdrawiam

[bohme]

@borrow
1. Wszędzie gdzie masz komentarze w stylu: // 1. WRITE TO DAT FILE powinna być nowa metoda.
2. Po co robisz coś takiego: Integer k = Integer.valueOf(i);? zobacz jakim typem jest i
3. Po co klasa Garage implementuje interfejs Map? Przez to niepotrzebnie masz takie kwiatki jak metoda entrySet która zwraca nulla.
4. Formatowanie kodu!! Puste linie, wcięcia itp...
5. Taki catch niewiele daje: catch(IOException io) {System.out.println("IO Exception" +

[srgs]

@borrow: obstawiam ze Garage.printGarage nie dziala tak jak chcesz, zwroci 1szy wynik a nie wszystkie

[A.....t]

@borrow:
1. zamiast zapisu do takiego pliku txt mógłbyś pomyśleć o csv
2. zmienne w Car mogą być final
3. klasa Garage według mnie jakaś taka bez sensu - część operacji implementujących interfejs map działa poprawnie, część niepoprawnie, zastanowiłbym się czy rzeczywiście warto implementować mapę
4. Ta metoda printGarage po pierwsze nie printuje, po drugie zwraca reprezentacje tekstową pierwszego samochodu lub nulla - zła jest albo nazwa funkcji albo implementacja, a w ogóle wywaliłbym tą metodę z tej klasy i zrobił nową odpowiedzialną za to
5. cała logika w mainie - konkretne funkcje do wydzielenia do innych metod, a

[borrow]

@srgs: tak masz rację, to martwa metoda której zapomniałem usunąć
@bohme: formatowanie robię głównie CTR ALT L w intelliJ.

@AwizisieAkat: bardzo cenne wskazówki
Dzięki panowie! Niedługo pojawi się V2.0

[63274682374]

@borrow:
boolean eof = false;
while(!eof) {
try {
Car car = (Car)