Pierwsza rzecz która mi się rzuciła: to brak informacji jak to uruchomić.
2. Brak informacji o samym projekcie (Co to właściwie jest)
3.Metody testów nie za bardzo mi mówią co robią. (klasa CardSetGeneratorTest)
Jedna rzecz mnie ciekawi:
(new Object[][]{
{"amount;wielkość", ";", new FlashCard("amount", "wielkość")},
{"digit;cyfra", ";", new FlashCard("digit", "cyfra")},
{"dividend;dzielna", ";", new FlashCard("dividend", "dzielna")}
});
Jaki był zamysł tej arcitektury.? czy próbowałeś mapować obiekt na klucz, mógłbyś użyć hashmap?
Użależniłeś jedną klasę od drugiej co nie jest zalecane w testach. FlashCard mógłbyś spokojnie osobno przetestować.
Kolejna rzecz, której nie kupuje:
String[] strings = CardGenerator.getFlashCard(text, regex);
Rozumiem, get ale dlaczego podajesz mu argumenty. Po tej metodzie spodziewałbym że zwaraca mi obiekt flashCard. A nie jakąś tablicę. A sama nazwa tablica też mi za wiele nie mówi co to właściwie jest.
String[] strings = CardGenerator.getFlashCard(text, regex);
assertEquals(strings[0], expected.getText());
assertEquals(strings[1], expected.getTranslation());
A sama metoda właściwie sprawdza flashCard. A po drugie sprawdzasz z innego obiektu flashCard? Nie kupuje tego.
@Parameterized.Parameters
public static List<Object[]> data() {
return Arrays.asList(new Object[][]{
{"amount;wielkość", "amountwielkość"},
{"digit;cyfra", "amount;wielkość;"},
{"dividend;dzielna", ";;amount;wielkość"}
});
}
Możesz to spokojnie ustawić jako setUP, nie musiałbyś tego dwa razy pisać. Zdecydowanie za mało testów.
return (tab != null) ? new String[]{tab[0], tab[1]} : null;
Spokojnie można by było napisać do tego metodę bo na pierwszy rzut oka nie widać za bardzo co ona robi. np:
return (isTableEqualNull()) ? returnArray() : null;
for (int i = 1; i <= list.size()/5+1 ; i++) {
for (int j = (i*5)-1; j >= (i-1)*5; j--) {
if(list.size() > j)
table.addCell(createCell(font, list.get(j)[index]));
}
Mogłbyś utworzyć osobny obiekt Row, wtedy nie musiałbyś w takie rzeczy bawić.
Btw. gdzie masz widoki bo nie mogę znależć.