@mediateh7: Nie podoba mi się.

1. Formatowanie kodu
2. Brak pakietów
3. Zmienne instancji z defaultowym dostępem, czemu?
4. Klasa Reader, któraPokaż całość

@mediateh7:

Reader.java

Czemu w ogóle ta klasa jestPokaż całość

@mediateh7: A, no i fajnie, jak byś dodał testy jednostkowe.

@hilemz: apropos pakietów to chyba z czystego lenistwa, bo już miałem w tym katalogu coś i po prostu usunąłem i zacząłem pisać.
Co jest nie tak z formatowaniem ? Default dostęp, kalam się moja wina. Właśnie nie wiedziałem za bardzo jak to rozbić bo wydaje mi się, że to w miarę proste metody.
Dzięki za krytykę. Pisałem program jakbym to tylko ja miał go uruchomić ale to chyba z pośpiechu,Pokaż całość

Co jest nie tak z formatowaniem ?

@mediateh7: Np. to samo, co z tym pytaniem. Po co spacja przed '?'?

Raz dajesz pustą linię przed 'return', raz nie. Raz dajesz pustą linię na końcu pliku, raz nie. Raz pusta linia przed komentarzem, raz nie. Brakuje zupełnie spójności.

@kisi3l: Narobiłem tyle komentarzy, ponieważ w poprzednim wpisie dużo ludzi się czepiało, że ich brak więc starałem się opisywać wszystko. Jeśli te dwie metody zrobię statyczne to mam je wywołać w konstruktorze ?

@mediateh7: Tamta dyskusja o komentarzach nie należy do zbyt wartościowych, aczkolwiek przewinęło się kilka trafnych stwierdzeń. Ogólnie, komentować powinniśmy zadbać, aby uświadomić DLACZEGO coś tak działa, nie natomiast informując JAK działa (bo to powinno dać się odczytać z kodu).

Metody statyczne wołasz poprzez Klasa.metoda(), czyli w Twoim przypadku: Reader.readLinesFrom() wywołasz z klasy Main.java. Nie potrzebujesz w ogóle instancjonować obiektu klasy Reader, wręcz powinieneś to uniemożliwić.

@mediateh7 wywal wszystkie komentarze i pisz tak żeby dało się zrozumieć z kodu. ofc czasem komentarze są przydatne, ale nie tutaj.