Koodikatselmointi 1

[DarthKipsu]

Latasin projektin: Su 8.2 noin klo 9.00

En saanut processing käyttöliittymää avattua koska archin paketinhallinta ei jostain syystä sunnuntaina löytänyt pakettia, mutta polunetsntäalgoritmit kääntyivät nätisti. Molemmissa koodi näytti loogiselta, mutta pari pikkujuttua huomasin:

Edge luokan toString metodissa on turhaan? String s muuttujan luonti (sitä ei käytetä).

PathFinderin setPassable weight käy läpi kaikki solut ja asettaa niille arvon. Sitä ennen Grid luokka käy läpi kaikki solut palauttaakseen ne listana. Voisi olla parempi käydä solut läpi vain kerran.

Lisäksi voisi olla järkevää jakaa joitain metodeja pienempiin osiin, jotta koodin luettavuus paranisi. Sain tosin ihan hyvin selvää ohjelman toiminnasta näinkin. Myös testejä kannattaa parantaa, mutta tämän olit tiedostanut itsekin. :) Hyvää työtä!