Programování: záhadné chyby

3.3.2007 14:05 | Přečteno: 1665× | Programování | poslední úprava: 10.3.2009 13:32

Zrovna nedávno mě potrápila jedna "záhadná" chybička. Tak se s vámi o ni podělím...

Již celkem pozdě v noci, unavený jsem chtěl rychle přidat další funkce do jednoduchého miniprográmku, aby jsem ho mohl ihned použít a ulehčit si práci. Vždy otevře soubor, zjistí jestli je v něm to co chci a případně to vypíše na obrazovku. Má dva režimy používání. V tom prvním pracuje pouze s jedním souborem a v tom druhém projede rekurzivně všechny soubory v zadaném adresáři.

Nové funkce nechtěly stále fungovat, nu co tak přidám ladící printf část, abych odhalil kde je chyba. No jo, ale po kompilaci se mi nic na obrazovku nevytisklo. Říkam si, že je to nějaké divné, na ten řádek se to přeci musí tak jako tak dostat. A v tu chvíli jsem udělal zásadní chybu. Já byl býk, těch pár řádků pohybující se červená muleta (BTW: Býk je barvoslepý, dráždí ho pohyb mulety a bolest, kterou mu toreador způsobuje, ne barva. Taky jsem se totiž stal obětí tohoto mýtu ) a zbytek kódu byl toreador.

Nu což upravil jsem i přímo řádky, které provádějí výpis informací ze souboru, ale spuštěný program opět vypsal to samé jako prve. To přeci není možné. V programu není nikde jinde místo, kde by se tyto informace tiskly. Pustím program s parametrem --version a kontroluju čas kompilace, ten sedí, opravdu spouštím nově zkompilovanou binárku. V tuto chvíli chybuji znovu, začínám věřit na abnormální jevy místo, abych použil logiku a zjistil příčinu. Hlavou se mi honí podivné myšlenky. Zakomentuju printf a program mi stále tiskne informace. Jak je to možné?

Dám jeden velký printf("**********************************\n"); hned na začátek programu. Celou dobu jsem volal program s volbou pro rekurzivní zpracování více souborů v adresáři a teď se divil, že se mi hvězdičky vytiskly pouze při zpracování prvního souboru. Ha, konečně mi to došlo.

První verze programu uměla zpracovávat pouze jeden soubor a teprve po odladění jsem dodal podporu pro více souborů najdenou. A proč vymýšlet již hotové věci, že? Jako nejrychlejší řešení mi přišlo toto:

  if (user_arg.dirname[0] != 0) {
    char command[COMMAND_MAX];

    memset(command, 0, COMMAND_MAX);
    snprintf(command, COMMAND_MAX, "find %s -type f -regex '.*fit' | xargs -i %s -f {} -k '%s' -v '%s' -c '%s'",
             user_arg.dirname, APP_NAME, user_arg.keyword, user_arg.value, user_arg.compare);
    if (user_arg.ignore_case == 1)
      strcat(command, " -i");
    if (user_arg.print_filename == 1)
      strcat(command, " --print-filename");

    system(command);
    exit(EXIT_SUCCESS);
  }

Po 's/APP_NAME/argv[0]/' najednou zmizely nadpřirozené jevy, vztek, všechno začalo krásně fungovat a já se rozesmál. V /usr/local/bin byla stará binárka, takže jsem sice opravdu spouštěl binárku novou, ale vzápětí "skrytě" volal tu starou

Takže ponaučení zní: "Nikdy se přehnaně nesoustřeďte na jedno místo, ale vždy se na problém podívejte s dostatečným nadhledem a při jeho řešení berte v úvahu vše co s ním souvisí. Krok za krokem kontrolujte všechny články řetězce a dříve či později musíte narazit na ten vadný. Zachovejte chladnou hlavu."

Snadno se radí, špatně se koná. Občas je člověk v takovém psychickém stavu, že udělá věci, které by jinak neudělal...

       

Komentáře

[Lu-Tze]

Nelíbí se mi to Zdá se mi, že by bylo lepší, aby tvůj program akceptoval na vstupu libovolné množství souborů a v případě rekurzivního použití by se volal skriptíkem, kde by byl ten find atd. Tohle je fuj.

[tommy]

ja poznam jeden velmi zakerny kod v jave

ResultSet xy

..

..

while (xy.next());

{

xy.get(....

}

niekolko hodin som stravil premyslanim preco mi to vracia 1 zanam ked ich tam je 9 tisic....

[Bluebear]

Jo, na tohle už jsem si taky párkrát naběhl

Myslím, že na syntaktické anomálie, jako je tahle, by měl být v kompilátoru warning. Osobně bych dokonce byl pro to, aby ty složené závorky za podmínkou v příkazu "if" byly povinné.

[xxx]

Ha, konecne jsem pochopil o co slo

[jk]

Me Eclise tuhle konstrukci oznaci warningem - Emty flow statement.

[tommy]

Tiez pouzivam Eclipse, a vsimol som si az vted ked som tam kvoli testovaniu

zmenil na while (true);, a zrazu mi napisalo ze code unrechable....

Ja osobne by som bol aspon za nejaky kompiler warning, lebo hladanie

takychto chyb ma pripravuje o psychicke zdravie

[jk]

Code unreachable imho neni warning ale error. A napsal ti ho proto, ze tam mas nekonecnej cyklus, kterej neobsahuje return, break, nebo neco podobnyho, s tim strednikem nesouvisi.

Jinak ten empty flow statement warning mozna neni standartne zapnutej. Ja jsem si v Eclipse zapnul skoro vsechny mozny warningy, takze mozna byl mezi nima.

[happy barney]

Osobně bych dokonce byl pro to, aby ty složené závorky za podmínkou v příkazu "if" byly povinné.

V tom prípade, perl je jazyk pre vás
Inak, celý tento problém s bodkočiarkou je len nafúknutá bublina. Stačí mať otváraciu { na riadku s while/if/...
a potom už v len Makefile spočítať riadky, kde s m/;\s*{/ )

[Josef Kufner]

Hádej co mám v patičce

Btw, neinicializovaný pointer dokáže ještě lepší věci. Jednou mi program v GTK chcípnul, když jsem při inicializaci socketu předal jedné funkci neinicializovaný pointer. Všechno se zdálo být funkční, povídalo si to, nespadlo to, okno se překreslovalo, ale jakmile jsem z jednoho GtkHBoxu odstranil poslední widget, tak si to se mnou přestalo povídat. Okno se i nadále překreslovalo, ale prostě to nic nedělalo... Od té doby inicializuju úplně všecky pointery i když vím, že je to zbytečné. Pak to totiž spadne hned