skarpanton / labyrintin-ratkoja Goto Github PK

Projekti ladattu klo 21.00 torstaina 29.11.2018

Äärimmäisen hienosti toteutettu projekti mielestäni!

Tietokantarakenteet vaikuttavat fiksusti rakennetuilta ja ohjelmassa kaikki on järjestetty mukavasti eri pakauksiin selkeästi. Hieno idea käyttä eri värejä näyttääkseen A* ja BFS toiminta samassa labyrintissa, ei tarvinnut tulostaa kuvaa useampaa kertaa. Graafinen näkymä muutenkin makea.

Vaikea oikeastaan antaa neuvoa mitä tässä parantaisi tai muuttelisi.
Ajoittain voisi vaikka alustaa monta samantyyppistä muuttujaa samalla käskyllä, mutta tämäkin on ihan makuasia (esim. private int a, b, c, ...). Joissakin luokissa, kuten maze.datastructures:in luokissa tämä voisi lyhentää pitkiä muuttujien alustuksia luokkien alussa. Muutoin mielestäni koodi on äärimmäisen selkeärakenteista, ja eri toiminnot on jaettu loogisesti omiin luokkiinsa.

Enemmän javadoc:ia voisi olla, vaikka tämänkin tarvetta vähentää koodin selkeys. Lisää testejä voisi olla kanssa, vaikka nytkin jo hyvin testailtu :) Jos tarvitset testien kirjoittamisessa konsoliin tulostuvaan tekstiin käsiksi pääsyä, voit käyttää tälläistä tapaa:

public void testi() {
ByteArrayOutputStream outContent = new ByteArrayOutputStream(); //luodaan uusi printStream
PrintStream oldOut = System.out; //tallennetaan vanha konsolin printStream
System.setOut(new PrintStream(outContent)); //nyt konsoliin tulostettava teksti tulostuu outContent //streamiin tässä. Uuden printStreamin sisältöä voi tutkia helposti.
//Esim näin:
metoditJotkaTuottaaKonsoliinTekstiä() //tuotetaan sisältöä konsoliin
metoditJotkaTuottaaKonsoliinTekstiä() //tuotetaan sisältöä konsoliin
System.setOut(oldOut); //palautetaan vanha konsolin printStream käyttöön
double aika = (endTime - startTime) / 1000000000.0;
String vertailtava = "Labyrintin luominen vie : " + aika + "s"
assertEquals(vertailtava, outContent.toString().trim()); //vertaillaan outContent printStreamin sisältöä haluamaamme stringiin.
}

Mukavaa kurssinjatkoa!