northernlion / pathfindingmaze Goto Github PK

Koodikatselmointi 1

viikko 5
Projekti ladattu 22.8. klo 10:42

Ohjelmakoodi

Koodi on selkeää ja ohjelman rakenne on looginen. Ohjelmakoodia voisi kommentoida vähän enemmän ainakin lyhyesti joka luokkaan ja pisimpiin metodeihin, että mitä ne ovat/tekevät. Koko lailla koodi on kuitenkin ymmärrettävää ihan vaan tutkimallakin.

Tietokantoja on tehty ahkerasti itse. Koodia on pilkottu hyvin lyhyiksi metodeiksi. Jos tietorakenteiden kanssa meinaa tulla mahdoton kiire, voisi A* pitää kirjaa käsitellyistä Tile:istä HashSetin sijaan yksinkertaisemmalla tavalla. Omassa työssäni menin laiskimman kautta ja pidän kirjaa vain boolean array:lla, jossa listan indeksi on sama kuin solmu-olion uniikki id arvo. Tile-olioille voisi helposti alustaa tunnuksen varsinkin, jos niitä on verkossa aina 10x10. Tai sitten A* voisi hyödyntää Tile-olion checked-arvoa. Mutta nämä toki vain ehdotuksia, mikäli osasin edes lukea koodia oikein @.@

Muuten ohjelma näyttää hyvältä ja käynnistyy netbeanssissä/terminaalissa. GUI toimii ja käyttäjän kannalta on hyvä/hauskaa päästä tekemään verkkoja helposti itse käyttöliittymän kautta!

Testit

Testejä on hyvissä määrin ja niiden nimentä selkeää. Jotta testien kattavuutta pääse tarkastelemaan NetBeanssissä, kannattaa projektiin lisätä plugin JaCoCo:lle. Se onnistuu liisäämällä alla oleva pätkä pom.xml-tiedostoon <plugins></plugins> sulkeiden sisään.

  <plugin>
      <groupId>org.jacoco</groupId>
      <artifactId>jacoco-maven-plugin</artifactId>
      <version>0.7.1.201405082137</version>
      <executions>
          <execution>
              <goals>
                  <goal>prepare-agent</goal>
              </goals>
          </execution>
          <execution>
              <id>report</id>
              <phase>prepare-package</phase>
              <goals>
                  <goal>report</goal>
              </goals>
          </execution>
      </executions>
  </plugin>

Sen jälkeen kun right-clickaa projektin nimeä, tiputusvalikkoon pitäisi tulla kohta "Code Coverage" => "Show Report". Sen jälkeen auenneen ikkunan vasemmasta alareunasta "Run All Tests".
Tarkemmat ja selkeät ohjeet tähän löytyy täältä. Nuo ohjeet on jo aika vanhat ja jacocosta voi olla jo uudempikin versio, mutta omassa projektissani tuo paste-koodi toimii ja ainakin omalla koneellani myös sinunkin projektissa.