sinikala / ot-harjoitustyo Goto Github PK

Projekti ladattu 26.4.19 klo. 17:30

Sain ajettua ohjelman NetBeansissa, mutta jarria ajettaessa en päässyt nimen antamista pidemmälle. Tämä kannattaa itse kokeilla esim. yliopistonkoneilla.

DAO:
userdao -luokan metodit ovat hyvin nimetty ja koodi on selkeää luettavaa. Jokaisella metodilla on selkeä ja yksinkertainen tehtävä. Nähtävästi kuitenkin on unohtunut metodiin getOldUser(String username) turha parametri username, koska sitä ei kuitenkaan metodissa käytetä.
Lisäksi sen sijaan, että jokaisen metodin alussa luodaan yhteys kirjoittammalla suora polku tietokantaan niin varmaan kannattaisi eriyttää tämä oman luokan tehtäväksi. Tähän luokkaan voisi lisäksi luoda toiminnallisuutta, joka suoritetaan aina ensimmäisenä sovelluksen käynnistyessä.

LOGICS:
Gameservice -luokan metodit selkeitä ja kommentit avaavat luokan toimintaa hyvin. Metodit ovat riittävän lyhyitä ja helposti ymmärrettäviä.
Gamelibrary -luokka on minulle ehkä aavistuksen hämmentävä, koska nähtävästi se sisältää pelissä käytettävät sanat. Mielestäni nämä kannattaisi tallentaa tietokantaan, josta Gameservice -luokka ne aina saisi tarvittaessa käyttöönsä.
User -luokka on hyvin tehty ja sen metodit ovat selkeitä lukea ja niillä on selkeät tehtävät. Kommentit auttavat koodin lukemisessa.

UI:
Käyttöliittymä on kaikki laitettu samaan luokkaan, joka tekee koodista ja sen toiminnallisuuksista vaikeasti luettavaa. Jos mahdollista, niin tätä voisi yrittää pilkkoa omiin luokkiinsa.

Huomioita:
-userDaossa oma database -olio saattaisi autata tietokannan hallinnassa
-GameLibrary -luokka voisi ehkä luoda tietokantatauluksi, jota GameService -voisi kutsua

Projektissa on paljon hyvää, joista paras on selkeät kommentit, jotka selostavat metodien toimintaa. Tällöin koodia on helppo lukea ja sen ymmärtää avun kautta. Tällaista samaa tapaa voisi myös käyttää UI -luokassa.