hanninev/paskahousu

Ohjelmoinnin peruskurssien harjoitustyö. Paskahousua mukaileva korttipeli, jossa käyttäjä voi pelata tietokonetta vastaan.

paskahousu's Issues

Koodikatselmointi

Projekti ladattu 31.12.2016 13.47

Ohjelman rakenne on vähän sekava, minkä huomaa jo luokkakaaviosta. Siirtojenkäsittelijä olisi hyvä saada kiinni Peliin jollain tavoin, jotta käyttöliittymän suunnittelu helpottuisi. Nyt Paaohjelma joutuu käyttämään isoa läjää eri luokkien olioita.

Luokat ja metodit on nimetty hyvin ja kuvaavasti. Metodit ovat pääsääntöisesti järkeviä ja tarpeeksi lyhyitä. Pelaajan nimen voisi asettaa konstruktorissa helpoiten? Poikkeuksia on siirrelty eteenpäin aika huolettomasti, mutta ne toki ehtii käsittelemään myöhemminkin.

Kannattaa huomioida Saannot-luokan booleanin palauttavista metodeista, esim saakoKokeillaOnnea, ettei niihin välttämättä tarvitse lainkaan if-rakennetta.

Sovelluslogiikkaa on testattu kohtuullisen reilusti. Arvioiminen vähän hankalaa, kun en oikein muista pelin sääntöjä, mutta vaikuttaisi siltä että testaus on kattava logiikan osalta.

Pakka perii Korttijoukon, mutta yläluokkaa kutsutaan vähän hassusti. Voisi olla myös mahdollista tehdä Korttijoukosta interface tai abstrakti luokka, ja siitä sitten toteuttaa Pakka, Pino, Kasi ja Kaatuneetkortit?