klaufred
commented
8 years ago Kiitos avusta. Olin aika myöhään tappelemassa sen checkstylen kanssa. En ollut edes huomannut. Enkä tota testiäkään.
Open KaroliinaM opened 8 years ago
klaufred
commented
8 years ago Kiitos avusta. Olin aika myöhään tappelemassa sen checkstylen kanssa. En ollut edes huomannut. Enkä tota testiäkään.
Projekti haettu 8.10.2016 klo 13:55 Katsottuani ohjelmakoodia en osaa ehdottaa olennaisia parannusta vaativia kohtia. Logiikka ja Domain-pakkausten luokat näkyisivät olevan kunnossa. Testeissä näkyi että pelin toiminnot olivat hallinnassa. Käyttöliittymä taas näkyi olevan vielä vahvasti työn alla, mutta sielläkään ei ollut mitään mikä olisi osunut silmään. Checkstylestä, Javadoc-lisäys on väärässä paikassa. Se oli dokumentin lopussa, sen pitäisi olla Treewalkerin tagien välissä. Kokeilin siirtää sen paikalleen ja checkstyle lähti toimimaan. Testeissä oli yksi todella pieni juttu. Taking over -metodia testattaessa olet kokeillut sijoittaa tietttyyn ruutuun jonkin nappulan, ja "syödä" sen toisella erityyppisellä nappulalla. Nappulan tyypistä on nähty että toimiiko tuo metodi. Pawn-testin kohdalla molemmat testissä käytettävät nappulat ovat Pawn-tyyppisiä, joten lopputuloksesta ei voi päätellä onnistuiko toiminto vai ei.