Talk:Meeskond Tool

From ICO wiki
Revision as of 20:25, 23 April 2011 by Itelga (talk | contribs)
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


Teenuskihi arvustus

Teenuskiht ei ole täielikult dokumenteeritud. Enamus meetodeid on üldiselt kirjeldatud ja enamus meetodite parameetrid ja tagastused (ehk mida return-itakse) on lahti seletamata.

Vaatasin muuhulgas Kasutaja klassi. Kasutaja paroolist krüptoräsi (hash) tegemine on minu meelest liiga keeruliselt teostatud (kasutatud on räsi soolamist ehk hash salting). Mina tegin selle lihtsamalt, kasutades

System.Web.Security.FormsAuthentication.HashPasswordForStoringInConfigFile(pass, "SHA1")

meetodit. Samuti on olemasolevate kasutajanimede kontroll liiga pikalt tehtud. Mina tegin kasutaja registreerimise meetodi nii:

        /// <summary>
        /// Registreerib kasutaja andmebaasi ja tagastab true kui kasutaja loomine õnnestus
        /// </summary>
        /// <param name="userName">kasutaja nimi</param>
        /// <param name="pass">parool avatekstina</param>
        /// <param name="liikID">kasutaja liigi ID</param>
        /// <returns>tagastab true kui kasutaja loomine 6nnestus</returns>
        public string RegistreeriKasutaja(string userName, string pass, int liikID, String rakenduseNimi)
        {
            string result = "Kasutaja loomine õnnestus.";
            LinqtoSqlDataContext kont = new LinqtoSqlDataContext();
            if (!kont.Kasutajas.Any(user => user.Nimi == userName)) // kui sellise nimega kasutajat pole
            {
                Kasutaja kasutaja = new Kasutaja();
                kasutaja.Nimi = userName;
                kasutaja.Parool_hash = FormsAuthentication.HashPasswordForStoringInConfigFile(pass, "SHA1");
                kasutaja.Kasutaja_liik_ID = liikID;
                kont.Kasutajas.InsertOnSubmit(kasutaja);
                try { kont.SubmitChanges(); }
                catch { result = "Kasutaja loomisel tekkis probleem. Proovi uuesti."; }
            }
            else { result = "Sellise nimega kasutaja on juba olemas."; }
            kont.Dispose();
            LisaParingAB(rakenduseNimi, "RegistreeriKasutaja");
            return result;
        }

Itelga 21:25, 23 April 2011 (EEST)