Vyhledávání dopravních spojení v CLI potřetí

24.7.2008 12:57 | Přečteno: 1199× | poslední úprava: 24.7.2008 14:23

Při zveřejnění svého skriptu jsem tak trochu doufal, že se najde někdo, kdo by mi s jeho úpravou pomohl, respektive jej alespoň okomentoval (které části přepsat a jak, co je hrůza, co ujde...) a já bych měl tak možnost se naučit psát "čistší" programy. To se bohužel zatím nestalo, nicméně já na svůj příšerný skript nezanevřel a proběhla další várka úprav. Takže co je nového?

Nejvetší změnou je přechod od dlouhého_skriptu_s_minimem_procedur k programu sestávajícího se ze třídy a několika kratších metod. To podstatně usnadní jeho použití i v jiných programech. Další změnou je rozdělení komunikace se serverem IDOSu do více kroků a změna zpracování výstupů. Výsledky (nalezená spojení či možnosti na výběr při nepřesně zadaných koncových bodech) jsou dostupné i prostřednictvím seznamů. Dále jsem se zbavil dočasných souborů a (povetšinou) přešel na unicode retězce. A v neposlední řadě jsem opravil chybu, která nastávala při nepřesně zadaném počátečním i koncovém bodě hledaného spojení. Uživatelům, jsou-li jací, proto doporučuji přechod na novějši verzi.

Budu rád, když se ke skriptu vyjádříte a navrhnete další úpravy a vylepšení. Dostupný je stále na stejné adrese.

UPDATE: Jak už to tak chodí, chvíli po zveřejnění zápisku jsem nalezl chybu. Při zadání neexistujícího cíle se skript zacyklil. Již je to (doufám) opraveno, stahujte tedy verzi 0.35.

       

Komentáře

[luke7]

Uživatele máte, je to velmi zajímavý projekt a fandím mu. Jednou bych to rád použil jako jabberbota. S vývojem bych pomohl, ale vypadá to, že není třeba , navíc python zrovna nepoužívám...

[kkaarreell]

Diky za podporu. Pokud jde o jabberbota, tak jeden takovy jiz existuje, byl zminen v diskuzi u jednoho z predchozich blogu. A co se pomoci s vyvojem tyce, tak bych ji uvital, ikdyz, pravda, od nekoho v pythonu zkusenem. V blizsi dobe neplanuju pridavat zadne nove funkce, jen procistit kod. Stacilo by mi okomentovat "nepekne pasaze" (a ze jich je) a poradit, jak je upravit.

[rajcze]

No, ja ti s pythonem klidne poradim :) nejsem sice nejaky velky guru, ale nejake zkusenosti uz sem nabral :)

Kdyztak se ozvi na jabberu rajcze@njs.netlab.cz

Joza

[luke7]

ďěkuji za odkaz, to vypadá pěkně, ale stejně si možná udělám vlastní, hodila by se funkce nějaké proxy a prohlížení webu v osekané textové verzi pomocí bombusu v mobilu...

[manasekp]

mam trochu OT dotaz, pyhon nepouzivam a proto me prekvapily ty podminky ve skriptu, treba

if kod == KOD_NEJEDNOZNACNE_KONCOVE_BODY: ...
else: ...

da se v pythonu pouzit i "normalni" zapis if(podm){} ?
je nejaky duvod pro pouziti te syntaxe co tam mas ty?

[David Watzke]

Ne, protože Python není "normální" jazyk Bloky se odlišují pomocí odsazení... je to šílený.

[Václav "Darm" Novák]

Na druhou stranu to učí nováčky slušně formátovat kód.

[CEST]

Uraaa, a mame tady flame

No, jak se to vezme. Me jako navacka v pythonu a pokrocilyho v bashi, perlu a amatera jinych docela hodne zmatlo, kdyz jsem pripsal jeden radek v python kodu a pri spusteni me to vyfuckovalo, ze mam blby zarovnani. Podival jsem se znovu, zarovnani bylo ok. Pak mi teprve doslo, ze python sice rozpoznava bloky podle odsazeni, ale ne podle vizulaniho odsazeni, ale podle poctu znaku. Takze milej uzivateli, neni whitespace jako whitespace. 8 mezer PROSTE NENI stejne jako 4 taby nastaveni na sirku 2 znaky.

Jinak me to taky docela stve z toho duvodu, ze nemuzu napsat python kod na jeden radek, napr. v cmdline.

No, python je mozna pro zacatecniky, ale ne pro pokrocily. Proto se u me moc nechyt. Ale ne, trosku mi tam vadi i funkce, schovavany za pseudo metody. Na tom se pak zacatecnici uci pekne blbe OOP.

[JS]

A co je podle vas vizualni odsazeni? Pocitac to vidi jak je to v tom souboru. To ze ruzne programy interpretuji delku tabu ruzne, je problem s taby, ne Pythonem. Doporucoval bych v Python zdrojacich taby vubec nepouzivat (mit v editoru nastavene nahrad taby mezerami).

[Václav "Darm" Novák]

Rozumím tomu dobře že používáš ve svých projektech taby i mezery, podle toho jak se to hodí? Nevede to pak k rozházení vizuální stránky kódu když to otevřeš v editoru s jinak nastavenou délkou tabu?

[zde]

> 8 mezer PROSTE NENI stejne jako 4 taby nastaveni na sirku 2 znaky.

Kterej kokot si nastavuje TAB na dva znaky?

[Smajchl]

co je na tom spatne?

[manasekp]

zajimave, jak to teda pozna kde je konec podminky? kdyz budu met treba 10 vnorenych podminek a z toho nektere budou met i else cast tak uz prece neni mozne se v tom rozumne orientovat, kdezto pekne naformatovany kod s pouzitim zavorek je prehledny vzdycky.

[Raduz]

No...v pythonu se v tom vyznáš taky, protože máš "pěkně naformátovaný kód" tak nějak z principu věci.

[JS]

V Pythonu se konec podminky pozna dvojteckou. Jak proste!

Navic v dobe, kdy Guido jeste delal jazyk ABC, tak delali nejake usability testy, a zjistili, ze ta dvojtecka tomu pridava na citelnosti.

[Raduz]

Co to je "normální" zápis? Python to prostě takhle má - podmínka končí dvojtečkou, blok kódu pro danou podmínku je odsazený whitespacem (případně je napsaný hned za dvojtečkou, pokud je toho jen na jeden rádek).

[JS]

Probiram se trochu tim zdrojakem, a ma k tomu par doporuceni:

1. Misto dict_opts.has_key("-d") je doporuceno pouzivat zapis "-d" in dict_opts. Navic cela ta podminka v nastavovani parametru by sla prepsat pomoci int(dict_opts.get("-d",5)).

2. V te funkci parsuj_nalezene_spojeni by asi bylo lepe pouzit generator na radky, a misto toho oskliveho idiomu if faze_hledani == ?? pokazde iterovat skrze nej dokud by se nenarazilo na prislusny zaznam. Kratsi, citelnejsi a rychlejsi. Totez pro ostatni podobne funkce. Prave generatory maji vyhodu, ze se pomoci nich citelne implementuji stavove automaty, coz je tento pripad.

[kkaarreell]

Diky. O generatorech jsem precetl jen neco malo. Zkusim si k nim jeste neco najit.. Ale muzes mi zlehka nastinit, jak by ten kod vypadal? Neni mi uplne jasne, jak by se implementovaly ty jednotlive stavy (faze hledani). Respektive, tenkrat jsem to tak napsal prave proto, abych se v tom vyznal a nedari se mi vymyslet srozumitelnejsi reseni s generatory. Jeste jednou diky.

[JS]

Jsem ted na mitinku, takze to mozna nebude davat uplne smysl, ale zhruba takto:

def parsuj_spojeni():

  def generator_radku(file):
    for line in file:
      .. nejake zpracovani vstupni radky line
      if pocet_spoju > max_pocet_spoju: break
      yield line
    return

  pocet_spoju=0
  gr=generator_radku(file)
  
  for line in gr:
    while not re.search("cosi",line): gr.next()
    .. zpracovani prvniho stavu
    while not re.search("cosi dalsiho",line): gr.next()
    .. zpracovani druheho stavu
    ...
    pocet_spoju+=1

Zkratka, zakladni myslenka je, mit vstupy (v tomto pripade radky) ktera chcete prijimat tim stavovym automatem schovane v tom generatoru, a stavy toho automatu implementovat jako normalni body v kodu, ktere si proste vzdy vyzadaji prislusny ocekavany vstup (bud pomoci gr.next() nebo pomoci for cyklu pres gr).

Tak bych to asi delal ja, je to citelnejsi nez studovat prechod mezi stavy a co ktery stav znamena.

[kkaarreell]

Diky, to vypada dobre.