Talk:Meeskond Tool

From ICO wiki
Jump to navigationJump to search

Väga hea on kohvikutele vahelduseks ka mingit muud xml-i lugeda. Selge ja arusaadav teenus, hea lühike ja ülevaatlik, treppida oleks saanud muidugi paremini.

Täiesti mõtetu arvustus aga mis teha.

Priit Siimo D22

XML Arvustus

Meeldib, et XML on väga lühike ja kompaktne. Võib-olla võiks lisada ka pildi lingi antud lumelaua juurde. XMLi treppimine on natuke paigast. Mõnes kohas on taane kaks tühikut ja mõnes kohas koguni terve tab.

Oleks võinud XSLT-le ka veidi CSSi külge panna, et tabelile veidike silmailu anda.

Meeskonna idee on väga hea ja wiki leht sisukas. Ootan huviga projekti uuendusi.

Martin Lensment D21

Hea lühike xml, korralik wiki leht. Silma hakkasid xmli treppimisega tehtud vead. Mis siin muud ikka kirjutada :P.

Allar Anijago D21


Nagu ka eelnevalt kommenteeriti - tore et on valitud midagi peale kohviku. Kõik on ilus, tore, kena aga parameetrite osa oleks lahendanud teisiti.

Näiteks: <parameetrid>

 <parameeter>
    <tyyp>Pikkus</tyyp>
    <v22rtus>2000cm</tyyp>
 </parameeter>
 <parameeter>
    <tyyp>Laius</tyyp>
    <v22rtus>3000cm</tyyp>
 </parameeter>

</parameetrid>

See võimaldaks kasutada pmts lõpmatuseni erinevaid parameetreid. Praegu tuleks uue parameetri käiku võtmisel XML ümber teha. Kui aga olete kindlad, et seda ei juhtu, siis pole vahet :)

Kõik muu on hästi tehtud.


Kert Suvi D22


XML fail on selge ja hästi loetav, piisavalt lihtne, samas katab funktsionaalsuse. Pildid võimalus võiks ka tegelikult olla küll. Treppimine on so-so, loetav siiski. Hästi tehtud, ei ole vaja asja mõttetult keerukaks ajada, tublid!

Taivo Türnpu D21


Veebiteenuse arvustus

Üks vähestest projektidest, mis sai tähtajaks valmis ja on korraliku wiki lehega.

Vaatasin muuhulgas Kasutaja klassi. Kasutaja paroolist krüptoräsi (hash) tegemine on turvalisuse huvides keerulisemalt teostatud (kasutatud on räsi soolamist ehk hash salting). Kuid registreeritud kasutajate hulgast juba olemasolevate kasutajanimede kontroll on liiga pikalt tehtud. Oleks võinud kasutada LINQ meetodit Any()

    if (! kont.Kasutajas.Any(user => user.Nimi == userName)) // kui sellise nimega kasutajat pole
    {
        // siis sisesta kasutaja andmed baasi
    }

Kood on jaotatud klassidesse ja klassid ilusti regioonidesse (#region ja #endregion). Kuid dokumentatsioon ei ole täielik. Enamus meetodeid on üldiselt kirjeldatud ja enamus meetodite parameetrid ja tagastused (ehk mida return-itakse) on lahti seletamata.

Üldine mulje tööst on hea.

Ilmar Telga 21:25, 23 April 2011 (EEST)

Vastus Ilmarile

Aitäh kommentaari eest - võtame arvesse ja viime parandused sisse. Selline kasutajakontroll on kõvasti lihtsam tõepoolest ja dokumentatsioon saab ka parandatud.

Miina Sikk