nykanenj / texasholdemsitngoheadsuphud Goto Github PK

downloaded 5.2.2016 21.17

näyttää hyvältä :) tietoni pokerista on aika vähäinen ja suurin osa koodista on (vielä tässä vaiheessa) 'rakennuspalikoita' eikä tee juuri mitään joten minun oli hieman hankala hahmottaa lopputulosta.

Onnistuin myös lataamaan projektia päivitysten välissä joten handData on vielä handAnalysis joissakin paikoissa, mutta huomasin että tätä oli jo korjattu uudessa versiossa. Tiesit ehkä jo (enhän minä näe mitä tapahtui) mutta sanonpa kuitenkin että kannattaa käyttää Refactor - Rename.

En ymmärtänyt miksi HeadsupHUD:issa luodaan Hand:ia jonka cardsontable sisältää vain kolme korttia, mutta tämähän on tilapäinen koodi joten se johtuu ehkä syistä jotka liittyvät testaamiseen(?).

HandReader-luokassa on käytetty suomea ("lukija", "lataaTiedostonSisalto", "tulosta"), mutta muuten kaikkea on nimetty englanniksi, kuvaavasti ja muutenkin hyvin.

Vaikuttaa siltä että luokkia, metodeja ja pakkauksia on rajoitettu hyvin. Ne ovat mielestäni sopivan kokoisia ja noudattavat single responsibility-periaatetta.

Tästähän tuli kokonainen romaani (friday night fun, wohoo), toivottavasti siitä oli jotain hyötyä. Tsemppiä!