Luokat, metodit, attribuutit, parametrit ja muuttujat on nimetty kuvaavasti ja johdonmukaisesti Clean Code -periaatteiden mukaan. Metodien vastuualueet vaikuttaisivat olevan selkeästi rajatut ja noudattavan Single Responsibility -periaatetta. Käyttöliittymän tiedostossa EditPane.java oli pari todella pitkää metodia, mutta näidenkin vastuualueet olivat selkeästi rajatut ja koodi oli jäsennelty niin, että ainakaan itselleni ei ollut vaikeuksia seurata sitä. Projektin pakkausrakennekin on selkeä ja helposti navigoitava. Hyvää työtä!
Palaute liittyen ohjelman toimivuuteen valmiiksi raportoitujen toiminnallisuuksien suhteen:
Testatessani ohjelmaa, uusien huoltotoimenpiteiden luonti onnistui ja ne näkyivät tämän jälkeen käyttöliittymän View-näkymässä. Yrittäessäni luoda uutta huoltosuunnitelmaa, tämä ei kuitenkaan jostain syystä onnistunut. Uusi huoltosuunnitelma ei tullut näkyviin käyttöliittymän File-näkymään, kun valitsin "Select maintenance file".
Palaute liittyen käyttöliittymään/käytettävyyteen:
Käyttöliittymäluonnoksessa havainnollistettu idea käyttöliittymästä on mielestäni järkevä. Sovellusta testatessani olisin toivonut jonkinlaista palautetta sovellukselta, esim. uutta huoltosuunnitelmaa tai huoltotoimenpidettä luodessa siitä, että luonti onnistui. Uuden huoltotoimenpiteen luomisnäkymässä oli myös hieman hämmentävästi valinta, että milloin huoltotoimenpide on valmistunut, vaikka sitä ei oltu vielä luotu.
Zatyri
commented
2 years ago
Projekti ladattu 01.05.2022 klo 17:00.
Palaute liittyen ohjelmakoodiin:
Luokat, metodit, attribuutit, parametrit ja muuttujat on nimetty kuvaavasti ja johdonmukaisesti Clean Code -periaatteiden mukaan. Metodien vastuualueet vaikuttaisivat olevan selkeästi rajatut ja noudattavan Single Responsibility -periaatetta. Käyttöliittymän tiedostossa EditPane.java oli pari todella pitkää metodia, mutta näidenkin vastuualueet olivat selkeästi rajatut ja koodi oli jäsennelty niin, että ainakaan itselleni ei ollut vaikeuksia seurata sitä. Projektin pakkausrakennekin on selkeä ja helposti navigoitava. Hyvää työtä!
Palaute liittyen ohjelman toimivuuteen valmiiksi raportoitujen toiminnallisuuksien suhteen:
Testatessani ohjelmaa, uusien huoltotoimenpiteiden luonti onnistui ja ne näkyivät tämän jälkeen käyttöliittymän View-näkymässä. Yrittäessäni luoda uutta huoltosuunnitelmaa, tämä ei kuitenkaan jostain syystä onnistunut. Uusi huoltosuunnitelma ei tullut näkyviin käyttöliittymän File-näkymään, kun valitsin "Select maintenance file".
Palaute liittyen käyttöliittymään/käytettävyyteen:
Käyttöliittymäluonnoksessa havainnollistettu idea käyttöliittymästä on mielestäni järkevä. Sovellusta testatessani olisin toivonut jonkinlaista palautetta sovellukselta, esim. uutta huoltosuunnitelmaa tai huoltotoimenpidettä luodessa siitä, että luonti onnistui. Uuden huoltotoimenpiteen luomisnäkymässä oli myös hieman hämmentävästi valinta, että milloin huoltotoimenpide on valmistunut, vaikka sitä ei oltu vielä luotu.