wilzon / piilopokeri Goto Github PK

Piilopokeri, ladattu reposta 30.11.2013 klo 20:10

Selkeää koodia ja pelin idea ja toimintaperiaatteet käy hyvin selville, dokumentaatio ok. Muutama luokka vaikuttaa turhan isolta ja monivelvollisuuksiselta. Varsinkin piilopokeri- ja kasi -luokat ovat todella isoja eivätkä noudata single responsibility -periaatetta ihan loppuun saakka. Piilopokerista voisi erottaa erilleen jakaja-luokan, joka hoitaisi korttien käsittelyt pakasta käsiin yms.. Käsi-luokan voisi jättää huolehtimaan vaan kädessä olevien korttien säilyttämisestä ja organisoimisesta, ja mielestäni voitontarkastelu (siis että oliko kädessä voittoa ja jos oli, niin mikä voitto) joutaisi omaan luokkaansa. Koodin pilkkominen metodeihin noin muuten toimii hyvin ja on johdonmukaista, metodit noudattavat single responsibilitya.

Kaikki testit eivät mene läpi, eritoten jokerin ja ässän moniarvoisuuden kanssa pelin logiikalla näyttäisi olevan vielä pieniä ongelmia, samaten kuin joissain muissakin käden voittoa analysoivissa metodeissa. Sen analysointi, onko kädessä voittoa, on kieltämättä haastavaa.

Huomasin kasi.onkoSuora()-metodissa virheen:

if(apuKasi.getArvot().contains(15) || apuKasi.getArvot().contains(15))

sama ehto siis ||-operaattorin molemmilla puolilla.

Harmi kun ei vielä päässyt testamaan itse peliä, näin monimutkaisen pelin esittäminen tekstikäyttöliittymällä voi olla haastavaa. Olisi ollut hauskaa nähdä, miten olet ratkaissut tämän. Alt-symbolit (♥♦♣♠) voisi toimia tässä selkeämmin kuin kortin maan tekstuaalinen esitys?

Pelin toiminnallisuus hyvä ja monipuolinen, kustomoitavuudesta plussaa (voi poistaa jokerit, voi muokata pelaajien määrää). Tekoäly vaikuttaa hyvältä, siitä iso plussa myös!