Talk:Meeskond "Terav ASP"

From ICO wiki
Revision as of 11:39, 22 December 2010 by Alumberg (talk | contribs)
Jump to navigationJump to search

1.

Väga terav! Just nii peakski tiimitööd tegema! Meeskond on töötanud koos ja koordineeritult (projektijuhtimine!) ja liikmete tegevused on kohtumistelogi/committide abil hästi näha. Rakendusel võiks olla ka päriselt potentsiaali - eriti on vaeva nähtud otsinguga (vabatekstiotsing, sorditav igapidi, mõistlikud kontrollid jne). See osa koodist (MaterjalController.cs) on ka kõige põhjalikumalt kommenteeritud. Kasutajate/materjalide haldamiseks ja tegevuste logi vaatamiseks on loodud Admin kasutaja. Tegelikult võiks selle kohta wiki lehele vihje panna;)

Kodutöö nõuete täidetusega ei tohiks küsimust tekkida, ent kui rakendust sihipäraselt kasutama hakata, siis võiks lisada mõned võdurid:

  • Materjale/dokumente võiks saada mugavalt kustutada admini alt.
  • Uue faili laadimisel võiks olla kinnitus õnnestumise kohta. Samuti kasutaja kustutamisel
  • Õppeainete haldus ei jäänud silma - ilmselt võiks ka neid Admin hallata?
  • kommentaaride lehte saab võibolla paremini kujundada.
  • logi näitab vaid kasutajate sisse-väljalogimist, võiks näidata ka dokumentide üleslaadimist ehk.

Silma jäi ka paar pisikest probleemi:

  • Uue faili laadimisel: aasta välja üritatakse int-iks teisendada ilma lisakontrollideta - kui sinna mitte-numbriline väärtus sisestada (või tühjaks jätta), siis läheb leht katki. Mujal on kontrollid ilusasti tehtud, seega ju on lihtsalt ununenud:)
  • Admin saki võiks tavakasutajate eest ära peita....
  • veerg "viimati aktiivne" (admin lehel) näitab valet kuupäeva (logi avades on tegevuste kuupäevad õiged). Ma tegin oma kasutaja eile, ja viimati logisin sisse-välja täna. "viimati aktiivne" all on eilne kuupäev.

Ivo Lõhmus, AK31, 10.12.2010



Team on väga hea programmiga hästi hakkama saanud! Seletatud olid kõik njuantsid , küsimuseid ei olnud kuna kõik asjad olid hästi arusaanud.Kuna asjad on siis põhimõttelised uued saamegi öelda , et Teras ASP on väga hea näide C#.

Team PEA


Väga hea programm ja ülikasulik, selline asi võiks tulevikus olla iga kooli kõrval. Silma ilu oli ka olemas ja kindlasti peab mainima ära, nagu Ivogi ees, et hämmastav on selle tiimi põhjalikkus ja grupitöö teguviis. Väga tublid.

Liis Merisaar


Failide üles laadimine töötab kenasti, aga kasutajana ei leidnud võimalust üles laetud faile kustutada. Koodist näen, et see peaks käima Admin lehelt, aga kuidas ma admin kasutajaks saan? Failide alla laadimine lõppes server erroriga, võimalik, et kohalik IIS vajab eriseadistust. Otsing töötab, aga kui vajutada "otsi!" nuppu ilma välju täitmata, kuvatakse "SELECT [t0].[ID], [t0].[Nimetus], [t0].[KoostamiseAasta], [t0].[LaadimiseKp], [t0].[KasutajaID], [t0].[KommentaarideArv] FROM [dbo].[Dokumendid] AS [t0] ".

Kujundus on silmasõbralik ja intuitiivne.

Ulvar Petmanson, AK22

---

Funktsionaalsus on lahe ja idee on tore. Otsingu lehel ei suutnud kasutada sorteerimist - minu jaoks lõppes see tulemuste kadumise ja tühja otsinguvormiga. Kood on läbisegi eesti ja inglise keeles - võimalik, et polnud stiilis omavahel lõpuni kokku lepitud. Kommentaaride arv ei muutu kommentaaride lisandudes. Vormides välju tühjaks jättes kipub vigu lendama. Kripeldama jäi static muutujate kasutamine controllerites, ehkki ei osanud kiirel katsetamisel mingeid sessioonidevahelisi vigu esile manada... Kiidan kõvasti wikit, mis on informatiivne ja põhjalik. Paistab ka, et tiimitöösse olid kõik kaasatud.

Alvar Lumberg, DK21