coolnesss/max-flow

Algorithms for maximum flow problem

max-flow's Issues

Koodikatselmointi

Latasin koodin 2.10. kello 21:23

Koodi näyttää oikein hyvältä - kommentteja on erittäin runsaasti, muuttujat, funktiot yms. on nimetty kuvaavasti mutta kuitenkin ytimekkäästi ja rakenne on muutenkin ilmavan ja tiiviin välisellä kultaisella keskitiellä. Isoa plussaa siitä; se, että en ole aikaisemmin tökkinyt C++:aa ei haitannut ymmärtämisessä yhtään.

Suurin parannuskohde joka pisti silmään on ns. "magic numberit" - edmondskarpissa, ford_fulkersonissa ja scaling_flowissa kaikissa on annettu pathMin:lle alkuarvoksi paljon yhdeksikköjä. Tämän on oletettavasti tarkoitus olla luku, jota pienempiä kaikki potentiaaliset pathMin-arvot ovat. Ongelmia kuitenkin syntyy, jos joku käyttääkin ohjelmaa odottamattoman suurilla arvoilla - pathMinin oletusarvo saattaa jäädä voimaan! Yksi hyvä tapa ratkaista tämä olisi napata ensimmäinen pathMin-arvo vaikkapa ensimmäisestä vastaantulevasta polunpainosta.

Ainoa toinen "ongelma" jonka huomasin oli jonkinlaisten vinkkien siitä, mitä pitäisi syöttää, puuttuminen main.cpp:stä. Oletettavasti ohjeet eivät kuitenkaan ole ykkösprioriteetti ja ovat varmasti tulossa.

En alkanut sähläämään ohjelman kääntämisen kanssa testatakseni toimintaa tai testejä, mutta koodin ulkoasun perusteella luotto on kova.

koodikatselmointi

Latasin projektin lauantaina 15.10 klo 15:30

En saanut testejä enkä ohjelmaa suoritettua, mikä varmaankin oma tai windowssin syy, eikä tähän hätään ole linuxia saatavilla.

Ohjelman rakenne, sekä koodi selkeästi ja kompaktisti toteutettu. Vaikka en ole koskaan c++ kieleen koskenut, niin kommentointi teki koodin ymmärtämisestä hyvin helppoa. Ehkä ainoa selkeyttä haittaava asia voisi olla yksikirjaimiset muuttujat, mutta toisaalta suurin osa / kaikki näistä ovat yleisesti algoritmejä määriteltäessä käytettyjä lyhenteitä.

Testit näyttävät hyvin selkeiltä, sekä vaikuttavat testaavan algoritmejä hyvin laaja-alaisesti.

Projekti vaikuttaa kaikinpuolin laadukkaalta, enkä mahtavista c++ taidoistani huolimatta keksi mitään parannusehdotuksia.