otsha
commented
5 years ago Kiitos palautteesta! Päivämääräformaatti on nyt korjattu, ja kuten totesit, ongelmana oli null task, joka syntyi tyhjästä tiedostosta lukiessa (olin omissa testeissäni käyttänyt valmista dataa, minkä takia tämä tapaus jäi huomaamatta).
Jätän tämän auki, kunnes käyttöohje on kirjoitettu.
Ladattu 14:55 29.11.2018
Uutta taskia lisätessä kysytään päivämäärää muodossa (DD/MM/YYYY), joka kaataa ohjelman, vaikka oikeasti halutaan muotoa (DD.MM.YYYY). Lisäsin kolme taskia ja kokeilin Moore-Hodgson Schedulingia mutta ohjelma kaatui eikä taskeja tallentunut näköjään. Kun koitan katsoa taskeja näkyy alussa null, meneeköhän ohjelma siitä sekaisin? Nullin poistaminen aiheuttaa myös kaatumisen. UI näyttää selkeältä ja olisi ollut kiva päästä kokeilemaan juttuja. Näytit kampuksella ohjelman toimintaa mulle ja silloin ohjelma toimi halutusti.
Koodi on selkeää, hyvin jaoteltua ja kommentoitua. Testit on tehty omiin luokkiin ja on helppo nähdä mitkä testit testaavat mitäkin. Testit menivät läpi ja jacocoTestReport näytti lähes 90% kattavuutta.
Käyttöönotto-ohje ja käyttöohje olisi kiva olla olemassa. Esimerkiksi en heti tajunnut, että payment viittaa taskista saatavaan palkkaan. Dokumentaatio on muuten kattavaa, ja asiat on selitetty hyvin. Projekti näyttää kokonaisuudessaan aikalailla valmiilta, toivottavasti keksit vielä tekemistä loppukurssiksi.