Talk:DevHelp

Meeskond: Agraarbeib

Sissejuhatus

Käesolev retsensioon on koostatud DevHelp meeskonna C# projektitööle.

Rakenduse kasutamise esmamulje

Rakendus on sihipäraselt kasutatav. Loodud on UI funktsionaalsusi: 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. Päringute tegemisel antakse meetodisse kõik propertid üks haaval (nt AddNewUser(string username, string password, string firstname, string lastname, string email, string phonenumber), selle asemel oleks võinud teba uue objeti UserBo ja anda see AddNewUser siis nt nii AddNewUser(UserBo newUser). Nii oleks antud terve objekt kaasa koos kõigi parameetritega, mitte üks haaval.

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