Talk:Pie Piper: Difference between revisions

From ICO wiki
Jump to navigationJump to search
 
Line 21: Line 21:
=Lõpptoote retsensioon=
=Lõpptoote retsensioon=


TODO
Pie Piper on lubanud, et nad teevad uue ja võibolla ka parema messengeri, küll aga, et sinnani jõuda oleks vaja veidi rakenduse kallal töötada.
 
== Negatiivne pool ==
* Juhul kui admin-vaates kasutajat mitte valida ja vajutada change-pass jookseb rakendus kokku
 
Kinni ei ole peetud enda must-have tekstist, mis väidab, et kasutajal võib olla kolm rolli. Küll aga hetkel on olemas admin kasutaja ning tavakasutaja. Moderaator on puudu.
 
Lisaks on öeldud, et andmebaasimudel peaks olema 9 tabeliga, hetkel realiseeritud on vaid 6. Puudu on topic, chatroom-group ja chatroom.
 
Kui natukene kurjustada siis üldiselt C#-i kirjutades on muutuja nimed suure algustähega.
 
== Mida muuta? ==
* Good to have oleks ka logout nupp, et saaks kergelt välja logida.
* Disaini
 
== Positiivne pool ==
Idee oli väga huvitav ning ka tegelikult väga mõnusalt struktureeritud. Kasutatud olid erinevad kaustad ühe projekti all. See on OK lahendus kui meil on tegemist võrdlemisi pisikese rakendusega ( nagu ka see seda oli ). Küll aga suuremas projektis oleks vaja liikuda erinevate projektide alla.
Iseenesest pigistades silma kinni puuduste eest must-have projekti all on rakendus hästi kirjutatud ning mõnusalt kommenteeritud.
 
Mina hindaksin seda 80-90% vahel asuva hindega, sest vaeva oli nähtud ning puudused ei ole ülemäära suured. Eeldan, et aine eesmärk on täidetud.
 


''Retsenseeris meeskond Kuri Kohvikupidaja 15-16.01.2017''
''Retsenseeris meeskond Kuri Kohvikupidaja 15-16.01.2017''

Latest revision as of 01:00, 17 January 2017

Analüüsi retsensioon

Esmapilgul tundub analüüs lühike, aga kui läbi lugeda siis täiesti sisukas. Hea mõte oli lisada andmebaasi mudel. Idee on tervitatav, nostalgia mõttes võiks disain samuti sarnaneda Windows Messengeriga. Analüüs on päris hästi struktureeritud. Andmebaasi skeemilt ning selle olemi seletusest jääb ikkagi aru saamatuks olemi “Friendship message” funktsionaalsus/vajadus. Kas sellega saadetakse teineteise vahel privaatseid sõnumeid või hoopis sõbrakutseid? Samuti jääb konsoolirakenduse vajalikkus meie grupile arusaamatuks, kas seda soovitakse kasutada log’imiseks? Tööjaotus on kavalalt tehtud, sest tavaliselt ei olegi kõigil ühtset ülesannet. Eriti väikestes tiimides.


Must have: Kasutajate haldus kick/ban pole veel andmebaasi skeemis kirjas.


Nice to have punkt “Rakenduse osasid (server/klient) on võimalik jooksutada erinevates masinates” jääb arusaamatuks. Soovituseks : sõnumite krüpteerimine ja VoIP võiksid samuti veel nice to have funktsioonide alla käia.


Mugav kasutajakogemus on alati tervitatav.


Retsenseeris meeskond Hidden Mustard 08.11.2016

Lõpptoote retsensioon

Pie Piper on lubanud, et nad teevad uue ja võibolla ka parema messengeri, küll aga, et sinnani jõuda oleks vaja veidi rakenduse kallal töötada.

Negatiivne pool

  • Juhul kui admin-vaates kasutajat mitte valida ja vajutada change-pass jookseb rakendus kokku

Kinni ei ole peetud enda must-have tekstist, mis väidab, et kasutajal võib olla kolm rolli. Küll aga hetkel on olemas admin kasutaja ning tavakasutaja. Moderaator on puudu.

Lisaks on öeldud, et andmebaasimudel peaks olema 9 tabeliga, hetkel realiseeritud on vaid 6. Puudu on topic, chatroom-group ja chatroom.

Kui natukene kurjustada siis üldiselt C#-i kirjutades on muutuja nimed suure algustähega.

Mida muuta?

  • Good to have oleks ka logout nupp, et saaks kergelt välja logida.
  • Disaini

Positiivne pool

Idee oli väga huvitav ning ka tegelikult väga mõnusalt struktureeritud. Kasutatud olid erinevad kaustad ühe projekti all. See on OK lahendus kui meil on tegemist võrdlemisi pisikese rakendusega ( nagu ka see seda oli ). Küll aga suuremas projektis oleks vaja liikuda erinevate projektide alla. Iseenesest pigistades silma kinni puuduste eest must-have projekti all on rakendus hästi kirjutatud ning mõnusalt kommenteeritud.

Mina hindaksin seda 80-90% vahel asuva hindega, sest vaeva oli nähtud ning puudused ei ole ülemäära suured. Eeldan, et aine eesmärk on täidetud.


Retsenseeris meeskond Kuri Kohvikupidaja 15-16.01.2017