ikaevalko
commented
3 years ago Kiitos palautteesta.
Tuo shadow-plugin on JAR-tiedoston paketoimista varten. Sen pitäisi kyllä toimia projektille määritellyn gradlen version 6.7 kanssa. Luulen ongelman johtuvan siitä, että et ajanut komentoja projektin omalla gradlew-skriptillä. Huomaa, että Linuxilla tämä pitää antaa muodossa ./gradlew run, muuten se ei tunnista sitä. Gradlea kannattaa joka tapauksessa käyttää mieluummin komentoriviltä kuin IDE:n pluginilla.
Testauksesta huomasin sen, että sitä olisi hyvä lisätä ja ehkä tehdä testejä jotka testaisivat pienempiä kokonaisuuksia.
Erittäin hyvä huomio. Olen yrittänyt keksiä helppoa tapaa testata karttageneraattoreiden yksittäisiä metodeja, mutta tässä ongelmana on se, että jokainen generointiprosessin vaihe riippuu sitä edeltävistä vaiheista. Käytännössä ainoa tapa, jolla tämän voisi järkevästi toteuttaa, olisi refaktoroida koko systeemi uusiksi siten, että jokaiselle metodille olisi mahdollista antaa tiettyyn vaiheeseen asti generoitua karttaa kuvaava "mallikartta". Pitäisi siis tehdä paljon työtä minimaalisen hyödyn eteen.
Projekti ladattu 26.2.2021 klo. 9:30
Projekti vaikuttaa olevan hyvällä mallilla, lähellä valmistumista ja dokumentaatio on laajaa ja laadukasta.
En itse valitettavasti saanut gradlea toimimaan koneellani tai en saanut Netbeansin gradle pluginia toimimaan projektin "com.github.johnrengelman.shadow" version "6.1.0" riippuvuuden kanssa. Eli en valitettavasti pysty kommentoimaan tarkemmin sovelluksen toiminnasta tai testikattavuudesta.
Sanoisin, että koodi on pääasiassa laadukasta, mutta osa metodeista on hyvinkin pitkiä ja sisältävät useita if lauseita. Eli kannattaa mahdollisesti tutkia voisiko niitä pilkkoa pienempiin ja useampiin kokonaisuuksiin.
Testauksesta huomasin sen, että sitä olisi hyvä lisätä ja ehkä tehdä testejä jotka testaisivat pienempiä kokonaisuuksia. Mikä helpottaisi koodin pilkkomista pienempiin osiin ja helpottaisi jatkossa tehtäviä muutoksia. Lisäksi testikattavuudesta voisi lisätä kuvan dokumentaatioon.