Talk:DevHelp

From ICO wiki
Jump to navigationJump to search

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