search

alemati / simpleDungeonGeneratorTiralabra2019

0 stars 0 forks source link

Vertaisarviointi #2

Open antlammi opened 5 years ago

antlammi commented 5 years ago

Projekti ladattu 26.4. klo 16:31

Mitä tein

Kokeilin ajaa ohjelmaa useita kertoja erikokoisilla kartoilla ja erilaisilla syötteillä.

Suoritusaika siis vaikuttaisi noin kaksinkertaistuvan korkeuden ja leveyden kasvaessa sadalla. Tämän valossa ohjelman aikavaativuus näyttäisi olevan oikein ajateltu.

Järkevillä syötteillä ohjema toimi aina, kuten pitikin ja generoi toimivan kartan. Ohjelma kaatui mikäli antoi korkeudeksi tai leveydeksi negatiivisen arvon tai mikäli huoneen minimikoon asetti pienemmäksi kuin maksimikoon. Samoin, kun korkeuteen tai leveyteen syötti merkkijonoja. Tämä ei tietenkään ole toki mikään suuri puute.

Ajoin lisäksi testit. Testeistä kaksi -- testMapShow ja testRoomAndMazeAdded eivät menneet läpi, alla näiden tulokset

Failed tests: 
  MapTest.testRoomAndMazeAdded:49 expected:<[######
###+ #
## # #
#### #
#    #
#####]#
> but was:<[######
#    #
# # ##
##   #
#  # #
#####]#
>
  MapTest.testMapShow:27 expected:<[###
###
##]#
> but was:<[###
###
##]#
>

Ensimmäisessä testissä ei siis jostain syystä tehty huonetta ja/tai lisätty ovea. Toisessa näyttäisi, että vertailtavat asiat ovat samoja, mutta testi ei silti mene läpi. Kokeilin hieman NetBeansin avulla tutkia mistä on kyse ja löytyi tällainen:

tiralabra-vertaisarviointi-2

Oletan, että tämä liittyy jotenkin ByteArrayOutputStreamin toimintaan, mutta en yhtään tunne tätä, joten en sen enempää arvaile.

Koodi

Itse koodi on kautta linjan mainiota. Metodit ovat hyvin nimettyjä ja sopivan mittaisia. Oma satunnaislukugeneraattori on myös mukava nähdä.

Kaikki oleellinen tuntuisi toimivan niin kuin pitääkin ja koodi on laadukasta. Hyvää työtä!