Koodikatselmointi

[techstalgist]

Kommentit perustuvat 4.2.2017 11:49 ladattuun ZIP:iin.

Positiivista on, että jokaisen luokan vastuu on suht selkeä tällä hetkellä. Eli on helppo päästä kärryille siitä, mikä kunkin luokan tarkoitus on.

• Vaihtaja -luokka
  • Kun Vaihtaja- luokka otetaan osaksi muistipeliä, niin voisi harkita, että lisäisi esim. Pelimoottori –luokan, joka hallinnoisi pelin kulkua, ja Peli-luokka vaan pitäisi kirjaa pelin senhetkisestä tilanteesta
  • Muuten voi olla, että Peli-luokasta tulee turhan iso
• Peli -luokka
  • Pelin pelaa() –metodi on aika pitkä, eli sen voisi purkaa pienempiin palasiin, joita voisi ehkä testata toisistaan erillään
    • Esim. jos pelaaja löytää parin, niin tämän tilanteen käsittelylle voisi olla oma metodinsa
    • Myöskin korttien valinta voisi ehkä olla oma metodinsa
  • Löydettyjä kortteja ei nyt poisteta kortit- listalta, eli samat kortit voi aina valita uudestaan
  • Saman kortin voi valita vuoron aikana kahdesti
  • Kortit ovat aina samassa järjestyksessä, eli jos peliä pelaa monesti, niin pelaaja osaa ulkoa järjestyksen
  • Jos pelaaja löytää parin, niin teksti ”Sinulla on” tulostuu kahteen kertaan
  • Jos ruudukossa on alussa 64 korttia, ja vain kahden kortin paikkaa vaihdetaan keskenään, niin tuoko tämä tarpeeksi lisähaastetta, vai pitäisikö vaihtaa useamman kortin paikkaa keskenään?
  • Kun ruudukkoa ei ole vielä toteutettu, niin nyt pelaaja ei tiedä, kuinka monta korttia pelilaudalla on jos pelaaja valitsee ensimmäiseksi kortiksi kortin 20, niin ohjelma kaatuu
• PeliTest –luokka
  • Olisiko järkeä tehdä PeliTest –luokkaan testejä, joilla testattaisiin pelin pelaamista jossain yksinkertaisissa tilanteissa? Muistaakseni Ohjelmistotekniikan menetelmät –kurssilla oli jotain esimerkkejä siitä, kuinka System.in ja System.out:in voi korvata testeissä haluamillaan syötteillä ja tulosteilla
  • Testaaminen vaatisi varmaan sitä, että testipeleissä kortit olisivat aina tietyssä järjestyksessä, jotta lopputulosta olisi mahdollista testata (nyt kortit ovat aina samassa järjestyksessä, mutta tämä ei liene lopullinen toteutus)