nygrenh / laivanupotus Goto Github PK

Tiedosto ladattu GitHubista 29.9. klo. 14:50.

Laivanupotus vaikuttaa hyvältä ja monipuoliselta aiheelta. Luokkien jakaminen erillisiin paketteihin teki projektiin tutustumisesta helppoa ja selkeytti ohjelmasi rakenteen sisäistämistä. Hieman epäselväksi jäi dokumentaation vähäisyyden takia se, miten peli käytännössä tulee valmiina toimimaan kahden pelaajan välillä. En pahemmin keksinyt koodiisi parannusehdotuksia, mutta alla nyt pari asiaa, jotka tulivat mieleen. Tsemppiä loppukurssille, toivottavasti projekti valmistuu ajallaan.

Suosittelen, että Kayttoliittyma-luokassa määrität avattavan ikkunan ihan siisteyden vuoksi niin, ettei käyttäjä pysty sen kokoa itse muuttamaan. Ikkunaan piirrettävä grafiikkahan ei skaalaudu ikkunan koon muuttuessa. Esimerkiksi käyttäjän klikatessa ikkunan koko näytön kokoiseksi jää pelialue ruudun vasempaan yläreunaan ja ruutu täyttyy tyhjällä valkoisella tilalla. Tyylikäs ratkaisu olisi toki myös laittaa grafiikka skaalautumaan ruudun mukana.

Mietin Laiva-luokan kohdalla tuota laivan pituuden esittämistä. Käsittääkseni laivanupotuksessa laivat ovat tietyn ruutumäärän mittaisia. Voisiko laivan pituuden esittää sen täyttämän ruutumäärän mukaan int-arvon sijaan?

Ovatko Piirtoalusta- ja PelilaudanPiirtoalusta-luokat molemmat tarpeellisia? Voisiko luokkien toiminnot siirtää yhden luokan alle? Molemmat kuitenkin perustuvat perinnän kautta JPanel-luokkaan.

Tuli myös mieleeni, että onko pelilaudan koko loogista määrittää Pelaaja-luokan kautta? Mikäli pelilaudan koko määritellään aina pelin alussa pelaajien toimesta on tämä luonnollisesti fiksuin ratkaisu.