https://wiki.itcollege.ee/index.php?action=history&feed=atom&title=Meeskonna_IncMan_protot%C3%BC%C3%BCbi_retsensioon 2026-05-07T02:32:17Z Revision history for this page on the wiki MediaWiki 1.45.1 https://wiki.itcollege.ee/index.php?title=Meeskonna_IncMan_protot%C3%BC%C3%BCbi_retsensioon&diff=100197&oldid=prev 2016-01-09T16:59:17Z

<p><span class="autocomment">IncMan prototüübi retsensioon</span></p> <table style="background-color: #fff; color: #202122;" data-mw="interface"> <col class="diff-marker" /> <col class="diff-content" /> <col class="diff-marker" /> <col class="diff-content" /> <tr class="diff-title" lang="en"> <td colspan="2" style="background-color: #fff; color: #202122; text-align: center;">← Older revision</td> <td colspan="2" style="background-color: #fff; color: #202122; text-align: center;">Revision as of 19:59, 9 January 2016</td> </tr><tr><td colspan="2" class="diff-lineno" id="mw-diff-left-l1">Line 1:</td> <td colspan="2" class="diff-lineno">Line 1:</td></tr> <tr><td class="diff-marker" data-marker="−"></td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #ffe49c; vertical-align: top; white-space: pre-wrap;"><div>== IncMan prototüübi retsensioon ==</div></td><td class="diff-marker" data-marker="+"></td><td style="color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #a3d3ff; vertical-align: top; white-space: pre-wrap;"><div>== <ins style="font-weight: bold; text-decoration: none;">Meeskonna </ins>IncMan prototüübi retsensioon <ins style="font-weight: bold; text-decoration: none;">meeskonna Bob? poolt </ins>==</div></td></tr> <tr><td class="diff-marker"></td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"><br></td><td class="diff-marker"></td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"><br></td></tr> <tr><td class="diff-marker"></td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"><div>Üldiselt on rakenduse prototüüp korralikult tehtud ja kood on ilusti kihtidesse jaotatud. Siiski tooks välja mõned puudujäägid koodis.</div></td><td class="diff-marker"></td><td style="background-color: #f8f9fa; color: #202122; font-size: 88%; border-style: solid; border-width: 1px 1px 1px 4px; border-radius: 0.33em; border-color: #eaecf0; vertical-align: top; white-space: pre-wrap;"><div>Üldiselt on rakenduse prototüüp korralikult tehtud ja kood on ilusti kihtidesse jaotatud. Siiski tooks välja mõned puudujäägid koodis.</div></td></tr> </table>

Paarne https://wiki.itcollege.ee/index.php?title=Meeskonna_IncMan_protot%C3%BC%C3%BCbi_retsensioon&diff=100196&oldid=prev 2016-01-09T16:57:43Z

<p>Created page with &quot;== IncMan prototüübi retsensioon == Üldiselt on rakenduse prototüüp korralikult tehtud ja kood on ilusti kihtidesse jaotatud. Siiski tooks välja mõned puudujäägid ko...&quot;</p> <p><b>New page</b></p><div>== IncMan prototüübi retsensioon ==<br /> <br /> Üldiselt on rakenduse prototüüp korralikult tehtud ja kood on ilusti kihtidesse jaotatud. Siiski tooks välja mõned puudujäägid koodis.<br /> <br /> <br /> Esimene asi, mis silma jäi, oli BO-des olev ilma parameetriteta konstruktor, mis andis property&#039;tele vaikimisi väärtused. Minu arust seda vaja ei ole, kuna uut BO-d tehes saab ta alati väärtused andmebaasi objektist. Neid konstruktoreid nagunii ei kasutatud.<br /> <br /> <br /> Veel jäi silma selline rida klassist LogiService:<br /> <br /> if(id==1) //id: 1 - muutmine, 2 -lisamine<br /> <br /> Muutuja id asemel võiks teha enumi, millel on kaks väärtust, näiteks MUUDA ja LISA. Sellega oleks teisel programmeerijal palju lihtsam aru saada, millega tegemist on. Sellisel juhul muutuks see rida näiteks järgmiseks:<br /> <br /> if(tegevus == tegevus.MUUDA)<br /> <br /> Nüüd on arusaadav, mis toimub ja pole vaja kommentaare igale poole juurde lisada, et näidata, mida see teeb.<br /> <br /> <br /> Klassis IntsidentService on arvatavasti copy-pastega kaasa tulnud IntsidentBO property&#039;d. Need võiks ära kustutada.<br /> <br /> <br /> Projekt võiks ka rohkem importe kasutada, mitte pikalt välja kirjutada, kust iga meetod pärit on. See parandab loetavust ja teeb koodi lihtsamalt hallatavamaks. Näiteks selline rida klassis KasutajaService:<br /> <br /> Incman.BLL.Services.LogiService.lisaLogi()<br /> <br /> muutuks selliseks:<br /> <br /> LogiService.lisaLogi()<br /> <br /> Kui klassi importida Incman.BLL.Services.LogiService.<br /> <br /> <br /> Rakendus on suures osas kirjutatud eesti keeles, kuid osades kohtas on kasutatud ka inglise keelt. Kõik võiks olla samas keeles kirjtuatud, siis on kogu kood ühtne. Näiteks klassi SisseLogimineService meetodi muudaParool() sisendparameetrid:<br /> <br /> public static void muudaParool(int kasutajaId, string password)<br /> <br /> võiks olla:<br /> <br /> public static void muudaParool(int kasutajaId, string parool)<br /> <br /> <br /> Klassides IntsidentModel ja KontaktModel on kasutuses üleliigsed muutujad. Nendel mudelitel on juba olemas property&#039;d getterite ja setteritega.</div>

Paarne