#programowanie #java Czytając wpis @bloodguy postanowiłem spróbować swoich sił i też rozwiązać to zadanie ( student here, nie pracuje w zawodzie ale szukam stażu/juniora ) i chciałbym żebyście się wypowiedzieli na temat mojego rozwiązania, starałem się poprawić to co zarzuciliście autorowi. github przykladowe dane Wołam ludzi którzy się wypowiedzieli ( nie bijcie pls ) @wczoraj @kisi3l @NiebieskaSowa @Trytka @Legol
@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,
@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ć.
Czytając wpis @bloodguy postanowiłem spróbować swoich sił i też rozwiązać to zadanie ( student here, nie pracuje w zawodzie ale szukam stażu/juniora ) i chciałbym żebyście się wypowiedzieli na temat mojego rozwiązania, starałem się poprawić to co zarzuciliście autorowi.
github
przykladowe dane
Wołam ludzi którzy się wypowiedzieli ( nie bijcie pls )
@wczoraj @kisi3l @NiebieskaSowa @Trytka @Legol
1. Formatowanie kodu
2. Brak pakietów
3. Zmienne instancji z defaultowym dostępem, czemu?
4. Klasa Reader, która
Reader.java
Czemu w ogóle ta klasa jest
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,
@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.
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ć.