Talk:Meeskond: KTM Development - Revision history

Klukas: /* KTM valmistoote retsensioon */

2016-01-27T10:47:18Z

KTM valmistoote retsensioon

← Older revision		Revision as of 13:47, 27 January 2016
Line 129:		Line 129:

	Paraku kõiki ülalnimetatud prototüübi murekohti või soovitusi ei arvestatud, mistõttu jäid need endiselt alles.		Paraku kõiki ülalnimetatud prototüübi murekohti või soovitusi ei arvestatud, mistõttu jäid need endiselt alles.

	Miinused:		Miinused:
	* Suur osa koodi view taga (Peale 'mainwindow' ja 'Projects', kus loodakse view taga vm ja siis pöördutakse selle poole).		* Suur osa koodi view taga (Peale 'mainwindow' ja 'Projects', kus loodakse view taga vm ja siis pöördutakse selle poole).

Klukas: /* KTM valmistoote retsensioon */

2016-01-27T10:46:44Z

KTM valmistoote retsensioon

← Older revision		Revision as of 13:46, 27 January 2016
Line 128:		Line 128:
	* Arvestades aine raamistikku on ülesande püstitus ja teostus olnud		* Arvestades aine raamistikku on ülesande püstitus ja teostus olnud

	~~Negatiivid~~:		Paraku kõiki ülalnimetatud prototüübi murekohti või soovitusi ei arvestatud, mistõttu jäid need endiselt alles.
			Miinused:
	* Suur osa koodi view taga (Peale 'mainwindow' ja 'Projects', kus loodakse view taga vm ja siis pöördutakse selle poole).		* Suur osa koodi view taga (Peale 'mainwindow' ja 'Projects', kus loodakse view taga vm ja siis pöördutakse selle poole).
	* Korduvat koodi näiteks väljalogimisel, kus oli kaks funktsiooni sama sisuga, kuigi erineva event-i peale. Oleks võinud mõlema korral kolmanda funktsiooni välja kutsuda.		* Korduvat koodi näiteks väljalogimisel, kus oli kaks funktsiooni sama sisuga, kuigi erineva event-i peale. Oleks võinud mõlema korral kolmanda funktsiooni välja kutsuda.
Line 140:		Line 141:
	* Rakenduses puudub kasutajate haldus.		* Rakenduses puudub kasutajate haldus.

	Kokkuvõttes arvestades aine raamistikku, on ülesande püstitus ja teostus olnud head.		Kokkuvõttes arvestades aine raamistikku, on ülesande püstitus ja teostus olnud head ning võrreldes prototüüpi lõpptootega, on näha suurt arengut.

Klukas: /* KTM valmistoote retsensioon */

2016-01-27T10:27:05Z

KTM valmistoote retsensioon

← Older revision		Revision as of 13:27, 27 January 2016
Line 131:		Line 131:
	* Suur osa koodi view taga (Peale 'mainwindow' ja 'Projects', kus loodakse view taga vm ja siis pöördutakse selle poole).		* Suur osa koodi view taga (Peale 'mainwindow' ja 'Projects', kus loodakse view taga vm ja siis pöördutakse selle poole).
	* Korduvat koodi näiteks väljalogimisel, kus oli kaks funktsiooni sama sisuga, kuigi erineva event-i peale. Oleks võinud mõlema korral kolmanda funktsiooni välja kutsuda.		* Korduvat koodi näiteks väljalogimisel, kus oli kaks funktsiooni sama sisuga, kuigi erineva event-i peale. Oleks võinud mõlema korral kolmanda funktsiooni välja kutsuda.
	* Peale registreerimise ja sisselogimise ei saanud edasi testida, kuna uue projekti loomisel annab veateate. (käsitsi andmebaasi muutes sai)		* Peale registreerimise ja sisselogimise ei saanud edasi testida, kuna uue projekti loomisel annab veateate (käsitsi andmebaasi muutes sai).
	* Projektis on läbivalt ~~kasutud~~ inglise ja eesti keele segu.		* Projektis on läbivalt kasutatud inglise ja eesti keele segu.
	* Osades klassides on konstruktorid koodi lõpus, selgema arusaamise mõistes võiksid need olla esimesed, mitte peale gettereid ja settereid.		* Osades klassides on konstruktorid koodi lõpus, selgema arusaamise mõistes võiksid need olla esimesed, mitte peale gettereid ja settereid.
	* Projekti nime ja kirjeldust ei ole võimalik muuta.		* Projekti nime ja kirjeldust ei ole võimalik muuta.

Klukas: /* KTM valmistoote retsensioon */

2016-01-27T10:23:03Z

KTM valmistoote retsensioon

← Older revision		Revision as of 13:23, 27 January 2016
Line 131:		Line 131:
	* Suur osa koodi view taga (Peale 'mainwindow' ja 'Projects', kus loodakse view taga vm ja siis pöördutakse selle poole).		* Suur osa koodi view taga (Peale 'mainwindow' ja 'Projects', kus loodakse view taga vm ja siis pöördutakse selle poole).
	* Korduvat koodi näiteks väljalogimisel, kus oli kaks funktsiooni sama sisuga, kuigi erineva event-i peale. Oleks võinud mõlema korral kolmanda funktsiooni välja kutsuda.		* Korduvat koodi näiteks väljalogimisel, kus oli kaks funktsiooni sama sisuga, kuigi erineva event-i peale. Oleks võinud mõlema korral kolmanda funktsiooni välja kutsuda.
	* Peale registreerimise ja sisselogimise ei saanud edasi testida, kuna uue projekti loomisel annab veateate.		* Peale registreerimise ja sisselogimise ei saanud edasi testida, kuna uue projekti loomisel annab veateate. (käsitsi andmebaasi muutes sai)
	* Projektis on läbivalt kasutud inglise ja eesti keele segu.		* Projektis on läbivalt kasutud inglise ja eesti keele segu.
	* Osades klassides on konstruktorid koodi lõpus, selgema arusaamise mõistes võiksid need olla esimesed, mitte peale gettereid ja settereid.		* Osades klassides on konstruktorid koodi lõpus, selgema arusaamise mõistes võiksid need olla esimesed, mitte peale gettereid ja settereid.
Line 140:		Line 140:
	* Rakenduses puudub kasutajate haldus.		* Rakenduses puudub kasutajate haldus.

	~~Kokkuvõtes~~ arvestades aine raamistikku on ülesande püstitus ja teostus olnud head.		Kokkuvõttes arvestades aine raamistikku, on ülesande püstitus ja teostus olnud head.

Miraudse: /* KTM valmistoote retsensioon */

2016-01-27T10:16:44Z

KTM valmistoote retsensioon

Miraudse: /* KTM valmistoote retsensioon */

2016-01-27T10:15:17Z

KTM valmistoote retsensioon

← Older revision		Revision as of 13:15, 27 January 2016
Line 120:		Line 120:

	Plussid:		Plussid:
	*Õpetus olemas.		+ Õpetus olemas.
	*Korralikult kommenteeritud.		+ Korralikult kommenteeritud.
	*Hästi struktueeritud (kuigi mingipärast pole view-d 'Views' kaustas).		+ Hästi struktueeritud (kuigi mingipärast pole view-d 'Views' kaustas).
	*Registreerimisel kontrollid ja arusaadavad veateated.		+ Registreerimisel kontrollid ja arusaadavad veateated.
			+ Hea välimus.
			+ Kasutusjuhendi viit on olemas kohe rakenduse sees.
			+ Arvestades aine raamistikku on ülesande püstitus ja teostus olnud

	Negatiivid:		Negatiivid:
	*Suur osa koodi view taga (Peale 'mainwindow' ja 'Projects', kus loodakse view taga vm ja siis pöördutakse selle poole).		- Suur osa koodi view taga (Peale 'mainwindow' ja 'Projects', kus loodakse view taga vm ja siis pöördutakse selle poole).
	*Korduvat koodi näiteks väljalogimisel, kus oli kaks funktsiooni sama sisuga, kuigi erineva event-i peale. Oleks võinud mõlema korral kolmanda funktsiooni välja kutsuda.		- Korduvat koodi näiteks väljalogimisel, kus oli kaks funktsiooni sama sisuga, kuigi erineva event-i peale. Oleks võinud mõlema korral kolmanda funktsiooni välja kutsuda.
	*Peale registreerimise ja sisselogimise ei saanud edasi testida, kuna uue projekti loomisel annab veateate.		- Peale registreerimise ja sisselogimise ei saanud edasi testida, kuna uue projekti loomisel annab veateate.
			- Projektis on läbivalt kasutud inglise ja eesti keele segu.
			- Osades klassides on konstruktorid koodi lõpus, selgema arusaamise mõistes võiksid need olla esimesed, mitte peale gettereid ja settereid.
			- Projekti nime ja kirjeldust ei ole võimalik muuta.
			- Projektile võiks saada lisada liikmeid ka projekti vaates.
			- Rakendus ei ole skaleeruv ehk suurust ei saa muuta ja kujunduses ei ole kasutatud % väärtuseid.
			- Projekti vaates võiks ära kaotada nupu "värskenda" ja lahendada projekti nimistu uuendamine bindinguga.
			- Rakenduses puudub kasutajate haldus.

			Kokkuvõtes arvestades aine raamistikku on ülesande püstitus ja teostus olnud head.

Aameerik: /* KTM valmistoote retsensioon */

2016-01-26T20:06:14Z

KTM valmistoote retsensioon

← Older revision		Revision as of 23:06, 26 January 2016
Line 118:		Line 118:

	Prototüübi plussid säilisid, seega kena disain, andmebaas, projekti jagatus on kõik plussid.		Prototüübi plussid säilisid, seega kena disain, andmebaas, projekti jagatus on kõik plussid.

	Plussid:		Plussid:
	*Õpetus olemas.		*Õpetus olemas.

Aameerik at 20:05, 26 January 2016

2016-01-26T20:05:53Z

← Older revision		Revision as of 23:05, 26 January 2016
Line 116:		Line 116:

	=KTM valmistoote retsensioon=		=KTM valmistoote retsensioon=

			Prototüübi plussid säilisid, seega kena disain, andmebaas, projekti jagatus on kõik plussid.
			Plussid:
			*Õpetus olemas.
			*Korralikult kommenteeritud.
			*Hästi struktueeritud (kuigi mingipärast pole view-d 'Views' kaustas).
			*Registreerimisel kontrollid ja arusaadavad veateated.

			Negatiivid:
			*Suur osa koodi view taga (Peale 'mainwindow' ja 'Projects', kus loodakse view taga vm ja siis pöördutakse selle poole).
			*Korduvat koodi näiteks väljalogimisel, kus oli kaks funktsiooni sama sisuga, kuigi erineva event-i peale. Oleks võinud mõlema korral kolmanda funktsiooni välja kutsuda.
			*Peale registreerimise ja sisselogimise ei saanud edasi testida, kuna uue projekti loomisel annab veateate.

Vrokovan: /* KTM prototüübi retsensioon */

2016-01-26T19:57:21Z

KTM prototüübi retsensioon

← Older revision		Revision as of 22:57, 26 January 2016
Line 114:		Line 114:

	Prototüübi jaoks natuke vähe tehtud - oleks võinud luua vajalikud vaated, et saada parem ülevaade sellest, kuidas projekti tahetakse realiseerida.		Prototüübi jaoks natuke vähe tehtud - oleks võinud luua vajalikud vaated, et saada parem ülevaade sellest, kuidas projekti tahetakse realiseerida.

			=KTM valmistoote retsensioon=

Klukas: /* KTM prototüübi retsensioon */

2015-12-20T21:31:01Z

KTM prototüübi retsensioon

← Older revision		Revision as of 00:31, 21 December 2015
Line 85:		Line 85:

	*XAML		*XAML
	**Tõsta vaated eraldi kausta, aitab ~~arusaadavusel~~.		**Tõsta vaated eraldi kausta, aitab paremini aru saada.
	**CodeBehind-is olevad päringud üle viia Service-te alla		**CodeBehind-is olevad päringud üle viia Service-te alla
	Näiteks:		Näiteks:
	“bool emailOlemas = db.KASUTAJASet.Any(kasutaja => kasutaja.Email.Equals(txtEmail.Text));” päringu koht oleks ~~sammuti~~ KasutajaService klassis ja näiteks “IsUniqueEmail(txtEmail.Text);” meetodi sees.		“bool emailOlemas = db.KASUTAJASet.Any(kasutaja => kasutaja.Email.Equals(txtEmail.Text));” päringu koht oleks samuti KasutajaService klassis ja näiteks “IsUniqueEmail(txtEmail.Text);” meetodi sees.
	**Pole kasutatud Grid-i ~~võimalusi~~ laiuse ja kõrguse määramiseks joondamisel. Hetkel käe järgi sätitud elemendid.		**Pole kasutatud dünaamilist disaini ehk kasutamata Grid-i võimalused laiuse ja kõrguse määramiseks joondamisel. Hetkel käe järgi sätitud elemendid.
	**Registreerimise vormil peaks olema “tühista” vasakul ja ”registreeri” paremal~~. Nii~~ nagu tava ette näeb. Käsu täitmise nupp ~~paremal pool~~ teistest.		**Registreerimise vormil peaks olema “tühista” vasakul ja ”registreeri” paremal, nagu tava ette näeb. Käsu täitmise nupp paremalpool teistest.
	*~~Salasõnad on~~ andmebaasis lihttekstina.		*Salasõnasid hoitakse andmebaasis lihttekstina.
	*ViewModel-id on tühjad, praegu asub funktsionaalsus vaadete taga.		*ViewModel-id on tühjad, praegu asub funktsionaalsus vaadete taga.
	*Funktsionaalsus:		*Funktsionaalsus:
	**Registreerimisel tuleb kohe emaili kontrollimisel veateade, mis programmi kokku jooksutab.		**Registreerimisel tuleb kohe emaili kontrollimisel veateade, mis programmi kokku jooksutab.
	**Lisades uut projekti ei uuendata projektide nimekirja automaatselt, vaid selleks peab eraldi nuppu kasutama.		**Lisades uut projekti ei uuendata projektide nimekirja automaatselt, vaid selleks peab eraldi nuppu kasutama.
	**Andmebaasi lisades genereeritakse ise ID viimase sisestuse järgi~~. ID~~ peaks olemas autogenerated.		**Andmebaasi lisades genereeritakse ise ID viimase sisestuse järgi - peaks olemas autogenerated.
	*Pikad lohisevad "and if" kontrollid		*Pikad lohisevad "and-if" kontrollid
	Näiteks: “if(!string.IsNullOrEmpty(txtEesnimi.Text) && !string.IsNullOrEmpty(txtPerenimi.Text) && !string.IsNullOrEmpty(txtPwd.Password) && !string.IsNullOrEmpty(txtEmail.Text) && txtPwd.Password == txtPwd2.Password)”		Näiteks: “if(!string.IsNullOrEmpty(txtEesnimi.Text) && !string.IsNullOrEmpty(txtPerenimi.Text) && !string.IsNullOrEmpty(txtPwd.Password) && !string.IsNullOrEmpty(txtEmail.Text) && txtPwd.Password == txtPwd2.Password)”

Line 104:		Line 104:
	*Ühtlasem keelekasutus		*Ühtlasem keelekasutus
	**Andmebaas ja suur osa funktsioonidest on eesti keeles kirjutatud.		**Andmebaas ja suur osa funktsioonidest on eesti keeles kirjutatud.
	**ProjectsVM is leiduvad funktsioonid loadDataFromDB(), laemeilid(), lisaProjektiMeeskonda(string email). Kõigil kolmel on erinev tava.		**ProjectsVM-is leiduvad funktsioonid loadDataFromDB(), laemeilid(), lisaProjektiMeeskonda(string email). Kõigil kolmel on erinev tava.
	**~~Funktsiooni sisesed~~ muutujad on ~~osades kohates~~ imeliku nimega, arvatavasti on kopeeritud kuskilt mujalt ja jäänud muutmata.		**Funktsioonisisesed muutujad on kohati imeliku nimega, arvatavasti on kopeeritud kuskilt mujalt ja jäänud muutmata.

	==Kokkuvõte==		==Kokkuvõte==

	Näha et hetkel ~~ei ole projektiga palju tegeletud~~. ~~Põhi kihis~~ on loodud neli vaadet ning suurem osa loogikast on CodeBehindis.		Näha, et hetkel on projekt väga algusjärgus. Põhikihis on loodud neli vaadet ning suurem osa loogikast on CodeBehindis.
	Disain on kena, kuid tähtsam on saada projekt liikuma~~. Iluga~~ saab lõpus tegeleda. ~~Organiseerida~~ vaated, tõsta funktsionaalsused ~~Vied~~-de tagant ära.		Disain on kena, kuid tähtsam on saada projekt liikuma, iluga saab lõpus tegeleda. Soovitaks organiseerida vaated, tõsta funktsionaalsused View-de
			tagant ära.

	Prototüübi jaoks natuke vähe tehtud~~. Pigem~~ oleks võinud ~~ära~~ luua vajalikud vaated, et saada parem ülevaade sellest, kuidas ~~projekt~~ realiseerida ~~tahetakse~~.		Prototüübi jaoks natuke vähe tehtud - oleks võinud luua vajalikud vaated, et saada parem ülevaade sellest, kuidas projekti tahetakse realiseerida.

← Older revision		Revision as of 13:16, 27 January 2016
Line 120:		Line 120:

	Plussid:		Plussid:
	+ Õpetus olemas.		* Õpetus olemas.
	+ Korralikult kommenteeritud.		* Korralikult kommenteeritud.
	+ Hästi struktueeritud (kuigi mingipärast pole view-d 'Views' kaustas).		* Hästi struktueeritud (kuigi mingipärast pole view-d 'Views' kaustas).
	+ Registreerimisel kontrollid ja arusaadavad veateated.		* Registreerimisel kontrollid ja arusaadavad veateated.
	+ Hea välimus.		* Hea välimus.
	+ Kasutusjuhendi viit on olemas kohe rakenduse sees.		* Kasutusjuhendi viit on olemas kohe rakenduse sees.
	+ Arvestades aine raamistikku on ülesande püstitus ja teostus olnud		* Arvestades aine raamistikku on ülesande püstitus ja teostus olnud

	Negatiivid:		Negatiivid:
	- Suur osa koodi view taga (Peale 'mainwindow' ja 'Projects', kus loodakse view taga vm ja siis pöördutakse selle poole).		* Suur osa koodi view taga (Peale 'mainwindow' ja 'Projects', kus loodakse view taga vm ja siis pöördutakse selle poole).
	- Korduvat koodi näiteks väljalogimisel, kus oli kaks funktsiooni sama sisuga, kuigi erineva event-i peale. Oleks võinud mõlema korral kolmanda funktsiooni välja kutsuda.		* Korduvat koodi näiteks väljalogimisel, kus oli kaks funktsiooni sama sisuga, kuigi erineva event-i peale. Oleks võinud mõlema korral kolmanda funktsiooni välja kutsuda.
	- Peale registreerimise ja sisselogimise ei saanud edasi testida, kuna uue projekti loomisel annab veateate.		* Peale registreerimise ja sisselogimise ei saanud edasi testida, kuna uue projekti loomisel annab veateate.
	- Projektis on läbivalt kasutud inglise ja eesti keele segu.		* Projektis on läbivalt kasutud inglise ja eesti keele segu.
	- Osades klassides on konstruktorid koodi lõpus, selgema arusaamise mõistes võiksid need olla esimesed, mitte peale gettereid ja settereid.		* Osades klassides on konstruktorid koodi lõpus, selgema arusaamise mõistes võiksid need olla esimesed, mitte peale gettereid ja settereid.
	- Projekti nime ja kirjeldust ei ole võimalik muuta.		* Projekti nime ja kirjeldust ei ole võimalik muuta.
	- Projektile võiks saada lisada liikmeid ka projekti vaates.		* Projektile võiks saada lisada liikmeid ka projekti vaates.
	- Rakendus ei ole skaleeruv ehk suurust ei saa muuta ja kujunduses ei ole kasutatud % väärtuseid.		* Rakendus ei ole skaleeruv ehk suurust ei saa muuta ja kujunduses ei ole kasutatud % väärtuseid.
	- Projekti vaates võiks ära kaotada nupu "värskenda" ja lahendada projekti nimistu uuendamine bindinguga.		* Projekti vaates võiks ära kaotada nupu "värskenda" ja lahendada projekti nimistu uuendamine bindinguga.
	- Rakenduses puudub kasutajate haldus.		* Rakenduses puudub kasutajate haldus.

	Kokkuvõtes arvestades aine raamistikku on ülesande püstitus ja teostus olnud head.		Kokkuvõtes arvestades aine raamistikku on ülesande püstitus ja teostus olnud head.