Koodikatselmointi

[aarekr]

Hei

Ladattu ti 12.12.2023 klo 18:00

Yleistä

• Pelin graafinen yleiskuva on ammattimainen ja tuli heti mieleen jokin
  samantyyppinen peli.
• Metodit on nimetty kuvailevasti.
• Koodi on helppolukuista ja erityisesti main.py on erinomaisesti kommentoitu.
• Dokumentaatiossa kaaviot ovat yksityiskohtaisia.

Asentaminen ja pelaaminen

• Käyttöohje on perusteellinen ja sitä on helppo seurata.
• Käyttäjätilin luonti ja kirjautuminen toimivat.
• Zonen lisäämisessä kenttiin sain syötettyä tekstiä ja zone tallentui listaan.
• Cancel sulki ikkunan.
• Valitsemalla zonen Show zone -listalta avautui valkoinen ruutu ja siitä pääsi pois
  vain sulkemalla ohjelman. Konsoliin tulostui virheviesti. (README -tiedostossa oleva
  kuvailu liittyy ilmeisesti juuri tähän osa-alueeseen.)
• Exit map view -nappi toimii kuten ohjeissa sanotaan.

Testaaminen ja Pylint

• Kaikki 18 testiä menivät läpi.
• Käyttäjätilitestit testasivat tilin luomista ja kirjautumista oikeilla, väärillä ja
  puuttuvilla tiedoilla.
• Pylint antoi 4 virheilmoitusta, jotka vaikuttivat olevan helposti korjattavissa
  (esim. unnecessary 'else' after 'return').

Hyvää työtä! Mielenkiintoisen näköinen projekti. Toivottavasti ongelmakohdat ratkeavat pian. Tsemppiä.

Aare