Dotaz: Postfixova kalkulacka v C: osetreni neplatnych vstupu

[tomes.io]

Ahoj, tvořím postfixovou kalkulačku a potýkám se s problémem, jak ošetřit neplatné vstupy (špatná čísla -123abc není číslo-, příliš málo operandů pro operátor, úplně prázdný vstup).

Nize uvadim mou implementaci (bez definic fcí, pouze main, na pochopeni by to melo stacit). Ošetřil jsem případ dělení nulou a neplatného operátoru. Dotaz bych rozdělil do dvou:

PRVNI - zaporna čísla:

Tato podoba neumí pracovat se zápornými čísly - považuje je za neplatný vstup. Když však odstraním poslední ELSE s warningem:

           else {

             printf("CHYBA: Neplatny vstup!\n");
             exit(1);

tak mi kalkulačka počítá snadno i se zápornými čísly, ale také načítá veškeré další znaky, které při výpočtu ignoruje, ale na vstupu by být neměly.

Otázka tedy zní, jak tyto znaky odfiltrovat (či jinak ošetřit), aby na vstupu byla pouze čísla, platné operátory a vše ostatní by končilo errorem a koncem programu s příslušnou hláškou. Snad by se to dalo vyřešit nějakou vhodnou podmínkou na vstupu?

DRUHY - málo nebo žádné znaky na vstupu:

Zatím jsem nepřišel na to, jak ošetřit vstup, nak terém není nic, mezera, mezera a pouze číslo, mezera číslo a operátor. Prostě nedostatečný vstup. Stand nějak využít fci strncmp?

Byl bych rád, kdybyste mě někdo nakopnuli správným směrem. Píšu to od rána a už mi to nemyslí :)

Zde má dosavadní implementace.

int main()
 {
     //int vysledek;
     int numInp;
     char vstup[15];
     int op;
     int vysledek = 0;
     int i;
     int end = 0;
     
     while(end != EOF){

         end = scanf("%s", vstup);   //Program postupně bude brát slova ze vstupu...
         if(end == EOF)
             break;



     numInp = atoi(vstup);

     for(i = 0; i < strlen(vstup); i++){

         if(isdigit(vstup[i])) {     //pokud načetl číslo...

             numInp = atoi(vstup);
             push(numInp);           //...přidá jej na vrchol zásobníku...

         }



         else if(strlen(vstup) == 1){

                    switch(vstup[i])    /* ...pokud načetl operátor, vezme dvě čísla z vrcholu zásobníku,
                                        *aplikuje na ně načtený operátor a výsledek uloží zpět na zásobník.*/
                    {
                    case '+':
                        vysledek = push(pop() + pop());
                        break;
                    case '-':
                        op = pop();
                        vysledek = push(pop() - op);
                        break;
                    case '*':
                        vysledek = push(pop() * pop());
                        break;
                    case '/':
                        op = pop();
                        if (op == 0){                          
                             printf("Nelze delit nulou\n");
                             exit(1);
                           }
                        else
                        vysledek = push(pop() / op);
                        break;
                    default:
                        printf("CHYBA: Neplatny vstup!\n"); //všechny jiné znaky (jednoznakové) než výše uvedené operandy
                        exit(1);
                    }
             }

           else {

             printf("CHYBA: Neplatny vstup!\n"); // filtruje "smetí" typu sdf125, ale i záporná čísla
             exit(1);
         }

       }
     }

     printf("%d\n", vysledek);
 return 0;
}

---

---

Odpovědi

[kuka]

To si musis vyjasnit, co vlastne potrebujes. Pokud najdes cislici, tak na to pustis atoi. To samozrejme neni optimalni, viz man atoi. Pokud je vstup primo od uzivatele, je treba ho neustale kontrlovat s jista paranoia je na miste. Tzn. napriklad nacitat tak dlouho, dokud jsou na vstupu cislice nebo desetinna tecka (unarni minus, scientific notace atd...) a terprve pak to prevadet na cislo. Podobne se osetri nespravna sekvence oparatoru (dvakrat plus za sebou) apod.

[Sten]

atoi je zlo. Používejte strtol a příbuzené funkce.

[tomes.io]

Ano, strol to vyresil. Uzaviram tedy problem.

[tomes.io]

no tak koukam, ze ne uplne. pri odecitani napr 4 0 - mi to haze zaporny vysledek.

[tomes.io]

no, vyreseno, byla to jen drobnost :)

[Fuky]

$ man strtol strtod strtof strtold

int
main(int argc, char *argv[])
{
    int base;
    char *endptr, *str;
    long val;

    if (argc < 2) {
        fprintf(stderr, "Usage: %s str [base]\n", argv[0]);
        exit(EXIT_FAILURE);
    }

    str = argv[1];
    base = (argc > 2) ? atoi(argv[2]) : 10;

    errno = 0;    /* To distinguish success/failure after call */
    val = strtol(str, &endptr, base);

    /* Check for various possible errors */

    if ((errno == ERANGE && (val == LONG_MAX || val == LONG_MIN))
            || (errno != 0 && val == 0)) {
        perror("strtol");
        exit(EXIT_FAILURE);
    }

    if (endptr == str) {
        fprintf(stderr, "No digits were found\n");
        exit(EXIT_FAILURE);
    }

    /* If we got here, strtol() successfully parsed a number */

    printf("strtol() returned %ld\n", val);

    if (*endptr != '\0')        /* Not necessarily an error... */
        printf("Further characters after number: %s\n", endptr);

    exit(EXIT_SUCCESS);
}

$ ./a.out 123
strtol() returned 123
$ ./a.out '    123'
strtol() returned 123
$ ./a.out 123abc
strtol() returned 123
Further characters after number: abc
$ ./a.out 123abc 55
strtol: Invalid argument
$ ./a.out ''
No digits were found
$ ./a.out 4000000000
strtol: Numerical result out of range

[tomes.io]

Fuky, diky za hint. strtol() je ta fce, kterou asi hledam. Zkusim a dam vedet.

[rADOn]

Nehledáš náhodou nějaký ten flex/bison nebo něco podobného?

[tomes.io]

az doted, nez jsem si to diky tobe vygoolil, jsem o zadnem bisonu neslysel :)