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

Vertaispalaute #1

Closed
hjuha opened this issue Oct 11, 2018 · 0 comments
Closed

Vertaispalaute #1

hjuha opened this issue Oct 11, 2018 · 0 comments

Comments

@hjuha
Copy link

hjuha commented Oct 11, 2018

Arviointi tehty 11.10.2018 kello 19:54 ladatulle työlle.

Tekoäly pelaa (yhden) testauksen perusteella melko hyvin ihmistä vastaan ja rankaisee kyllä virheistä.

Projektin suorittamisen yhteydessä oli ongelmia, sillä oli haastavahkoa selvittää millä IDEllä ohjelma oli toteutettu. Tästä olisi hyvä olla maininta jossakin, tai vaihtoehtoisesti Java-projektin tapauksessa .jar-tiedosto voisi löytyä jostakin? Lopulta .gitignore paljasti kyseessä olevan IntellJ:n IDEA, mutta asennusongelmien vuoksi tiedostot tarvitsi manuaalisesti kopioida.

Myös testien ajossa oli joitakin ongelmia: Näytti siltä, että PerformanceTest luokka jäi jotenkin ikuiseen silmukkaan. Voi toki olla, että tämä oli ongelma, joka johtui tiedostojen manuaalisesta kopioinnista ja NetBeansista. Testausluokka GameTest ei nykyisellään toimi oikein, sillä move.isLegal() näyttäisi palauttavan aina true "oikeille siirroille" riippumatta pelaajan puolesta. Toisin sanoen pelaaja voi liikutella vastustajansa nappuloita.

Jos pelaaja valitsee värikseen mustan, niin pelaaja saa silti aloittavan vuoron tai peli skippaa siirron piirtämisen. Korjaukseksi tähän toiminee seuraava: new Game(color) on tehtävä vasta värin kysymisen jälkeen ja pelilauta tulisi piirtää vasta tietokoneen mahdollisen vuoron jälkeen.

Lisäksi laudan peilaus on nykyisellään hämmentävä: Toisin sanoen, kun pelaaja valitsee mustan värin, niin siirto b2 b3 tarkoittaakin b7 b6, toisin kuin laudan "koordinaatistosta" voisi päätellä.

Yleisesti ottaen projekti on hyvässä vaiheessa, loppuun vain pientä hiontaa vielä :)

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

2 participants