jkostet/stormsurge

StormSurge is a Battleships/Sea Battle inspired turn based game with bunch of ships and surprises.

StormSurge is developed as a course project at Department of Computer Science, University of Helsinki. See course instructions here.

• Work Log
• Trello Board

stormsurge's Issues

Koodikatselmointi

Yleisiä huomioita (.zip ladattu 1.4.2015)

• Koodi on helppolukuista ja järkevästi sisennettyä.
• Ohjelma on jaettu loogisesti pakkauksiin ja luokkiin.
• Luokat toteuttavat Single Responsibility Principlen onnistuneesti.
• Luokat ja metodit on nimetty selkeästi ja loogisesti.
• Metodit ovat riittävän yksinkertaisia ja lyhyitä.
• Ytimekkäät kommentit oikeissa paikoissa tekevät koodista helposti ymmärrettävää.

Luokkakohtaisia huomioita

• TextBasedUserInterface: addShipToSea metodin tulisi (?) tarkistaa ettei syötteenä anneta meren sivun pituutta pidempää lukua. Nyt voin luoda vaikkapa 100000:n ruudun pituisen laivan. Tekstin syöttäminen kaataa ohjelman, jonkinlainen tarkistus olisi siis paikallaan.
• Ship: Metodit ovat loogiset ja selkeät. Kommentti siitä että koordinaatit alkavat vasemmasta yläkulmasta helpotti hahmottamista huomattavasti. Tarkempaa analyysiä on vaikeampi antaa kun laivat eivät vielä piirry merelle.
• ShipList: Onko tarpeellinen luokka vai olisiko sama käyttää suoraan ArrayListiä?
• Testit: Yksinkertaisia, hyvin nimettyjä ja loogisia. Ei pahaa sanottavaa.

Summa summarum

Vaikea keksiä pahaa sanottavaa, ohjelma on mielestäni todella hyvin kirjoitettu. Jatka samaan malliin!