Ladattu 17.2.2016 klo 17:45

Dokumentointi

Aihekuvausta voisi hiukan tarkentaa, vaikka aihe onkin selkeä. Itse lisäisin vaikkapa kuvan graafisestä käyttöliittymästä selkeyttämään kuvausta. Selkeyttä lisäisi myös mainin sijainnin lisäys luokkakaavioon.

Ohjelma

Ohjelma toimi tosi hyvin. Yritin löytää bugeja, taikka muuten vaan rikkoa ohjelmaa, ja en onnistunut kummassakaan. Printteri ja värivalikot oli mahtava lisäys, ja ainakin värivalikko toimi oikein. Printtaamista en päässyt harmikseni kokeilemaan. Ohjelma suostui jopa lukemaan jotain oudompia syötteitä. Esim ¸¸♬·¯·♩¸¸♪·¯·♫¸¸Happy Birthday To You¸¸♬·¯·♩¸¸♪·¯·♫¸¸ .
Pientä viilausta: Filteröi pelkkää välilyötiä olevat syötteet. Jos voit niin disabloi musta taustaväreistä, koska siitä ei pysty lukemaan mitään.

Koodi

Koodi on täynnä ihmeellisiä swing -olioita ja niiden kutsuja, jotka ilmeisesti luovat kaiken käyttöliittymä magian. Voisit hiukan kommentoida niiden toiminnallisuutta, koska jos nyt todella haluaisin ymmärtää kaiken mitä tapahtuu joutuisin googlettamaan aika paljon.

Omat luokkasi ovat hyvin jäsenneltyjä ja luettavuus on huippuluokkaa. Vaihtaisin itse luokan CreateTask nimen TaskFactoryksi tai joksikin vastaavaksi, koska se kuvaisi sen toimintaa paremmin.

Siis koodi on tosi selkeää muutamaa pientä kohtaa lukuunottamatta. Muutenkin ohjelman rakenne on selkeä ja järkevä.

Testit

Testikattavuus on tosi hyvä. Voisit jakaa jotain testejä osiin, koska tarkoitus on testata yhtä asiaa per testi.

---

Eli siis tosi hyvältä näyttää. Ohjelmahan on käytännössä valmis, vaikka aikaa on vielä paljolti. Siis tässä muutama kehitysidea:

1. Tapahtumille vapaaehtoinen kuvaus, joka esim pop-uppaisi esille, kun tapahtumaa klikkaa käyttöliittymästä.
2. Tapahtuman alkamisajankohdan voisi valita hiirellä taulukosta.
   2.1 Tapahtuman ajankohdan voisi maalata taulukosta (alkamisaika --- loppumisaika)
3. Lisää päivämäärät
   3.1 Mahdollista viikkojen välillä liikkuminen
   3.1.1 Viikottain/Päivittäin jne toistuvat tapahtumat

**-**Tykkäsin aiheenmäärittelystä, oli selkeä ja sovelluksen yleinentoiminta tuli esiin.

**-**Osa testeitä aika yksinkertaisia, esim. DayTest testit. Gui:n testejä oli aika vähän.

**-**Kun ajoin ohjelman toimi hyvin, mutta oli kohtia joihin ohjelma tyssäsi, esim. toisen merkinnän laittaminen sanoi aina "invalid output". Joitain koodinpätkiä olit laittanut kommenttien sisään.

**-**Dokumentaation kuvassa ei näkynyt luokkien välisiä viivoja. (en varma temppuileeko oma koneeni...) mutta tarkistamisen arvoinen asia.

**-**PlannerGui-luokan luomisesta plussaa.

Lataus aika: 23:33 05.02.2016