Talk:Meeskond "KRTK": Difference between revisions

From ICO wiki
Jump to navigationJump to search
Mmahar (talk | contribs)
No edit summary
Ljansen (talk | contribs)
No edit summary
 
Line 48: Line 48:


Mikk Mähar
Mikk Mähar
============================================================
Codeplexist kättesaadav variant oli küll katkine või vähemalt minul viskab hunniku erroreid ette. Kommentaare pole miskipärast üldse.
Lauri Jansen
D21
============================================================

Latest revision as of 10:36, 3 January 2011

Kõik on omal kohal ja asi töötav ja värvi üle ka ei nuriseks, enamus kontori tarkvara on hall ja valge. Samas nuriseks pika tee üle et tööle saada asi :)

Kõikke paremat tiimile, Liis Merisaar.

================================================

Kiidan erakordselt ausat ja illustratiivset wikit. Daunlõudisin koodi ja isegi baasi tekitamise SQLi ja isegi lasin ta mingis augus jooksma, aga tulemuseks sain hunniku erroreid "near to GO"... nii, et baasi ma rakendusele ise tekitada ei suutnudki. Samas on ikka natuke vähe juhendatud mismoodi ma peaksin midagi veel konfima, et see kõik tööle hakkaks. Nii et piirdun uskumisega, et see asi teil töötab. Minu usku kinnitavad ka koodivaatlused - see on jälle see Linq!

Anneli Rasu DK21


================================================

Huvitav projekt, aga lahenduse GUI jättis soovida (oli arusaamatu), samuti polnud võib-olla LINQ kasutamine hea otsus, sest tundus, et selle toimimisest ei saanud arendajad hästi aru kahjuks.

Tanel Lebedev


================================================

Olen sarnaselt Anneliga nõus, et wiki erakordselt aus ja illustratiivne. Hea on näha välja toodud pilt andmebaasi tabeli struktuuridest ja relatsioonidest. Kuid ikka jälle koodi vaadates, kommentaarid, kommentaartid ja veelkord kommentaarid (praktiliselt mitte ühtegi kommentaari), mis lihtsustaks edaspidi seda kellegi teise poolt edasi arendada. (muidugi kui seda vaja on) Kuid kahjuks puudub täpne juhis asja käima saamiseks, sest tõesti lihtsalt allalaadides ja käima lastes see asi ei käivitu.

Karel Niine

===============================================

Esiteks oleks pidanud arendajad veidi rohkem vaeva nägema, et kasutajad saaksid koodiga kaasa toimiva andmebaasilahenduse. Hetkel tuleb ilma igasuguse juhiseta hakata andmebaase seadistama enne kui programmi toimima saab. Teiseks probleemiks on kommentaaride puudumine, näiteks on vormi nuppude/väljade nimed jäetud a la 'button3' ja tuleb koodi järgi tuletada, mis ta teha võiks. Programmi kasutusmugavus polnud ka ehk see, mida ootaks - eriti raamatute laenutamis- ja tagastusprotsess. Aga muidu asi töötab ja teeb, mis vaja.

Lauri Õunmaa DK21

===============================================

Püüan hinnata tehtud tööd arvestades tegijate tausta, aga pole nõus nende argumentatsiooniga. Nagu eelkõnelejad on märkinud, jätab soovida vormikomponentide nimetus. Samuti on kehv jälgida automaatselt genereeritud ja üpris paljusõnalist koodi, kus tehtud mõned endale vajalikud muudatused (näiteks Form1.Designer.cs). Projekti ei õnnestunud paraku käivitada. Kiitust väärib wiki - päris viisakas ja piltidega varustatud juhend.

Alvar Lumberg DK21

===============================================

Wiki meeldis, samas üllatas koodis täielik kommentaaride puudus. Ei hakka kordama eelkõnelejate osa koodi loetavuse ja linq kasutuse koha pealt. Käima saamine oli ka nagu juba mainiti omaette tegevus andmebaasi loomise scripti puudumise tõttu(kuna kasutusel on MS SQL andmebaas oleks selle genereerimise skripti saanud paari klõpsuga kätte SQL Server Management Studio seest). Muidu GUI tundus minule sobiva struktuuriga (võimalik, et adminnid mõtlevad üht moodi) ja programm ise asjalik.

Wiki-s kurdetud riiuli muutmisega seotud probleemi lahendamiseks soovitaks lihtsalt ühe vahetabeli juurde tekitada, kus seostatakse raamat ja riiuliga.

Mikk Mähar

================================================

Codeplexist kättesaadav variant oli küll katkine või vähemalt minul viskab hunniku erroreid ette. Kommentaare pole miskipärast üldse.

Lauri Jansen D21

================================================