Talk:ParkWell: Difference between revisions
| (2 intermediate revisions by the same user not shown) | |||
| 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 | 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 ning parooliväljad on avalikult nähtaval, mis on potentsiaalne turvarisk. Mõned 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. | 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> | ||
Latest revision as of 22:59, 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 ning parooliväljad on avalikult nähtaval, mis on potentsiaalne turvarisk. Mõned 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
