https://wiki.itcollege.ee/index.php?action=history&feed=atom&title=Talk%3AFoodCab 2026-05-01T19:37:05Z Revision history for this page on the wiki MediaWiki 1.45.1 https://wiki.itcollege.ee/index.php?title=Talk:FoodCab&diff=117437&oldid=prev 2017-01-31T12:44:45Z

<p><span class="autocomment">Retsensioon projektile</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 15:44, 31 January 2017</td> </tr><tr><td colspan="2" class="diff-lineno" id="mw-diff-left-l11">Line 11:</td> <td colspan="2" class="diff-lineno">Line 11:</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>== Retsensioon projektile ==</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>== Retsensioon projektile ==</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" 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><del style="font-weight: bold; text-decoration: none;">TBD</del></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;">Retsenseerime pigem koodi, kui UI'd. Projekt käivitus ning esialgsel ülevaatusel fataleid  ei leidnud.  Projekti maht on korralik ning vastab aine mahule. </ins></div></td></tr> <tr><td colspan="2" class="diff-side-deleted"></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;">Kood on struktureeritud ning solution jaotatud mitmeks alamprojektiks. Küll ei saa aru, milleks on alles jäetud FoodCabConsole projekt, tegemist tühja projektiga. Domeenimudelid oleks võinud olla ehk eraldi projektis. Vaatemudelid kenasti eraldatud (admin vs klient) ning nimetus ühtlane (lõppevad VM). Samas meetodite nimed on ebaühtse stiiliga - osad nimed algavad kohati suure, kohati väikse tähega. On ka olukordi, kus meetodite  Objekt orienteerituse koha pealt oleks võinud veel tükeldada. Näiteks OrderViewVM::380 hakkab selline koodiosa, mida oleks saanud kirjutada kindlasti efektiivsemalt. Kasutajate paroolid hashitakse, mis annab turvalisusele juurde. Kui koodile peale vaadata, on tunne nagu rakendus poleks kirjutatud Visual Studios, kuna väga palju naminguid ja koodi optimeerimisi, mis Visual Studio pakub on tegemata. </ins></div></td></tr> <tr><td colspan="2" class="diff-side-deleted"></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> </div></td></tr> <tr><td colspan="2" class="diff-side-deleted"></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;">Järgnev on näide, mida oleks saanud paremini lahendada (esines korduvalt):</ins></div></td></tr> <tr><td colspan="2" class="diff-side-deleted"></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;">&lt;code&gt;if (String.IsNullOrEmpty(FirstName))&lt;/code&gt; vs &lt;code&gt;if (FirstName == null || FirstName == String.Empty) &lt;/code&gt;</ins></div></td></tr> <tr><td colspan="2" class="diff-side-deleted"></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> </div></td></tr> <tr><td colspan="2" class="diff-side-deleted"></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;">Kokkuvõttes arvan, et autorid on näinud palju vaeva, et antud rakendus valmis ehitada. Kindlasti ollakse õigel teel, et omandada OOP. </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>Autor: Meeskond [[Autoparandaja]]</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>Autor: Meeskond [[Autoparandaja]]</div></td></tr> </table>

Mviilvee https://wiki.itcollege.ee/index.php?title=Talk:FoodCab&diff=117377&oldid=prev 2017-01-30T07:37:53Z

<p><span class="autocomment">Retsensioon projekti analüüsile</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 10:37, 30 January 2017</td> </tr><tr><td colspan="2" class="diff-lineno" id="mw-diff-left-l6">Line 6:</td> <td colspan="2" class="diff-lineno">Line 6:</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>Rakenduse sisu on ajaga kaasaskäiv, inimesed otsivad mugavusteenuseid ning innovatiivsust oleks sellistesse ettevõtmistesse vaja. Projekti suurus on sobiv sellele ainele ning meeskonna 4 liiget saavad kõik väga hea kogemuse. Vaadates meeskonna blogi soovitame alustada töödega niipea, kui võimalik, sest programmeerides on üllatusi kerge tulema.</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>Rakenduse sisu on ajaga kaasaskäiv, inimesed otsivad mugavusteenuseid ning innovatiivsust oleks sellistesse ettevõtmistesse vaja. Projekti suurus on sobiv sellele ainele ning meeskonna 4 liiget saavad kõik väga hea kogemuse. Vaadates meeskonna blogi soovitame alustada töödega niipea, kui võimalik, sest programmeerides on üllatusi kerge tulema.</div></td></tr> <tr><td colspan="2" class="diff-side-deleted"></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;"></ins></div></td></tr> <tr><td colspan="2" class="diff-side-deleted"></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;">Autor: Meeskond [[Autoparandaja]]</ins></div></td></tr> <tr><td colspan="2" class="diff-side-deleted"></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;"></ins></div></td></tr> <tr><td colspan="2" class="diff-side-deleted"></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;">== Retsensioon projektile ==</ins></div></td></tr> <tr><td colspan="2" class="diff-side-deleted"></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;"></ins></div></td></tr> <tr><td colspan="2" class="diff-side-deleted"></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;">TBD</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>Autor: Meeskond [[Autoparandaja]]</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>Autor: Meeskond [[Autoparandaja]]</div></td></tr> </table>

Mviilvee https://wiki.itcollege.ee/index.php?title=Talk:FoodCab&diff=114354&oldid=prev 2016-11-09T19:55:26Z

<p><span class="autocomment">Retsensioon projekti analüüsile</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 22:55, 9 November 2016</td> </tr><tr><td colspan="2" class="diff-lineno" id="mw-diff-left-l6">Line 6:</td> <td colspan="2" class="diff-lineno">Line 6:</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>Rakenduse sisu on ajaga kaasaskäiv, inimesed otsivad mugavusteenuseid ning innovatiivsust oleks sellistesse ettevõtmistesse vaja. Projekti suurus on sobiv sellele ainele ning meeskonna 4 liiget saavad kõik väga hea kogemuse. Vaadates meeskonna blogi soovitame alustada töödega niipea, kui võimalik, sest programmeerides on üllatusi kerge tulema.</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>Rakenduse sisu on ajaga kaasaskäiv, inimesed otsivad mugavusteenuseid ning innovatiivsust oleks sellistesse ettevõtmistesse vaja. Projekti suurus on sobiv sellele ainele ning meeskonna 4 liiget saavad kõik väga hea kogemuse. Vaadates meeskonna blogi soovitame alustada töödega niipea, kui võimalik, sest programmeerides on üllatusi kerge tulema.</div></td></tr> <tr><td colspan="2" class="diff-side-deleted"></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;"></ins></div></td></tr> <tr><td colspan="2" class="diff-side-deleted"></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;">Autor: Meeskond [[Autoparandaja]]</ins></div></td></tr> </table>

Mviilvee https://wiki.itcollege.ee/index.php?title=Talk:FoodCab&diff=114353&oldid=prev 2016-11-09T19:55:04Z

<p></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 22:55, 9 November 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>Retsensioon meeskonna <del style="font-weight: bold; text-decoration: none;">Autoparandaja poolt:</del></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;">== </ins>Retsensioon <ins style="font-weight: bold; text-decoration: none;">projekti analüüsile ==</ins></div></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><del style="font-weight: bold; text-decoration: none;">TODO</del>.</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> </div></td></tr> <tr><td colspan="2" class="diff-side-deleted"></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;">Projekti analüüs on piisavalt sisukas, et anda üsna hea ülevaade rakenduse funktsionaalsusest. Lugedes tekkis väike soov, et oleks võinud olla natukene paremini liigendatud. Hetkel on jutt väga sisukas ja iga järgnev lause võib alustada juba täiesti teise rakenduse funktsionaalsuse kirjeldamist. On toodud eraldi välja nice to have alampunkt, kuid üks punkt, mis on analüüsi jutu sees seal alampunkti all välja ei tule. Analüüsile on lisatud ka andmemudel, kuid kahjuks on andmemudelist puudu väljade suurused. </ins></div></td></tr> <tr><td colspan="2" class="diff-side-deleted"></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> </div></td></tr> <tr><td colspan="2" class="diff-side-deleted"></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;">Mille kallal natukene nuriseks on see, et hetkel ei saa olla kasutaja 2s rollis. Ehk siis UserType on kirjutatud otse kasutaja andmemudelisse. Samas hoiab see kokku jällegi arendus aega. Tekkis ka küsimus, et kes sisestab MenuItemCategory tabelisse kategooriaid. Kas admin? Kuna antud nimekiri pole kuidagi seotud võiks olla antud list näiteks enum. See hoiaks jällegi aega kokku (teeks ka haldamist lihtsamaks) ning sellest üle jääva aja saaks kasutada muu funktsionaalsuse arendamiseks. Hea tava tellimuste salvestamisel on see, et toote kriitiline info (nt nimi, hind) võiks olla duubeldatud tellimuse toote tabelisse. See väldiks ajalookadu. Ei ole kindel, et süsteemis hoida piirkonna ja addressi paare on hea mõte. Eriti hea plaan ei tundu hoida piirkondade ja aadresside paare andmebaasis. Selleks võiks kasutada välist teenusepakkujat (nt Google maps) (nice to have all kirjas) või siis pigem viia risk kliendile - kui ta tellib kaugemalt, siis läheb kauem aega. </ins></div></td></tr> <tr><td colspan="2" class="diff-side-deleted"></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> </div></td></tr> <tr><td colspan="2" class="diff-side-deleted"></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;">Rakenduse sisu on ajaga kaasaskäiv, inimesed otsivad mugavusteenuseid ning innovatiivsust oleks sellistesse ettevõtmistesse vaja. Projekti suurus on sobiv sellele ainele ning meeskonna 4 liiget saavad kõik väga hea kogemuse. Vaadates </ins>meeskonna <ins style="font-weight: bold; text-decoration: none;">blogi soovitame alustada töödega niipea, kui võimalik, sest programmeerides on üllatusi kerge tulema</ins>.</div></td></tr> </table>

Mviilvee https://wiki.itcollege.ee/index.php?title=Talk:FoodCab&diff=114263&oldid=prev 2016-11-08T20:39:34Z

<p>Created page with &quot;Retsensioon meeskonna Autoparandaja poolt: TODO.&quot;</p> <p><b>New page</b></p><div>Retsensioon meeskonna Autoparandaja poolt:<br /> TODO.</div>

Mviilvee