Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Koodikatselmointi #1

Open
jhakkari opened this issue May 3, 2022 · 0 comments
Open

Koodikatselmointi #1

jhakkari opened this issue May 3, 2022 · 0 comments

Comments

@jhakkari
Copy link

jhakkari commented May 3, 2022

Projekti ladattu: 3.5. klo. 18:30.

Koodin tiedostot on jaettu omiin kansioihinsa tyyppinsä mukaan, hienoa!

Koodi on kirjoitettu selkeästi ja jaettu omiin luokkiinsa siististi, Clean Code -periaatteita noudattaen.

Ohjelman eri osat toimivat kuten pitääkin, bugeja en löytänyt. Ohjelmalle tehty uniikki kuvake!

Olisi hyvä, jos peli-ikkunan reunoista ei pääsisi liikkumaan yli, mikä päättää meneillä olevan suorituksen. Luultavasti tarkoituksella näin suunniteltu, makuasioita :)

Peliin voisi rakentaa "Pause" valikon. Nyt ainut tapa päästä etusivulle tai katselemaan tilastoja on pelin häviäminen.

Edelliseen liittyen myös pelaajan sijainnin voisi tallentaa tietokantaan jatkoa ajatellen,

Kokonaisuudessaan ohjelma on vakaa, ulkoasuun on kiinnitetty huomiota ja se täyttää vaatimusmäärittelyssä esitellyt ominaisuudet. Hyvää työtä!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant