Talk:Meeskond Tool: Difference between revisions

From ICO wiki
Jump to navigationJump to search
Line 49: Line 49:




== Teenuskihi arvustus ==
== Veebiteenuse 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.  
Ü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 minu meelest liiga keeruliselt teostatud (kasutatud on räsi soolamist ehk ''hash salting''). Mina tegin selle lihtsamalt, kasutades
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()''
<source lang="csharp">
<source lang="csharp">
System.Web.Security.FormsAuthentication.HashPasswordForStoringInConfigFile(pass, "SHA1")
    if (! kont.Kasutajas.Any(user => user.Nimi == userName)) // kui sellise nimega kasutajat pole
</source>
    {
meetodit. Samuti on olemasolevate kasutajanimede kontroll liiga pikalt tehtud. Mina tegin kasutaja registreerimise meetodi nii:
        // siis sisesta kasutaja andmed baasi
<source lang="csharp">
    }
        /// <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;
        }
</source>
</source>


Aga muidu on okei ma arvan :-)
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.


[[User:Itelga|Ilmar Telga]] 21:25, 23 April 2011 (EEST)
[[User:Itelga|Ilmar Telga]] 21:25, 23 April 2011 (EEST)

Revision as of 19:28, 26 April 2011

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)

Retrieved from "https://wiki.itcollege.ee/index.php?title=Talk:Meeskond_Tool&oldid=28592"