Talk:DevHelp: Difference between revisions
Line 9: | Line 9: | ||
Rakendus on sihipäraselt kasutatav. Loodud on UI funktsionaalsust. Peites ja tuues nähtavale nuppe, mis peavad olema kasutatav alles peale mingeid konkreetseid tegevusi (nt esimese projekti lisamisel tekib alles siis "Kustuta projekt", kui projekt on lisatud). | Rakendus on sihipäraselt kasutatav. Loodud on UI funktsionaalsust. Peites ja tuues nähtavale nuppe, mis peavad olema kasutatav alles peale mingeid konkreetseid tegevusi (nt esimese projekti lisamisel tekib alles siis "Kustuta projekt", kui projekt on lisatud). | ||
Osa välju oleks võinud lahendada datepicker-ga (nt lisa iteratsioon, "Algus" ja "Lõpp" textBox), kuid samas võis see olla | Osa välju oleks võinud lahendada datepicker-ga (nt lisa iteratsioon, "Algus" ja "Lõpp" textBox), kuid samas võis see olla taotluslik, et jäta sisestamisel mänguruumi. | ||
Hästi on vaatetes kasutatud hästi Bindigut ja baasi meetodeid (nt base.NotifyPropertyChanged("Users");). | Hästi on vaatetes kasutatud hästi Bindigut ja baasi meetodeid (nt base.NotifyPropertyChanged("Users");). | ||
Revision as of 22:40, 31 January 2017
Meeskond: Agraarbeib
Sissejuhatus
Käesolev retsensioon on koostatud DevHelp meeskonna C# projektitööle.
Rakenduse kasutamise esmamulje
Rakendus on sihipäraselt kasutatav. Loodud on UI funktsionaalsust. Peites ja tuues nähtavale nuppe, mis peavad olema kasutatav alles peale mingeid konkreetseid tegevusi (nt esimese projekti lisamisel tekib alles siis "Kustuta projekt", kui projekt on lisatud). Osa välju oleks võinud lahendada datepicker-ga (nt lisa iteratsioon, "Algus" ja "Lõpp" textBox), kuid samas võis see olla taotluslik, et jäta sisestamisel mänguruumi. Hästi on vaatetes kasutatud hästi Bindigut ja baasi meetodeid (nt base.NotifyPropertyChanged("Users");).
Koodi head omadused
Projekt on ülesehitatud database first lähenemist. Projekt on loogiliselt jaotatud erinevate folderite vahel jälgides MVVM mustrit. Kood on selgelt ja arusaadavalt kirja pandud. Objektide, meetodite, propertite nimed on kasutamise eeesmärgi päraselt nimetatud ja seetõtu hõlbustavad koodi lugemist.
Koodi puudused
Koodi on vähe kommenteeritud. Puudub osaliselt sisestuse kontrollid. Samuti tekkis viga viimase projekti kustutamisel ja andmebaasi muudatuse salvestamisel.
Kokkuvõte
Projekt on hästi tehtud. Tugev eeltöö on varasemalt tehtud ja seetõttu tagab põhjalikud database struktuur projekti töökindluse ja paindlikkuse. Puudusena siis ainult kontrollide puudumine nii andmete sisestamisel. Oleks võinud lisada projekti ka Window elemente, et katsetada erinevate windowide ja page käitumist ühes projektis. Oleks tagatud parem UI keskond.
Retsenseeris meeskond Agraarbeib
Liikmed:
- Kaia Runthal
- Marianne Trubetskoi
- Taavi Tilk
- Mihkel Matson