Talk:Meeskond "FilmiAhi"

From ICO wiki
Jump to navigationJump to search

Oleks oodanud, et projekti saaks alla laadida kokkupakitud failina. Kuid mis siis ikka, tegin TortoiseSVN-iga checkout

http://filmiahi.googlecode.com/svn/

ja tšekkisin välja, mis värk on. Panin ka andmebaasi faili test.db kausta ...\Visual Studio 2010\Projects\FilmiAhi\Filmiahi\trunk\Filmiahi\bin\Debug\ ja testimine võis alata.

Esimese asjana märkasin, et nupud ei eristu nuppudena, vaid paistavad nagu raamitud Label-id. Soovitan kasutada taustaga võrreldes muid värve. Positiivne on see, et kasutaja ja filmi lisamise aknas on nupud eristuvad.

Lisa Film, Lisa kasutaja, ja Seaded nuppudel klikkides saab kasutaja läbustada ja avada väga palju aknaid. Soovitan panna mingid piirangud, kui mitu akent saab lahti olla või siis kohe peale esimese akna avamist teha nupp inaktiivseks ( nupp.IsEnabled = false ). Kuigi samas valitud nuppude stiili juures ei saa peale vaadates aru, kas nupp on aktiivne või mitte. See probleem on näha Kasutajad tabi all, kus alguses on nupp Salvesta muudatused inaktiivne kuid vajutades Ava saab sellele vajutada.

Kasutaja lisamisel ei ole muutust kohe DataGrid-is näha vaid tuleb vajutada Otsi kasutajat.

Wikis on korralik kasutusjuhend ja programmi kood on suhteliselt põhjalikult kommenteeritud.

Ilmar Telga (D21)

Itelga 15:47, 21 December 2010 (EET)



Paigaldamiseks pidi svn checkouti tegema, ekstra 3 minuti tööga oleks saanud installeriks publishida. Midagi muud õnneks polnud vaja installida.

Wiki's lingitud eeltäidetud andmebaas on reaalsuses sama tühi andmebaas, mille ka rakendus loob.

Rakendus ise on täiesti toimiv lahendus ja kõik nõuded on täidetud, kuid nõustun ülalpool Ilmariga - nuppude kujundus jätab tõesti mulje, et nad on disabletud ning ka mitme akna loomise peaks piirama. Laenutamise loogika vajab mõningast harjumist (just selle funktsionaalsuse üles otsimine) ja ka žanritega toimingud on piiratud (muutmine ja kustutamine)

Üldiselt, kui tõsta paar elementi ringi ja muuta värve, on tulemus hea.

Kood on antud projekti jaoks piisavalt kommenteeritud ning klassidesse jaotatud.

Wiki on sisukas (kasutusjuhend, klasside kirjeldus olemas), kuid paar pilti ei teeks paha.


Mihkel Sokk D21


Programmi installimiseks tuli see SVN'i kaudu alla laadida. Selle asemel oleks võinud luua väikse installeri või vähemalt zip faili lihtsamaks alla laadimiseks ja paigaldamiseks.

Programm töötas stabiilselt ja kasutamise ajal kokku ei jooksnud. Seega töökindlusega võib üsnagi rahule jääda.

Kasutajaliides oli arusaadavalt ülesse ehitatud, ehkki võinuks veidi mugavam olla. Probleemseks võib osutuda uute akende pidev avanemine, mis võib kasutajas segadust tekitada.

Kood on võrdlemisi hästi klassideks jaotatud ning hästi struktureeritud ja korrastatud. Üldiselt on see üsna lihtsalt loetav.

Wikis olid olemas kõigi klasside kirjeldused ning üsna korralik kasutusjuhend.

Üldiselt on programm hästi ülesse ehitatud ning töötab võrdlemisi korralikult, ehkki kasutajaliides võinuks olla veidi paremini läbi mõeldud. Muidugi, korraliku liidese disainimine võib üksjagu aega võtta.