Harjoitustyö Lähijunat ladattu torstana 23.4.2020 klo 11:30.
Oikein hieno ja ainakin minun näkökulmastani melko monimutkainen sovellus. Vaikuttaa myös hyvin toimivalta jo tässä vaiheessa. Sisältää jonkun verran itselleni tuntemattomampaa tekniikkaa, johon täydelliseen perehtymiseen voisi helposti kulua koko viikko tai enemmänkin, eikä tähän ole valitettavasti aikaa. Niinpä täytyy antaa palautetta hieman pinnallisemmista asioista, kun osaamiseni ei ole riittävää syvällisempään koodin analysointiin. Koodi kuitenkin näyttää hyvin asialliselta ja selkeältä.
-Sovellus täyttää vaatimusmäärittelyssä suunnitellut toiminnallisuudet jo tässä vaiheessa viimeistä lukuunottamatta.
-Testit ovat default package -kansiossa, vastoin ohjetta. Ohjelmakoodi taas on hyvin jaettu kuvaavasti nimettyihin kansioihin. Tämän voisi korjata.
-Luokat, muuttujat, metodit yms. on hienosti ja selkeästi nimetty.
-Metodit ovat riittävän pieniä ja selkeitä, tekevät yhden asian.
-Minun valitsemani juna näkyi ja liikkui punaisena pisteenä junan omalla välilehdellä, mutta jostain syystä en nähnyt ko. junaa vastaavaa pistettä kartalla jossa näkyivät kaikki junat. Toisen valitsemani junan kohdalla taas se näkyi myös kaikkien junien kartalla.
-Dokumentoinnissa useampia kirjoitusvirheitä joissa yksi merkki puuttuu tai kirjaimet väärinpäin, nämä voisi korjata lopulliseen versioon.
-Kaukaisempia lähijuna-asemia puuttuu valikosta, voisiko ne lisätä? Itse asun Lahdessa joten yritin katsoa miten Z-junat lähtee, mutta Lahti ja muut pohjoisemmat asemat puuttuivat.
-Vastaavissa sovelluksissa on nähty liikkuvien pisteiden sijaan ainakin vetureita. Voisiko siihen ladata minkä tahansa käyttäjän valitseman hassun hahmon junaa mallintamaan?
-Itse koodia ja arkkitehtuuria en osaa parantaa, se näyttää minusta erittäin hyvältä.
armijuha
commented
4 years ago
Harjoitustyö Lähijunat ladattu torstana 23.4.2020 klo 11:30.
Oikein hieno ja ainakin minun näkökulmastani melko monimutkainen sovellus. Vaikuttaa myös hyvin toimivalta jo tässä vaiheessa. Sisältää jonkun verran itselleni tuntemattomampaa tekniikkaa, johon täydelliseen perehtymiseen voisi helposti kulua koko viikko tai enemmänkin, eikä tähän ole valitettavasti aikaa. Niinpä täytyy antaa palautetta hieman pinnallisemmista asioista, kun osaamiseni ei ole riittävää syvällisempään koodin analysointiin. Koodi kuitenkin näyttää hyvin asialliselta ja selkeältä.
-Sovellus täyttää vaatimusmäärittelyssä suunnitellut toiminnallisuudet jo tässä vaiheessa viimeistä lukuunottamatta. -Testit ovat default package -kansiossa, vastoin ohjetta. Ohjelmakoodi taas on hyvin jaettu kuvaavasti nimettyihin kansioihin. Tämän voisi korjata. -Luokat, muuttujat, metodit yms. on hienosti ja selkeästi nimetty. -Metodit ovat riittävän pieniä ja selkeitä, tekevät yhden asian. -Minun valitsemani juna näkyi ja liikkui punaisena pisteenä junan omalla välilehdellä, mutta jostain syystä en nähnyt ko. junaa vastaavaa pistettä kartalla jossa näkyivät kaikki junat. Toisen valitsemani junan kohdalla taas se näkyi myös kaikkien junien kartalla. -Dokumentoinnissa useampia kirjoitusvirheitä joissa yksi merkki puuttuu tai kirjaimet väärinpäin, nämä voisi korjata lopulliseen versioon. -Kaukaisempia lähijuna-asemia puuttuu valikosta, voisiko ne lisätä? Itse asun Lahdessa joten yritin katsoa miten Z-junat lähtee, mutta Lahti ja muut pohjoisemmat asemat puuttuivat. -Vastaavissa sovelluksissa on nähty liikkuvien pisteiden sijaan ainakin vetureita. Voisiko siihen ladata minkä tahansa käyttäjän valitseman hassun hahmon junaa mallintamaan? -Itse koodia ja arkkitehtuuria en osaa parantaa, se näyttää minusta erittäin hyvältä.