Koodikatselmointi

[ilarinie]

Ladattu 10.10. 21:44

• Ulkoasu näyttää siltä miltä tetriksen olettaisikin näyttävän
• I-paloja ei tule kun niitä tarvisi -> aito tetrisfiilis
• Kannattanee laittaa pom.xml pit-conffiin luokat, mitkä kuuluvat testauksen piiriin niin saa raportista mukavamman näköisen. (kts. javalabra pit-ohje)
• Luokat ovat yksinkertaisia ja selkeitä, kuten myös nimeämiskäytännöt
• Pisteitä tms toiminnallisuutta jäi hieman kaipailemaan, mutta kaikki aihemäärittelyssä määritellyt ominaisuudet löytyvät ja toimivat
• Dokumentointi kattavaa ja selkeää
• Kovasta yrityksestä huolimatta en onnistunut rikkomaan

[Rochet2]

Pisteet lisätty ja pom.xml muutokset tehty.