forbaya/dungeon-generator

Tira-labra

dungeon-generator's Issues

Koodikatselmointi

Projekti ladattu 29.11.2017 klo 19:30
Aluksi ohjelmaa kääntäessä, jokin vaikutti jäävän ikuiseen looppiin. Tarkemmin katseltuna havaittu että syyllinen lienee DungeonTest-testiluokka jos tuli oikein tulkittua, mutta noin ajettaessa vaikutti sinällään toimivan ja ruudulle piirtyvän jokin muodostelma.
Luokkiin tutustuttaessa kaikki vaikutti päällisin puolin loogisilta. Kun tarkemmin luokkiin perehtyi niin ehkä pinnalle nousi joitain kysymyksiä, vaikkakin voi hyvin olla että vielä tässä vaiheessa kaikki luultavasti on kohtalaisen keskeneräistä. Axis-enum -luokan olemassa olo hieman yllätti pienellä tarjonnallaan/koollaan. Epäilemättä tilanne voi muuttua. Vähäpätöisenä huomiona kooditiedostonpituuden optimoinnissä voisi tietysti käyttää importeissa .* import javafx.scene.;import javafx.scene.paint.;import javafx.scene.shape.*; tyyppisiä ratkaisuja javan itse huolehtiessa turhien poisjättämisestä. Ehkä se on makuasia myöskin. JavaFX:stä ei sen suurempaa kokemusta ole.
Testiluokat vaikuttivat toimivan edellämainittua lukuunottamatta ja lienivät riittävät tässä vaiheessa työtä eikä niistä selkeästi mitään huomautettavaa siksi löytynyt. Varmasti nekin kurssin edetessä vielä laajenevat.
Noin vinkkinä voi suositella Gradle, Travis ja Codecov yhdistelmää Mavenin sijasta.
Menestystä loppukurssille!

Koodikatselmointi

Erittäin mielenkiintoinen aihe.

-Jostain syystä kaikki IDE:t joilla avasin projektin eivät tunnistaneet javafx:ää. Terminaalista sain ohjelman käynnistettyä ja saattaa olla ongelma omassa koneessa, tai ehkä puuttuu jokin dependency/plugin mavenistä. EDIT: Vaikuttaa olevan ongelma Linuxilla. Windowsilla IDE:t tunnistavat javafx;n

-Jos muistan oikein, ei omissa luokissa saa käyttää javan valmiita ratkaisuja, joten ArrayList luokassa pitäisi ehkä muuttaa increase listSize() metodia, sillä se käyttää valmista arraylist luokkaa.

-Ohjelman suorituksen jälkeisessä ruudussa, kaikki solut eivät vaikuta mahtuvan ruudulle. Ohjelma myös vaikuttaa luovan aina dungeonin, joka kulkee ylhäältä vasemmalta, alas oikealle. Muutaman kymmenen runnin jälkeen en ole nähnyt yhtäkään dungeonia, joka olisi eri suuntainen.

-Ohjelmalla kesti suhteellisen pitkään luoda dungeon (noin 5500-6500ms), ja hetken asiaa tutkittuani päätin kääntää Dungeon luokan rivillä 112 olevat x ja y coordinaatit päittäin, ja dungeonin luonti tippuin vain 1500-2500ms. Eli tässä muutos:
int amountToMove = axis == Axis.X_AXIS ? yCollisionDistance : xCollisionDistance;

-Jos lopussa ilmestyvän ruudun laittaa fullscreeniksi, ei osalla alueista ole ruudukkoa. Vaikka ei ole ollenkaan tärkeä asia, ei ole kaikista mieluisinta katseltavaa, varsinkin kun koko dungeon ei näy ruudulla.

Onnea loppu kurssille.