olegtervo / otm-harjoitustyo Goto Github PK

Latasin annetun projektin 03.05.2018 kello 13:53

• Käyttöliittymä vaikuttaa toimivan niin kuin kuuluu ja en omilla testauksillani ainakaan onnistunut sovellusta kaatamaan.

• En huomannut toisteisuutta kummassakaan sovelluslogiikka luokassa ja metodit pääasiassa niissä vaikuttivat kompakteilta ja siisteiltä.

• Luokan Aikalista metodit on nimetty vähän epäselvästi esim. varauksia voisi olla listaaKayttajanVaraukset tms.

• Luokan Aikalista metodin kaikki koodi on vähän vaikealukuista ja sen voisi mahdollisesti kirjoittaa vähän selvemminkin.

• Luokan Aikalista metodi poistaVaraus on vähän epäselvä, koska siinä ei ole asetettu käsiteltävää käyttäjää muuttujaan, vaikka sitä tarvitaan useammassa kohtaa. Suosittelisin sen muuttujaan asettamista.

• Koodissa on sekaisin suomea ja englantia. Olisi parempaa tyyliä kirjoittaa koko koodi samalla kielellä, niin että koodi olisi yhdenmukaista. Ohjeessa luki, että englanniksi, mutta en ole täysin varma, että käykö myös suomi, niin sitä kannattaa kysyä vielä assarilta.

• Luokassa Kayttaja on kaksi importtia, joita ei käytetä mihinkään.

• Luokan Main metodien varTime, Time ja Times nimet eivät kerro hyvin, mitä metodit tekevät. Niille olisi hyvä keksiä paremmin kuvaavat nimet.

• Mainissa olevat metodit varsinkin start vaikuttavat olevan vähän turhan pitkiä ja niitä voisi varmastikin eriyttää. Ehdottaisin, että kävisit assarin kanssa läpi, ellet osaa itse, että miten niitä tai ainakin metodia start kannattaisi eriyttää järkevästi.

• Jos käytät NetBeansia koodaamiseen, kannattaa katsoa kaikki NetBeansin omat huomautukset läpi, koska osa niistä voi olla hyödyllisiä.

• Testikattavuus näytti olevan hyvä, mutta testit ovat kaikki samassa luokassa ja niiden nimet eivät ilmaise kauhean tarkasti mitä ne tekevät. Koodin laatuvaatimus -ohjeessa myös puhuttiin siitä, että testiluokkien nimien pitäisi vastata sitä luokkaa mitä ne testaavat. Siispä tässä tilanteessa nimien pitäisi olla luokkaa Aikalista testaaville testeille AikalistaTest ja Kayttajaa testaaville KayttajaTest. Nyt kaikki testaus tapahtuu koko projektin nimellä varustetussa testiluokassa TimeAppTest.

• Testipaketin nimen tulisi olla jokin muu kuin default package esimerkiksi timeapp tms. Kannattaa kysyä assarilta, koska ohje oli vähän epäselvä tämän nimeämisen osalta.

Käyttöliittymä näyttää mielestäni tosi hyvältä ja tekee jo sen mitä odotinkin sen tekevän, mikä on tosi hieno juttu. Oli kiva, kun ohjelmaa pystyi oikeasti testaamaan graafisen käyttöliittymän kautta.