En ole kuitenkaan varma pitäisikö Korttipakka-luokassa olla metodia kaannaKortit. Se tuntuisi kuuluvan pikemminkin Peli-luokkaan. Tai ehkä parempi olisi nimetä Korttipakka joksikin muuksi kuten Korttinäkymä. Kortithan eivät ole enää pakassa pelattaessa.
Luokkien Jarjestys ja Pari nimet voisivat ehkä olla kuvaavammat, kuten PariPakka ja JarjestysPakka. Myöskin se että Jarjestykselle annetaan parien määrä parametrinä ei ole kovin selkeää koodia, mutta en ole varma miten se olisi paras korjata. Ehkä parien määrän voisi kysyä erillisessä käyttöliittymäikkunassa ja yliluokka ei saisi parametrejä. Olitkin jo sanonut tästä.
Käyttöliittymä on aiemmi neuvottu irrottamaan omaksi paketikseen, joten sen voisi ehkä tehdä.
Hyviä pitkiä ja kuvaavia metodien nimiä, varsinkin testeissä.
Näyttäisi olevan ihan hyvin testejäkin, ja menevät läpi.
Selkeästi nimettyjä ja rakennettuja metodeita ja luokkia. Single responsibility principlen suhteen voisi ehkä aivan aavistuksen parantaa mutta muuten tosi hyvä :)
Koodin rakenne on varsin selkeä.
En ole kuitenkaan varma pitäisikö Korttipakka-luokassa olla metodia kaannaKortit. Se tuntuisi kuuluvan pikemminkin Peli-luokkaan. Tai ehkä parempi olisi nimetä Korttipakka joksikin muuksi kuten Korttinäkymä. Kortithan eivät ole enää pakassa pelattaessa.
Luokkien Jarjestys ja Pari nimet voisivat ehkä olla kuvaavammat, kuten PariPakka ja JarjestysPakka.
Myöskin se että Jarjestykselle annetaan parien määrä parametrinä ei ole kovin selkeää koodia, mutta en ole varma miten se olisi paras korjata. Ehkä parien määrän voisi kysyä erillisessä käyttöliittymäikkunassa ja yliluokka ei saisi parametrejä.Olitkin jo sanonut tästä.Käyttöliittymä on aiemmi neuvottu irrottamaan omaksi paketikseen, joten sen voisi ehkä tehdä.
Hyviä pitkiä ja kuvaavia metodien nimiä, varsinkin testeissä.
korttiNappulat[i].setBackground(Color.pink);
korttiNappulat[i].setText("Voitit!!!!! :D:D:D");
Hieno voittoteksti 👍
Näyttäisi olevan ihan hyvin testejäkin, ja menevät läpi.
Selkeästi nimettyjä ja rakennettuja metodeita ja luokkia. Single responsibility principlen suhteen voisi ehkä aivan aavistuksen parantaa mutta muuten tosi hyvä :)