Latasin projektin 3.5 kello 18.30

Palautetta:

• Koodi on selkeää ja metodien nimet ovat hyviä. Koodia lukiessani pysyin kohtuu hyvin kärryillä paitsi google spreadsheets-metodit olivat tajunnan rajoilla. Main-luokan nimi TestUI:n voisi tosin olla joku vähän ilmeisempi.
• Excercise-luokka herätti hieman mietteitä siitä, miksi juuri kaksi parametriä? Ohjelma voisi kenties kysyä parametrejä käyttäjältä kunnes käyttäjä päättää, että parametrejä on tarpeeksi.
• Testit olivat hyvin nimettyjä ja vaikuttivat testaavan luokkia kattavasti. Kenties ainoa miinus oli että ExcerciseTestin metodi nameIsCorrect() testaa periaatteessa neljää eri asiaa samassa testissä testatessaan eri konstruktoreilla tehtyjä Excerciseja.

Itse ohjelman käytössä huomasin hieman enemmän ongelmia:

• Ohjelman pyytäessä käyttäjältä tietynlaisia arvoja ja tyhmän käyttäjän ignooratessa nämä pyynnöt, ohjelma kaatuu. Näin kävi antaessa päivämäärän tilalla kirjaimia ja sessioista kertoessa antaessa kirjaimia.
• Tallennus spreadsheetsillä toimii ensimmäisellä kerralla mainiosti. Loistava lisäys ohjelmaan kyllä! Kuitenkin opin kantapään kautta, että ohjelma pyyhkii edellisen Routinen tiedot uuden tieltä. Ohjelma voisi kenties varoittaa tästä käyttöliittymässä. Vaikka vanha rutiini pyyhkiytyy, vanhat suoritettujen treenien tiedot pysyvät ja uudet tulevat niiden perään. Tämäkin on hieman epäloogista.
• Ääkköset eivät näyttäneen näkyvän spreadsheetsissä ja niistä tuli vain kysymysmerkit taulukkoon.
• Ohjelma pyytää käyttäjää kertomaan sessiosta Integer numerolla. En ihan tajunnut mitä tämä tarkoittaa. Ohjelma voisi selittää tätä kenties tarkemmin.

Kehitysideoita

• Ohjelman Dao-interfacet voisi yhdistää yhdeksi Dao interfaceksi jonka save metodia ei olisi määritelty tarkasti. Esim tyyliin Public Interface Dao<T> ja siinä metodi save(T element, ....). Kumpikin tätä implementoiva luokka voisi käyttää T paikalla tarvitsemaansa luokkaa ja säästyttäisiin samankaltaisilta Dao-interfaceilta. Esim. siis tyyliin public class SheetRoutineDao implements RoutineDao<Routine> jolloin RoutineDao käyttäisi T paikalla Routinea.

Hyvää työtä ja tsemppia vikalle viikolle!