Koodikatselmointi

[reykjaviks]

ladattu 29.12.2016 kello 11:30

KorttiTest.java

• Kenttien kortti1 ja kortti2 nimet voisi olla kuvaavammat, esim. ristiKakkonen ja pataKolmonen. Nyt jos testejä lukee, niin on vaikea hahmottaa, miten kortit eroavat toisistaan.

• Ehkä "Toimii"-verbin voisi vaihtaa johonkin substantiiviin? Jos käyttäjä tietää, että kyseessä on testi, niin testin perustarkoitusta ei välttämättä tarvitse silloin havainnollistaa. Itse nimeäisin testit esim. testToString ja testEquals, koska kyseessä on metodi, joka testaa toista metodia ts. "testi". Etuliitteen käyttö luo myös testiluokkaan johdonmukaisuutta.

• Vaikeuttaako englannin ja suomen kielen käyttö sekaisin koodin lukua?

[reykjaviks]

ladattu 31.12.2016 kello 11:50

Main.java

• While-loopista kannattaisi ehkä tehdä kokonaan oma metodinsa, koska nyt sen toimintaa ei voi esim. testata ollenkaan. Nimeäminen helpottaisi myös loopin ymmärtämistä.

• i-muuttuja alkaa tavallisesti nollasta, eikä lukija välttämättä huomaa, jos tästä tavasta poiketaan.

• Nopealla googlauksella pelipakassa on 52 korttia, ja loopin arvoalueeseen kuuluu nyt vain 50 kierrosta. Jos tämä ei ole virhe, niin looppiin voisi lisätä kommentin väärinkäsitysten varalta.