Talk:DevHelp: Difference between revisions
| Line 10: | Line 10: | ||
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 taotluslik, et jätta sisestamisel mänguruumi. | 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ätta sisestamisel mänguruumi. | ||
Hästi on vaatetes kasutatud | Hästi on vaatetes kasutatud Bindingut ja baasi meetodeid (nt base.NotifyPropertyChanged("Users");). | ||
| Line 16: | Line 16: | ||
'''Koodi head omadused''' | '''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. | Projekt on ülesehitatud kasutades 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. | ||
| Line 26: | Line 26: | ||
'''Kokkuvõte''' | '''Kokkuvõte''' | ||
Projekt on hästi tehtud. Tugev eeltöö on varasemalt tehtud ja seetõttu tagab | Projekt on hästi tehtud. Tugev eeltöö on varasemalt tehtud ja seetõttu tagab põhjalik database struktuur projekti töökindluse ja paindlikkuse. Puudusena siis ainult kontrollide puudumine andmete sisestamisel. Oleks võinud lisada projekti ka Window elemente, et katsetada erinevate windowide ja page käitumist ühes projektis. Oleks tagatud parem UI keskkond. | ||
Revision as of 22:42, 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ätta sisestamisel mänguruumi. Hästi on vaatetes kasutatud Bindingut ja baasi meetodeid (nt base.NotifyPropertyChanged("Users");).
Koodi head omadused
Projekt on ülesehitatud kasutades 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õhjalik database struktuur projekti töökindluse ja paindlikkuse. Puudusena siis ainult kontrollide puudumine andmete sisestamisel. Oleks võinud lisada projekti ka Window elemente, et katsetada erinevate windowide ja page käitumist ühes projektis. Oleks tagatud parem UI keskkond.
Retsenseeris meeskond Agraarbeib
Liikmed:
- Kaia Runthal
- Marianne Trubetskoi
- Taavi Tilk
- Mihkel Matson
