mremes / bblackjack Goto Github PK

Projekti ladattu: 13.6.2015 klo 13.50

1. Pääosin kenttien ja metodien nimet ovat selkeät. Metodit myös sopivan pituisia. Kenttien ja metodien nimissä käytetty sekä englantia että suomea. Olisi parempi käyttää vain joko suomea tai englantia.
2. Käyttöliittymässä on paljon logiikkaa. Voisi olla tarvetta "peli" luokalle, johon tämä logiikkaa voitaisiin laittaa ja käyttölittymä tuntisi vain "peli" luokan ja "peli" luokka pelaajan, jakajan ja kierroksen.
3. Pelaaja luokan nimi voisi olla PelaajanSaldo tai Saldo, koska se sisältää vain saldoon liittyviä kenttiä ja metodeja.
4. Se, millä logiikalla luokat on jaettu paketteihin, ei oikein aukea minulle. Voisiko samaan pakettiin laittaa luokat, jotka toimivat yhdessä luoden yhden kokonaisuuden?
5. Kierroksen ja jakajan vastuut eivät ole minusta selkeät. Jakajahan jakaa vain kortteja -ei käsiä.
   Kierros muodostaa korteista käden, jonka pelaaja voi esim. pyytää jakamaan kahdeksi kädeksi.
   Jakaja tarvitsee palveluja kierrokselta, jotta tietää voiko vielä jakaa (käden summa).
6. Jakajan ja pakan vastuut eivät ole minusta selkeät. Jakallahan on pakat, eli ei pitäisi operoida kortteja vaan pyytää pakkaa esim. antamaan kortin tai sekoittamaan pakan
   (eikö lopputulos ole sama vaikka korttien sijaan sekoitetaan pakat?). - No eihän se ole sama, kun käsittelee kahdeksan pakan korttijoukkoa. Voisiko olla erillinen "Pakat" luokka, joka "hallinnoisi" pyydetyn suuruista korttijoukkoa?
7. En löytänyt testejä kierros -luokalle. Korttipakka luokan testeissä tarkistettin, että pakassa on oikea määrä kortteja, mutta ei sitä mitä kortteja pakassa on.
8. Korttien kuvat on talletettu kahteen kertaan (sekä other sources että source packages).