question - program na vstupy od užívateľa (diskuse)

[Kníže Ignor]

Hodnotit smysluplnost programu nebudu. Jen mě zaráží, že kvůli třicetiřádkovýmu programu, musím tahat celých 11 kB

Popravdě jakmile jsem ve zdrojáku viděl třikrát ručně napsanou konstantu 256, přestal jsem to studovat.

Ale ať jsem tedy aspoň trochu konstruktivní:

  odpoved=(char*)malloc(sizeof(char)*256);
...
  for (i=0;i<=256;i++) 
        if (odpoved[i]=='\n') odpoved[i]='\0';

Stane se, no...

[ajikdpoe]

Nerozumiem presne co si tym chcel povedat, alebo to mam ku tvojej spokojnosti prepisat takto:

#define DLZKA

  odpoved=(char*)malloc(sizeof(char)*DLZKA);
...
  for (i=0;i<=DLZKA;i++) 
        if (odpoved[i]=='\n') odpoved[i]='\0';

[Kníže Ignor]

Možná mně vadí spíš to "<=" v tom for-cyklu, ale jsem hloupý, což ty dobře víš.

[ajikdpoe]

Ok, uz vidim... vdaka...

[Michal Vyskočil]

hmm, a mě se vždycky smějí za != ... přece si to budeme pamatovat, že mez je o jednu menší ... já si to nikdy nepamatuji, proto raději používám "nerovná se" .

[Michal Kubeček]

To by samozřejmě bylo lepší z hlediska stylové čistoty, ale buffer overflow vám tam zůstal.

[Michal Kubeček]

Builtin příkaz select se vám nelíbil?

[ajikdpoe]

Najaka ukazka by nebola ???, z manualu som nepochopil ako to mam pouzit...

[David Watzke]

help select

[ajikdpoe]

# select ano in ano nie; do echo napisal si $REPLY; break; done
1) ano
2) nie
#? 2
napisal si 2

asi este stale nechapem, ako spravit aby ked dam 2 napisalo "napisal si nie"

[Michal Vyskočil]

select ano in ano nie; do echo napisal si $REPLY; break; done
1) ano
2) nie
#? nie
napisal si nie

??

[Michal Vyskočil]

Aha, jak kdybych nic neřekl (pomalé vedení, celé ráno se na booku nastavuji s tomcatem, tak mě omluvte)

[ajikdpoe]

REPLY=`question "Otazka" -nodefault ano nie; echo napisal si $REPLY
Otazka (ano/nie): anoo (akoze preklep)
Otazka (ano/nie): ano
napisal si ano

Nieje to lepsie?

[Michal Vyskočil]

je

[Michal Kubeček]

A co takhle místo REPLY použít tu proměnnou, do které jste si výsledek nechal uložit, tedy ano?

[Semo]

select name in ano nie; do
    if [[ ! -z $name ]]; then
        echo napisal si $name
        break
    fi
done

1) ano
2) nie
#? 3
#? 4
#? 5
#? 2
napisal si nie
semo@nb ~ $    

[ajikdpoe]

select ano in ano nie; do echo napisal si $REPLY; break; done
1) ano
2) nie
#? 3
napisal si 3

hmm... nieje to to prave orechove...

[Michal Kubeček]

Hodnotu odpovědi máte v proměnné ano, číslo v REPLY.

[Michal Kubeček]

1. Na buffer overflow už jste byl upozorněn.

2. Nikde nevidím, že byste dealokoval pointer, který jste dostal malloc(). Jistě, systém to po vás uklidí, ale po vzoru klasika bych řekl: "To neděláš dobře, Jaromíre, jednou se ti to vymstí."

3. Parsování parametrů je velmi chaotické a neošetřuje spoustu situací, se kterými jste nepočítal. Např. pokud je použit přepínač '-nodefault', je '-nodefault' považováno i za platnou odpověď.

4. Vyžadovat, aby byl přepínač až za argumentem je dost neobvyklé a v tomto případě neopodstatněné.

5. Nemíchejte odsazování mezerami a tabulátory.

[ajikdpoe]

vdaka za pripomienky

[ajikdpoe]

vacsina chyb je dosledkom mojho stylu "cesta mensieho odporu"

[Michal Kubeček]

Ono ani tak nejde o tento prográmek, tam je to celkem jedno. Jde spíš o to, abyste si nezvykl takto programovat, protože až jednou budete psát něco většího, bude mnohem těžší se tyto zlozvyky odnaučit.