Talk:Ostukorv: Difference between revisions
(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...") |
No edit summary |
||
| Line 11: | Line 11: | ||
* 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”. | * 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. | Projekti teostamise juhendi nõuetest lähtudes tuleb mainida, et andmebaasi asemel on kasutatud JSON-i põhist veebiteenust ning logimist ei ole kasutatud. Ülaltoodud probleeme pehmendab asjaolu, et projekti arendas üheliikmeline meeskond ja selles valguses on tehtud siiski korralikult tööd. | ||
Retsenseeris [[Meeskond: Nocturne No. 20 in C-sharp Minor]] | Retsenseeris [[Meeskond: Nocturne No. 20 in C-sharp Minor]] | ||
Revision as of 01:02, 28 January 2016
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. Ülaltoodud probleeme pehmendab asjaolu, et projekti arendas üheliikmeline meeskond ja selles valguses on tehtud siiski korralikult tööd.
Retsenseeris Meeskond: Nocturne No. 20 in C-sharp Minor
