[dzeksondzekson]

#java #naukaprogramowania #codereview

https://pastebin.com/TJVCd2W9

Takie coś finalnie wyszło, dzięki za pomoc wszystkim, nie wiem jedynie czym zastąpić te przestarzałe get Year, poza tym działa.

Byłbym również wdzięczny za pomoc jeśli chodzi o testy jednostkowe, tzn. co testować i jak to ugryźć, bo czytam sobie teraz o tym ale nigdy testów nie pisałem.

[A.....t]

@dzeksondzekson: ja bym się nie brał za testy na tym poziomie, najpierw ogarnij porządnie język i programowanie obiektowe

[dzeksondzekson]

@AwizisieAkat: A może bardziej konstruktywnie?

[A.....t]

@dzeksondzekson: bo w sumie to masz wszystko w jednej klasie, spróbuj sobie powydzielać różne części do osobnych klas, np. klasę która ładuje ci userów z pliku, która korzysta z kolejnej klasy - konwertera pojedynczej linii z pliku do usera, itp.
wywal metody statyczne, i co do języka to może warto przerzucić się ze starych dat na te nowsze (LocalDateTime) - jak widzisz dostajesz warningi, że "deprecated", do czytania plików są też

[A.....t]

@dzeksondzekson: i wtedy np. masz klasę
StringToUserConverter {
public User convert(String line) {
String[] attr = line.split(",");
return new User(attr[0], attr[1], itp);
}

i wtedy do takiej klasy możesz sobie napisać test
np.
StringToUserConverterTest {
private final StringToUserConverter converter = new StringToUserCOnverter();

@Test
public void convertTest() throws Exception {
User user = converter.convert("jan,kowalski,555-123-546,2000-01-01");
assertEquals("jan", user.getFirstName());
itp.
}}

[dzeksondzekson]

@AwizisieAkat: Zamieniłem na DateTimeFormatter + LocalDate, i hula, mam teraz

// obliczanie wieku
 private static Integer obliczWiek(String dataUrodzenia) throws ParseException {

 DateTimeFormatter formatter = DateTimeFormatter.ofPattern("yyyy-MM-dd");
 LocalDate urodziny = LocalDate.parse(dataUrodzenia, formatter);
 LocalDate teraz = LocalDate.now();

 Integer wiek = teraz.getYear() - urodziny.getYear();

 return wiek;
 }

Co do programu nie bardzo wiem jak to przebudować żeby nie przestało działać, a trochę goni mnie czas.

[A.....t]

@dzeksondzekson:
w klasie User możesz zrobić metodę:

public int getAge() {
return dataUrodzenia.until(LocalDate.now()).getYears();
}

lub gdy dataUrodzenia to string
public int getAge() {
return LocalDate.parse(dataUrodzenia, formatter).until(LocalDate.now()).getYears();
}

[dzeksondzekson]

@AwizisieAkat: no nie bardzo chyba bo u mnie dataUrodzenia to jest String

[A.....t]

@dzeksondzekson: poszedł update przed chwilą, a tam jeszcze tego foramttera musisz zrobić w tej metodzie

[dzeksondzekson]

@AwizisieAkat: Super, mając tę metodę getAge mogłem wywalić tę całą metodę obliczWiek i ograniczyć się do

// zamiana daty urodzenia na wiek
 for (User user : list) {

 Integer wiek = user.getAge();

 user.setDataUrodzenia(wiek.toString());
 }

rozumiem że o to chodziło? :P

[A.....t]

@dzeksondzekson: nie o to, może napisz jakie masz zadanie to przygotuję jakiś przykład

[dzeksondzekson]

@AwizisieAkat: no żeby zrobić działający program i do niego testy

[WormRider]

Na testy to fajnie opracowane jest na drugim kursie na stronie https://www.edx.org/course/introduction-programming-java-writing-uc3mx-it-1-2x
Spokojnie mógłbyś napisać proste testy do swojego programu.

[sorhu]

@dzeksondzekson

no nie bardzo chyba bo u mnie dataUrodzenia to jest String

No i słabo. String to ciąg znaków. Nie nadaje się do reprezentacji daty.
Czytasz tę datę z pliku jako String. Wiadomo. Ale musisz to jak najszybciej zamienić na właściwy typ.