Talk:ParkWell: Difference between revisions

From ICO wiki
Jump to navigationJump to search
Jkoort (talk | contribs)
Jkoort (talk | contribs)
Line 18: Line 18:
Olles teinud varem retsensiooni meeskonna ParkWell analüüsile, tundus nende rakendus väga potentsiaalirikas. Kahjuks jäi rakenduse potentsiaal siiski ainult paberile. Kahjuks polnud rakendusega kaasas ka kasutusjuhendit.</p>
Olles teinud varem retsensiooni meeskonna ParkWell analüüsile, tundus nende rakendus väga potentsiaalirikas. Kahjuks jäi rakenduse potentsiaal siiski ainult paberile. Kahjuks polnud rakendusega kaasas ka kasutusjuhendit.</p>
<p>
<p>
Selleks, et projekt tööle saada on vaja kommenteerida UserService.cs failis rida 144. Kui see on tehtud, siis saab programmi käivitada. Koheselt vaatab vastu roheline ekraan, kust tuleb valida kas sisse logimine või registreerumine. Juba registreerimisel on näha, et asi on natukene liiga lihtsalt tehtud: kasutajanime pikkus peab olema 4 tähemärki vähemalt, kuid vastu võtab ainult on vähemalt 5 tähemärki, emaili ja telefoninumbri väli ei kontrolli, mida sinna sisse reaalselt kirjutatakse. Samuti ei õnnestu miskipärast alati sisse logimine.
Selleks, et projekt tööle saada on vaja kommenteerida UserService.cs failis rida 144. Kui see on tehtud, siis saab programmi käivitada. Koheselt vaatab vastu roheline ekraan, kust tuleb valida kas sisse logimine või registreerumine. Juba registreerimisel on näha, et asi on natukene liiga lihtsalt tehtud: kasutajanime pikkus peab olema 4 tähemärki vähemalt, kuid vastu võtab ainult on vähemalt 5 tähemärki, emaili ja telefoninumbri väli ei kontrolli, mida sinna sisse reaalselt kirjutatakse. Tärniga väljad nõuavad sisendiks 0 tähemärki miinimumina. Kuvatakse nt veateade "minimum length of that field is 0", aga kasutajal pole võimalik teada, millisest konkreetsest väljast on juttu, kuna välja ei tooda kuidagi esile. Samuti ei õnnestu miskipärast alati sisselogimine.
</p>
</p>
<p>
<p>
Line 25: Line 25:
<p>
<p>
Must have punktidest toimib: kasutaja registreerimine, süsteemi sisse logimine, sõiduki registreerimine. Kõik muu funktsionaalsus puudulik.
Must have punktidest toimib: kasutaja registreerimine, süsteemi sisse logimine, sõiduki registreerimine. Kõik muu funktsionaalsus puudulik.
Rakendus on tehtud MVVM mudeli järgi. Kood on ilusti jaotatud klassidesse, kasutatud on palju BO-sid ning samuti on palju service klasse. Lisaks on xaml ära jagatud iga vaate peale eraldi, mis lihtsustab sellega töötamist. Koodi vormistus on üldiselt korralik ning kommenteeritud, kuid mõnes kohas leidub veel tegemata TODO-sid.
Rakendus on tehtud MVVM mudeli järgi. Kood on ilusti jaotatud klassidesse, kasutatud on palju BO-sid ning samuti on palju service klasse. Mõned klassid on tühjad nagu nt ProfileService.cs ja car.cs. Xaml on jagatud ära iga vaate peale eraldi, mis lihtsustab sellega töötamist. Koodi vormistus on üleüldiselt korralik ning kommenteeritud, kuid mõnes kohas leidub veel tegemata TODO-sid.
</p>
</p>
<p>
<p>

Revision as of 22:51, 31 January 2017

Analüüsi retsensioon

Wiki leht on üles ehitatud loogiliselt ja korrektselt. Analüüsi käigus on kõik vajalikud ja nõutud punktid täidetud, tänu millele on projektisisu lugejale arusaadav. Must have funktsioonid on põhjalikult välja toodud, aga võiks miskit ka nice to have all olla. Kasutatav tehnoloogia vajab natukene täpsustamist. Lisaks on korralikult välja toodud töö kulgemine koos kuupäevadega. Projekt tundub olevat sobilik kursuse programmeerimine C# keeles jaoks.

Projektiidee on iseenesest huvitav. Kui lõpptoote saaks valmis tehtud, siis võib see reaalses maailmas kasutust leida. Projekt oleks alternatiivlahendus tavapärasele tõkkepuuga parkimise reguleerimisviisile.

Tiim on toonud välja potentsiaalse probleemi, mis võib projekti loomisel tekkida, kuid lisaks sellele võib tekkida veel ootamatuid olukordi. Projekti probleemiks võib osutuda olukord, kus kasutajaks on vanemaealine inimene, kellel pole ligipääsu internetile või piisavalt oskusi, et programmiga ümber käia. Taolises olukorras on raske saada selline süsteem toimima.

Edu projekti kirjutamisel!

Retsenseeris meeskond ForeverNotes

07.11.2016

Lõpptoote retsensioon

Olles teinud varem retsensiooni meeskonna ParkWell analüüsile, tundus nende rakendus väga potentsiaalirikas. Kahjuks jäi rakenduse potentsiaal siiski ainult paberile. Kahjuks polnud rakendusega kaasas ka kasutusjuhendit.

Selleks, et projekt tööle saada on vaja kommenteerida UserService.cs failis rida 144. Kui see on tehtud, siis saab programmi käivitada. Koheselt vaatab vastu roheline ekraan, kust tuleb valida kas sisse logimine või registreerumine. Juba registreerimisel on näha, et asi on natukene liiga lihtsalt tehtud: kasutajanime pikkus peab olema 4 tähemärki vähemalt, kuid vastu võtab ainult on vähemalt 5 tähemärki, emaili ja telefoninumbri väli ei kontrolli, mida sinna sisse reaalselt kirjutatakse. Tärniga väljad nõuavad sisendiks 0 tähemärki miinimumina. Kuvatakse nt veateade "minimum length of that field is 0", aga kasutajal pole võimalik teada, millisest konkreetsest väljast on juttu, kuna välja ei tooda kuidagi esile. Samuti ei õnnestu miskipärast alati sisselogimine.

Kui kasutaja on sisse logitud, on tal võimalus endale autosid lisada, kahjuks eemaldamise funktsioon puudub. Kõikidel teistel tabidel funktsionaalsus puudub, on üritatud teha tabidesse valmis textblockid ning alustatud tööga, kuid rohkemat midagi pole.

Must have punktidest toimib: kasutaja registreerimine, süsteemi sisse logimine, sõiduki registreerimine. Kõik muu funktsionaalsus puudulik. Rakendus on tehtud MVVM mudeli järgi. Kood on ilusti jaotatud klassidesse, kasutatud on palju BO-sid ning samuti on palju service klasse. Mõned klassid on tühjad nagu nt ProfileService.cs ja car.cs. Xaml on jagatud ära iga vaate peale eraldi, mis lihtsustab sellega töötamist. Koodi vormistus on üleüldiselt korralik ning kommenteeritud, kuid mõnes kohas leidub veel tegemata TODO-sid.

Meeskond ParkWell kahjuks ei suutnud oma nägemust täielikult ellu viia, kuid kuna projektil on palju potentsiaali, siis võib loota, et projekti jätkatakse.

Retsenseeris meeskond ForeverNotes

31.01.2017