Dotaz: jazyk c, argv na unsigned int

[Tomasko]

Ahoj. Začínám tak prosím laické vysvětlování, děkuji. Potřebuji jeden parametr argv[1] převést na unsigned int a ještě zamezit, aby se v parametru nevyskytovaly písmena atd. Jen číslo od 0 do UINT_MAX.

char *end; //pouzito pro kontrolu konverze
unsigned int pocet_zidli= strtoul(argv[1], &end, 0);
	if (*end != '\0')
	{//osetreni vstupu argv[1]
		fprintf(stderr, "%s\n", chybove_zpravy[CHYBA_ARGV1]);
		return -1;
	}

zkoušel jsem to takhle. strtoul ale převádí string na unsigned long a já to uložím do unsigned int, je to problém? A pak ještě mi to hází když zadám např -1, nějaké hodně velké číslo, jak to prosím ošetřím děkuji.

Odpovědi

[Jirka P]

Asi nejjednodušší je napsat si ten převod string -> int sám, není to nic těžkého. Budete tak mít plnou kontrolu nad tím, co to dělá. Takhle máte kromě záporných čísel neošetřené vstupy typu "010" (záleží, co s tím chcete).

Ukládání do unsigned int není problém, dokud se ta hodnota do typu unsigned int vejde :)

[stativ]

No, při použití strtoul() musíš zkontrolovat ještě hodnotu errno. To je právě ten případ kdy ti to vrátí velké číslo (konkrétně ULONG_MAX), protože jsi mimo rozsah unsigned long.

To, že to ukládáš do int může a nemusí být problém. Záleží na tom, jestli se může stát, že uživatel zadá číslo větší než je rozsah int. Pro ten případ by možná nebylo od věci před konverzí zkontrolovat, zda se vstup vleze do rozsahu int.

Pak máš ještě možnost použít sscanf().

[Jirka P]

Ono strtoul("-1", 0, 0) sice vrátí ULONG_MAX, ale errno nenastaví, protože k přetečení nedojde. Ono je to v té manuálovce trochu jinak...

[stativ]

No, k přetečení dojde, akorát ho strtoul zřejmě není schopno detekovat. V tom případě by bylo lepší použít strtol() a kontrolovat záporné hodnoty ručně.

[stativ]

Tak koukám, že jsi mě se strtol() předběhl.

[Tomasko]

napsal jsem to takhle:

        char *end;
        int pocet_zidli= strtoul(argv[1], &end, 0);
	if (errno == ERANGE || *end != '\0' || pocet_zidli < 0)
	{//osetreni vstupu argv[1]
		fprintf(stderr, "%s\n", chybove_zpravy[CHYBA_ARGV]);
		return -1;
	}

btw nejsou nekde man. stranky v CZ, angličtinu neovladam a resim skolni projekt s procesy a vubec nevim kde hledat info :/ jak zacit. Pritom pri prihlasovani na Bc. nikde nebylo psano, ze vsechno bude v AJ, kdyz se anglictina vyucuje az na Bc. To uz je jina kapitola..

[Jirka P]

1) To není dobrý nápad. Konverze unsigned int -> int je nedefinovaná pro (unsigned)-1. Když už jdete do znaménkových typů, použijte strtol.

2) Chybu (téměř) vždy indikuje návratová hodnota funkce, ne errno, to pouze označuje, ke které konkrétní chybě došlo. Kdyžtak to errno aspoň vynulujte na začátku, aby tam nezůstávaly staré zbytky.

3) Manuálovky v češtině asi těžko, ale existují knihy (i české) o C, kde je to popsané. Poptejte se v knihovně.

[Sten]

strtoul převádí na unsigned long, nikoliv unsigned int. Pokud to rovnou převedete na unsigned int, nemáte šanci detekovat přetečení mezi UINT_MAX a ULONG_MAX. Takže lépe si takto nadefinujte strtoui:

int strtoui(const char *str, char **end, int base)
{
    unsigned long l = strtoul(str, end, base);
    if (l > UINT_MAX) {
        errno = ERANGE;
        return UINT_MAX;
    }
    return l;
}

A nezapomeňte na to, že strtoul nemaže errno, pokud nedošlo k chybě, takže si jej před jejím voláním vymažte, tedy volání by mělo vypadat nějak takto:

char *end;
errno = 0;
unsigned int pocet_zidli = strtoui(argv[1], &end, 0);
if (errno == ERANGE || *end != 0) {
    // Chyba
}

[Tomasko]

Děkuji Vám. Ale chci se zeptat, stejně když dám na vstup -100, tak to vypíše 4294967295. Mělo by to vyhodit chybu.

[Sten]

man strtoul:

Negative values are considered valid input and are silently converted to the equivalent unsigned long int value.

Takže řešení je použít strtol (strtoll pokud chcete celý rozsah unsigned long) a ošetřit obě meze.