mjaakko / gtfs-route-finder Goto Github PK

Verkon luonti riskialtista, algoritmit ovat siisteja.

• Itse hakualoritmi ja verkko-rakenne ovat siistejä.

• Keko on altis virheitä vastaan, kun pollataan tyhjä keko pariin kertaan. itemcount menee negatiiviseksi ja jää siksi kunnes lisätään tarpeeksi solmuja.

• Tampereella heittää erroria, vaikka sanottiin toimivan. Tästä ei sen enempään pystynyt jatkavan itse ohjelman toimintaa.

Exception in thread "main" java.lang.IllegalArgumentException: Mapping for stop_id not found, expected one of [stop_id, stop_code, stop_name, stop_lat, stop_lon, zone_id]
	at org.apache.commons.csv.CSVRecord.get(CSVRecord.java:102)
	at xyz.malkki.gtfsroutefinder.gtfs.model.core.Stop.lambda$parseFromFile$0(Stop.java:67)
	at xyz.malkki.gtfsroutefinder.gtfs.utils.GTFSParser.lambda$parseFromFile$0(GTFSParser.java:26)
	at java.base/java.lang.Iterable.forEach(Iterable.java:75)
	at xyz.malkki.gtfsroutefinder.gtfs.utils.GTFSParser.parseFromFile(GTFSParser.java:26)
	at xyz.malkki.gtfsroutefinder.gtfs.model.core.Stop.parseFromFile(Stop.java:66)
	at xyz.malkki.gtfsroutefinder.gtfs.utils.GTFSGraphBuilder.buildFromFiles(GTFSGraphBuilder.java:30)
	at xyz.malkki.gtfsroutefinder.Main.main(Main.java:37)

• Geohashissä puhutaan tasoista, mikä näiden tarkoitus on?

• Testit ovat aika keskeneräisiä. Haulla meitä kiinnostaa lyhimmät reitit yleensä ja tällöin polun hyppymäärä ei ole validi testattavaksi. -> Kannattaa laskea polun paino ja vertaa siihen testattavaa arvoa.

• Main luokka on stantardin mukaan vain startter luokka. Tämä toisaalta on makuasia. Haluuko tätä jakaa omaksi app luokaksi vai ei.

Itseäni eniten häiritsi syötteen polun antaminen kansioon.

• Onko polku suhteellinen softan sijaintiin vai ei vai molemmat?
• Halutaanko kansion koko polku vai kansion sijainti?
• Rajaus polulle puuttuu. mm polku verkolle sijaitsevaan kansioon.
• Voisiko tällä olla jokin oletus sijainti josta haetaan.
• Halutaanko zippiä vai purattua kansiota syötteeksi vai kumpikin käy?

Had some problems getting the thing to start, but once I got it to work, all was good. A file ready in the system could be nice, in order to lessen the amount of hassle with the gtfs files.

Also some added fault tolerance would be nice, so the system wouldn't default into crashing.

Otherwise code is clear, well commented, tests are well made.