Koodikatselmointi #1
Description
Projekti ladattu 29.8.2017 noin klo 21:30.
- Projektissa on noudatettu kattavasti Clean code -periaatteita ja koodista on helppo saada selvää.
- Metodit on nimetty kuvaavasti niin, että koodi dokumentoi itse itseään.
- Ohjelma on jaoteltu selkeisiin paketteihin ja rakennetta on helppo seurata.
- Javadoc on huolella laadittu ja helpotti entisestään koodin lukemista.
- Tiedostit jo hyvin itsekin, että Desktop-luokassa on vielä muokattavaa. Siistimisen ohessa voisit miettiä, olisiko tarkoituksenmukaista tehdä käyttöliittymästä useita luokkia. Yksi vaihtoehto olisi tehdä esimerkiksi luokka/luokkia näkymän kontrollointiin ja lisäksi sellaisia luokkia, jotka keskittyvät pelkästään näkymän näyttämiseen. Tällöin koodia ei kertyisi niin pitkiä tekstinpätkiä samaan luokkaan ja sen seuraaminen helpottuu.
- Testaus on kattavaa ja myös testiluokkien teossa on noudatettu hyvin Clean codea (metodit nimetty kuvaavasti, riittävän lyhyitä).
- Olet varmaan jo huomannutkin, että sovellus bugittaa jotenkin kun yrittää luoda uutta ryhmää eli painaa ”Create New Group”. Näkymän vasemmassa laidassa on kirjaimia, joista ei oikein saa selvää ja jos painaa nappia, jossa on kolmen pisteen kuva, sovellus tulostaa virheilmoituksia outputiin. Tämä toiminnallisuus varmasti siis on kesken vielä.
- Sovelluksesta on todella vähän mitään kriittisestä sanottavaa. Kannattaa jatkaa samaan malliin ja työstää vielä vähän käyttöliittymäpuolta, niin tulee loistavaa jälkeä. Projektiisi oli ilo tutustua!