Aktywne Wpisy
![](https://wykop.pl/cdn/c0834752/69dcb0b73589885c5bbde7f7855c63b3426dd69ea2e85b63b34d4721f04e4feb,q60.jpg)
SzymonXes +209
Straciłem możliwość i żałuję. Co zrobić, taki los incela (╥﹏╥)
#p0lka #logikarozowychpaskow #tinder #podrywajzwykopem #blackpill
#p0lka #logikarozowychpaskow #tinder #podrywajzwykopem #blackpill
![](https://wykop.pl/cdn/c3201142/c710b4e951cc9adc473941e0ed47a8484acc50bd12d0117d382e4c71e1b8efaa,w150.jpg?author=SzymonXes&auth=dee1196a174d94381a4ca01a46149f12)
![](https://wykop.pl/cdn/c3397992/Minieri_sZhOBQPFjD,q60.jpg)
Minieri +61
#!$%@?ąc już od poziomu meczu to doping na Narodowym jak zwykle nie istniał, a zrobili nawet specjalne miejsce do intonowania przyśpewiek przez spikera z poziomu murawy xD I co? Jajco, oczywiście tylko "Polska! Biało - Czerwoni!", "My chcemy gola" i "Kuba Błaszczykowski" gdy schodził. Pod koniec było jeszcze "Wojtek Szczęsny Wojtek". Generalnie nic, typowo piknikowa atmosfera, niestety jak zwykle. Fajnie że coś próbują z tym zrobić ale jak wczoraj pisałem lepiej dać
![](https://wykop.pl/cdn/c3201142/a23c605ce1ebe24c0d6f81643a29fa1eb808af849ce5f2bb7334ec6bec13c62b,w150.jpg)
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
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" + io.getMessage());}
To
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
@bohme: formatowanie robię głównie CTR ALT L w intelliJ.
@AwizisieAkat: bardzo cenne wskazówki
Dzięki panowie! Niedługo pojawi się V2.0
boolean eof = false;
while(!eof) {
try {
Car car = (Car) locFile.readObject();
newGarage.put(car.getNumber(),car);
System.out.println(car.toString());
} catch(EOFException e) {
eof = true;
}
}
Błędne użycie wyjątków. Nie powinno się ich używać do sterowania logiką programu tylko do obsługi sytuacji wyjątkowych/błędów.
Do pliku powinieneś zapisać najpierw liczbę samochodów a potem same samochody.