search

k0psutin / TiraLabra

Data structures and algorithms course project. Compares three different pathfinding algorithms: A*, Jump Point Search and Iterative Deepening A*.
0 stars 0 forks source link

Vertaisarviointi 2 #2

Open silmish opened 3 years ago

silmish commented 3 years ago

Vertaisarviointi 2

Projekti ladattu 3.5.2021 klo 8.20

Yleistä

Huomaa että selkeästi tehnyt koodausta hieman enemmän ja tämän tyyliset projektit eivät olet aivan uusi aihe. Projektin rakenne on todella selkeä ja hyvin tehty, jokainen paketti ja luokka nätisti eroteltu ja loogisesti rakennettu. Koodi itsessään on hyvin selkeää ja helppolukuista.

Kokonaisuus on erittäin hyvin toteutettu, hyödyntäen abstrakteja luokkia ja periytymistä. Koodissa ei ole täten toisteissuutta ollenkaan ja kokonaisuus on hyvin selkeä.

Täytyy kyllä sanoa että oma tietotaito ei kyllä tällä hetkellä riitä siihen että osaisin sanoa mikä voisi olla pielessä tai tarvitsisi jotain viilausta koodin puolelta, omaan silmään erittäin hieno kokonaisuus.

Huomioita

Testailin ohjelmaa muutamalla eri kuvalla ja jokaisessa IDA* antaa timeouttia, en itse ole perehtynyt kyseiseen algoritmiin ollenkaan ja täten en osaa sanoa tarkemmin että onko vain ominaisuus ja ei pysty ratkomaan timeout rajaan mennessä reittiä. Onkohan tässä jokin ongelma vai ominaisuus?

GUI puolella pieni ehdotus olisi että kuvat voisi valita suoraan dropdown listauksesta ja täten nopeasti vaihtaa maisemaa. Ymmärrän että tiedoston valitseminen antaa lisää vapautta mutta mikäli ei ole tarkoitus laajentaa projektia kauheasti niin tämä olisi pieni QOL parannus.

Lopuksi vielä

Vaikka puhuinkin GUI puolella omaan makuun vähän parannuksia, niin en kyllä osaa sanoa mitään huonoa projektista. Selkeästi osaava tekijä, hienosti tehty! Lähinnä tästä sai itselleen mallia että miten asia oikeasti pitäisi tehdä :)

k0psutin commented 3 years ago

Kiitos vertaisarvioinnista ja kehuista!

Täytyykin miettiä, että toteuttaisiko karttoihin dropdown listauksen viimeiseen versioon! :)

IDA*:ssa olen huomannut sen että jos on vähänkään monimutkaisempi ympäristö (muu kuin arena.png), niin reitinhaku kestää todella kauan tai timeouttaa. Esteettömillä reiteillä tai estettä sivuuttaen se suoriutuu yleensä alle aikarajan. IDA* on alunperin tehty puu-rakenteeseen, eli ehkä en ole saanut implementoitua algoritmia oikein syklillisiin suuntaamattomiin verkkoihin. :)