Talk:DevHelp: Difference between revisions

From ICO wiki
Jump to navigationJump to search
Ttilk (talk | contribs)
Kpikaru (talk | contribs)
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 taotsulik, et jäta 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ä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