Talk:Ostukorv

From ICO wiki
Revision as of 00:50, 28 January 2016 by Ntingas (talk | contribs) (Created page with "== Retsensioon lõpptootele == Meeskonna Konsa lõpptoode Ostukorv on Windows Phone rakendus, mis võimaldab koostada ostunimekirja. Projekti struktuur on selge ja loogilise...")
(diff) ← Older revision | Latest revision (diff) | Newer revision → (diff)
Jump to navigationJump to search

Retsensioon lõpptootele

Meeskonna Konsa lõpptoode Ostukorv on Windows Phone rakendus, mis võimaldab koostada ostunimekirja. Projekti struktuur on selge ja loogilise ülesehitusega - erinevad osad on jaotatud sobivatesse kataloogidesse ja projekti koodibaasist saab esimese pilguga hea ülevaate. Samas ei ole arendamisel kasutatud MVVM arhitektuurimustrit.

Seoses kõnealuse rakenduse funktsionaalse poole ülevaatega tuleb kahjuks mainida, et mitte ühelgi retsenseeriva meeskonna liikmetest ei õnnestunud seda rakendust sõltuvuste (Windows 8.1 SDK jms) tõttu käivitada. Siiski paistab, et lubatud funktsionaalsus on koodi tasandil implementeeritud.

Koodi kohta tekkisid järgnevad tähelepanekud:

  • Funktsioonide kommenteerimisel on kasutatud n.n. mitmerealise kommenteerimise stiili (/* */), kuid Visual Studio pakub kommenteerimiseks ka otseteed (///), mis loob meetodile või klassile vastava dokumentatsiooni malli.
  • Rakenduseüleste konstantide jaoks on eraldi klass (Constants), milles on vastavad muutujad märgitud static’uks, kuid mitte konkreetselt konstantideks (const). Kõigile selle klassi muutujatele võiks vastava täpsustuse lisada.
  • Klassid SuspensionManager, RelayCommand, ObservableDictionary ja NavigationHelper on pärit GitHub’ist ja teistest avalikest allikatest, kuid otseselt ei ole neile viidet.
  • Koodis on kasutatud tühje avalikke konstruktoreid, nt. andmemudelites, kuigi C# loob need ka ise automaatselt. Samuti on osades konstruktorites ka kasutamata parameetreid, nt. Session klassil “json”.

Projekti teostamise juhendi nõuetest lähtudes tuleb mainida, et andmebaasi asemel on kasutatud JSON-i põhist veebiteenust ning logimist ei ole kasutatud.

Retsenseeris Meeskond: Nocturne No. 20 in C-sharp Minor