Talk:Meeskond "Tyhik"

Arvustus

Meeskonna Tyhik loodud kinokavade xml faili struktuur on nõuetekohane, kenasti stiilis 
<kinod>
   <kino>
      ...
ning 
<filmid>
   <film>
      ...
Stiili- ja abifailid on olemas ning kui ma neid läbi vaatasin, siis vigu ei tuvastanud.
Lehe esialgne kujundus on hea, mulle meeldib.
Kokkuvõttes: minu arvates hea raam, millele veebirakendus ümber ehitada.

Indrek Tamm, meeskond Tool, 4.III.2011

Võib öelda, et XML on väga põhjalik ja seal on kajastatud kõik andmed, mis kinokülastajale huvi võiksid pakkuda. Isiklikult ei leidnudki kohta, kus oleks tahtnud midagi teistmoodi teha. Sellise XML põhjal on kindlasti võimalik korralik teenus ja klientrakendus ehitada. Samuti on olemas skeemifail ja korralik XSLT, mis esitab andmeid vägagi loetaval ja ülevaatlikul kujul.

Siim Aaver, D21

Uurides XML, uurisin mis uurisin aga ei oska kohe midagi ette heita. XML fail tundub ilusti struktureeritud, ja paistab et on olemas ka kõik vajalikud asjad, mis kinosüsteemis peaksolemas olema. Kõik failid on korralikult loetaavad ja arusaamisega probleeme ei teki. Lihtne ülesehitus.

Jaak Lehtsalu D22

XML fail keskendub elementidele, struktuur on loogiline. Andmed hoitakse võimalikult detailselt, nii et midagi halba pole öelda. Skeemifailis on seatud elementidele vastavusse andmetüübid, mida neis hoitakse, korralik töö. Stiilifail mitte ainult ei loe lihtsalt XMLi elementide sisu vaid paneb need ka stiliseeritud tabelisse, mis parandab loetavust. Tore oleks tõesti sait, kust kerge kinodes toimuvast infot saada ning see on hea põhi sellise teenuse loomiseks.

Reigo Männiste D22

Väga põhjalik XML fail ning väga hästi ülesehitatud stiilifail. Kõik vajalik on olemas ning veel lisaks ka muudki.

Nimetused oleks võinud teistmoodi üles ehitada. Seal on olemas originaal ning tõlgitud filmi nimi, kuid sinna võiks olla võimalik lisada ka muid keeli. Kuigi kui rakendus jääbki ainult eesti keelseks siis pole vahet.

Jarko Kaskmaa D21

Teenuse arvamus

Erinevaid teenuseid tundub olevat piisavalt palju, et antud veebiteenusele
saaks ehitada ühe korraliku ja kasuliku rakenduse. 
Natuke tekitavad kahtlused näitkes võta hind, kustuta hind - mille jaoks neid vaja on?
Kui see käib mingi seansi kohta, siis võiks see olla näiteks filmi objekti küljes
koos teiste filmi parameetritega. Kui liiga palju meetoteid on, pole ka alati hea.
Aga samas, see on minu arvamus. Edu!

Taavi Ilves D21

----

Vastus: 

Tegelikult saab filmi küsimise meetodile öelda, kas ta annab ka alamobjektid (hind, zanr, inimesed jne) või mitte. 
Eraldi ongi sellepärast, et saaks küsida ainult filmi kohta infot. Pannes üheks parameetriks "true", siis pistetakse
hind filmi objekti külge. Samas on vaja hinda eraldi pärida näiteks kino hinnakirja genereerimiseks. Ning hindasid
peab saama eraldi ka kustutada, kuna hinnad ei ole ju jäigalt seotud filmiga vaid ikka kinoga.

Aga täname arvamuse eest :)

Kert Suvi D22

Teenuse arvustus

Teenuseid on korralik hulk, kuid Eestikeelsed API nimed ei ole arusaadavad välismaalastele. Segadust tekitab veel "anna*" ja "võta*", kas mina annan talle ja ta võtab, või võtab tema ja annab mulle - nimi peaks ütleme kohe mis suunas liiklus käib. Tundub et mõned päringud saaks kokku mergeda, näiteks annaFilmid ja annaFilmidNimi. Üldiselt aga on kena :)

-Tanel Liiv D22

Teenuse arvustus

Esimese asjana hakkas silma, et te olete iga päringu tüübi jaoks teinud eraldi teenuse. Minu arvates koodi loetavuse mõttes on see hea, aga teenust ennast oleks mugavam kasutada kui kõik meetodid oleks üheks teenuseks koondatud. Üldiselt tundub, et teenusel on olemas kõikvõimalikud meetodid, mida vaja võiks minna. Positiivne on, et peaaegu kõigil meetoditel on olemas ka kirjeldus, et mida see teeb ning mis parameetreid vajab. Projektiga võiksite .sln või vähemalt .csproj failid ka kaasa panna, et visual studios parem vaadata oleks ning käivitada saaks.

Siim Aaver D21

---

Vastus:

Üks põhjus, miks jagasime tükkideks oligi loetavus. Teine põhjus on see, et kui keegi peaks avaldama soovi rakenduse tegemiseks meie teenusele, siis saame anda ligipääsu ainult "AnnaTeenus"-ele. Kui keegi teeb lihtsalt kinokava kuvamise rakenduse, siis pole tal vaja ei lisamise ega kustutamise teenust/meetodeid näha.

Aga täname arvamuse eest :)

Kert Suvi D22

Teenuse arvustus

Wiki lehel on teenuse all välja toodud kõik meetodid, aga pole ühelegile meetodile juurde kirjutatud mida too teeb, aga mis on hea - kui lähtekoodi uurida siis seal on iga meetodi juures ära kirjeldatud, mida too teeb ja mida ta tagastab. Lisaks häirib kasutatud "estenglish" kasutamine lähtekoodis, st meetodi nimed on eesti keelsed. Üldiselt kood on loetav ja ilusti struktureeritud.

Toomas Soha D21

Arvustus veebiteenuse klientidele

Kuna viibin hetkel arvuti taga kus ma ei saa visual studiot kasutada siis ei saa nii täpset arvustust anda(nägin teie esitust klassis). Tahaks öelda mis häirib on see, et meetoid ja asjad on eestikeelsed. Rohkem oleksite võinud kommenteerida ka, kuigi esmapilgul tundus kõik suhteliselt arusaadav. Muidu on idee täitsa hea ja kindlasti seda täiendades saaks täiesti reaalselt kasutatava rakenduse ;)

Madis Sildaru D32

Veebiteenuse arvustus

Kino teema on väga huvitav ja hea teema. Proovisin ka ise midagi sellist ühe teise aine raames teha. Lisan siia, kuidas mulle meeldib koodi kirjutada (mitte mis valesti on):

Vähegi suurema projekti puhul ei kasutaks linq't, kuna ta genereerib palju tarbetud koodi. Kui ma ei eksi, siis teil on hetkel nii linq kui oma loodud klassid. Andmebaasist kustutamise arvelt saaks palju kokku hoida, kuna tegelikult on ju parem kui jääb kõik ajalugu alles. Filmi / kino saab lihtsalt märkida suletuks. Mida suurem projekt, seda rohkem tuleb klasse nagunii, seega ma ei teeks igale suvalisele väärtusele klassi, vaid realiseeriks täpselt niipalju meetodeid, kui parasjagu tarvis läheb. Samuti, kui vähegi võimalik, koondaks ühe teema funktsioonid ühte klassi (n: select, insert, delete jne).

Kliendi pool oli lahedalt kujundatud ja ka repeaterite kasutamine oli hea mõte. Ma kasutaks ka webUserControl elemente, siis ei peaks nii palju erinevaid lehti tegema ja saaks ka teinekord kasutada.

PS! Seda, ma tulin teie projekti hindama lootuses, et saan mõne hea kommenteerimis vihje aga... ei saa öelda, et ma just palju kommentaare oleks leidnud  :)

Priit Siimo

Kokkuvõttev arvustus

Meeskond valis hea teema. Kino infosüsteemi on kindlasti midagi, mida saaks ka reaalselt kasutada, muidugi kui kõik väga korralikult teha ning rakendust piisavalt siluda. Samas oleks võinud selle teema kohta alguses wikis ka natuke juttu olla, et millist lõpptulemust oodatakse, millise funktsionaalsusega ning mis rakendustega.

XML tehti igati korralikult nagu ka teenusekiht, kus kõik teemade kaupa ilusasti ära oli jagatud. Rakenduse puhul meeldis mulle välimus, mis jäi kohe positiivse asjan silma. On küll sees väiksemaid buge, kuid põhifunktsionaalsus paistab töötavat.

Meeskonna toimimise poole pealt on hea näha seda, et võeti kasutusele teises aines õpitud ekstreemprogrammeerimise praktika, et rakendust tõhusamalt ehitada. Lisaks tehti enamus asju ühiselt valmis, seega ilmselt kõik meeskonnaliikmeid panustasid korralikult lõpptulemusele.

Kui jätta välja teine klientrakendus, siis tundub, et kõik nõuded said täidetud, muidugi wiki oleks võinud natuke põhjalikum olla. Aga vaadates üldist projektide taset, siis igati korralik tulemus !

Rauno Rüga, D21

Rruga