Latasin projektisi Sunnuntaina 28. Huhtikuuta 2019 klo 20:23:12
Aluksi pahoittelut, etten ole tehnyt tätä aikaisemmin. En tiedä, saako tästä enää pisteitä, mutta annan mielelläni palautetta toisen työstä.
Ensivaikutelma reposta
Luin aluksi läpi vaatimusmäärittelyn. pelinäkymä oli avattu hyvin ja rogueliket tuntevalle oli selvää, mitä tuleman pitää. Avasin arkkitehtuurikuvaus -asiakirjan huomatakseni sen olevan varustettu vain rikkinäisellä linkillä. Löysin "kuvat" kansiosta sekvenssikaavion jonka avulla aloin tutkailemaan koodiasi.
Kommenttini koodista
Koodi on selkeästi kirjoitettua eikä mitään ylimääräistä näytä jääneen pyörimään oman ripeän silmäilyni perusteella. Olet eriyttänyt erinomaisesti single-responsibility periaatteen mukaisesti luokat omiksi asioikseen, pakkausrakenne oli myös looginen. Uskon, että tapasi käyttää perintää luokissasi vastaa pelisuunnittelun hyviä perusteita, hyvä! Lisäksi, ENUM:n käyttö on asia jonka toivoisin ymmärtäväni yhtä hyvin, kuin miten sinä sen vaikutat hallitsevan.
Kommenttini pelistä
Käynnistin pelin sekä komentoriviltä että Netbeansista. On aina innostavaa saada mahdollisuus vaikuttaa hahmoonsa, vaikkakin vain nimen muodossa, hyvä. Pelin alettua etsin käyttöohjeen käsiini jotta pääsisin pelaamaan peliä niin kuin sitä on tarkoitettu pelattavan. Käyttöohjeessa olisi voinut lukea, mikä komento ohjaa mihinkin suuntaan, vaikka toki, koodista logiikan lukieassani ymmärsin numpad-asettelun. Käytin itse läppärini ainoata vaihtoehtoa eli numeroriviä joka aiheutti välillä sekaannusta liikkumiseen.
Muut huomiot
Mainitsit määrittelydokumentissa, että pelaajan hp ja pelattava taso näkyy pelissä. En itse nähnyt näitä lukuja. Tehtyjen toimien raportointi toimi muuten hyvin. Pelin lataaminenkaan ei toiminut. En löytänyt bugeja syötteeseen tai esim. seinien läpäisyyn liittyen, hyvä! Rappuset toimivat ilmoitetulla tavalla.
Luin kehitysideasi, ne olivat järkeviä. Nykymuodossaan pelin jännitys mudostui minulle liikkumisen rytmittämisestä niin, että sain minimoitua vihollisten aiheuttaman vahingon. Jännitystä ja tekemistä on siis hyvä saada enemmän. Värilliset kirjaimet osoittamaan erilaisia vihollisia ja/tai aarteita voisi olla yksi helppo tapa tuomaan lisää näyttävyyttä. Vaikutat kokeneelta ohjelmoijalta sillä koodin rakenne ja tehokkuus oli miellyttävää katsottavaa.
Toivon, että jatkat pelin kehittämistä kirjoittamasi määrittelydokumentin mukaisesti!
Yt.
Akseli