Ošetření vstupu – C / C++ – Fórum – Programujte.com

Zdravím,
potřeboval bych pomoci s následujícím problémem:

Uživatel zadá číslo z klávesnice, které musí být v rozsahu 2-50 včetně, pokud vloží cokoliv jiného (jiné číslo než z rozsahu, znak,...), tak musí opakovat svojí volbu. V případě vložení správného čísla, se číslo uloží do proměnné.

Díky za pomoc

do{ }while()

Do a while je sice fajn, ale jenom s touhle radou to nevyřeším. Původně jsem měl napsáno tohle:

printf("Zadejte pocet radku (min. 2, max. 50): ");                                               

              while(1){                                                                                              

                       scanf("%d",&pocetR);                       

                       if(pocetR>1 && pocetR<51)

                       {

                                  break;

                       }

                       printf("Spatne zadany pocet radku! Opakujte volbu.\nZadejte pocet radku (min. 2, max. 50): ");

              }        

To Caken : zkus if( scanf("%d",&pocetR)){ if(pocetR>1 && pocetR<51) {}}

To Anonymní uživatel :
To mi nefunguje, pokud to takhle změním a zadám nějaký písmeno, tak se mi to zůstane zacyklené.

To Caken : tak se zbav pismenka

To KIIV : Kdybych věděl jak to ošetřit, tak nepíšu sem :smile1:

To Caken : kdyz se ti nepovede nacist cislo, tak nezvladnes nacist znak ?

#include <stdio.h>

int main()
{
printf( "Zadejte desetinne cislo v intervalu od 2 do 50 : \n" );

int cislo;

while( 1 )
{
scanf( "%d", &cislo );

if( cislo <= 50 && cislo >=2 )
printf ( "Zadali jste : %d", cislo );
else {
printf( "Musite zadat cislo v intervalu od 2 do 50 !" );
continue;
}
}
}

Na ošetření, že to není znak, bych použil knihovnu <ctype.h> a následně funkci isdigit(x);

Tady je to v c++ :

Akorat nevim proc mi program pri zadani neciselneho vstupu zacykli.
vstup jsem vymazal pomoci metody clear ...

#include <iostream>
#include <ctype.h>

int main()
{
std::cout << "Zadejte desetinne cislo v intervalu od 2 do 50 : " << std::endl;

int cislo;

while( 1 )
{
std::cin >> cislo;

if( cislo <= 50 && cislo >=2 )
std::cout << "Zadali jste : " << cislo << std::endl;
else if( !isalnum( cislo ) ) {
std::cout << "Vami zadany znak neni cislo !" << std::endl;
std::cin.clear();
continue;
}
else {
std::cout << "Musite zadat cislo v intervalu od 2 do 50 !" << std::endl;
continue;
}
}
}

To pakr93 : protoze se prekvapive nastavi priznak "neuspech" a dokud se ho nezbavis tak uz se nic dalsiho nenacte

To pakr93 : metoda clear ti právě smaže pouze ten příznak "neúspěch" (o kterém psal KIIV), ale navstupu ti stále ten znak zůstane...

To crAzY^ :
Jak se tedy zbavim toho co je na vstupu a nechám uživatele zadat hodnotu znovu?

To Caken : jednoduše... někam to načti...

string trash;

cin >>trash;

cin.clear();

Akorát, tohle asi funguje jen na C++ ne? Já bych to potřeboval na klasické C. A ještě bych potřeboval - ošetří mi to např. když dám vstup 5a? nebo dejme tomu a5? Aby to obojí vyhodnotilo jako chybu? Jak to popř. upravit? Díky

Nenačítavaj od užívatela vstup v podobe čísla, ale v podobe reťazca znakov

char vstup[50]={0};

scanf("%s", vstup)

To Caken :
Standardní postup:
Opakovat
Načíst vstup jako řetězec.
Převést řetězec na číslo odpovídající funkcí (procedurou)
V případě, že převodní funkce vrátí chybu při převodu, upozornit uživatele.
Skončit opakování jen pokud převod proběhl bez chyby

To JoDiK :
Zkusil jsem fci atoi, jenže tam je ten zádrhel, že když uživatel zadá 0, tak to fce atoi vyhodnotí taky jako chybu... a strtol nebo strtod nevím jak správně použít...
Když budu mít

printf("Dolni mez: ");        

scanf("%s",&vstup);

dm = strtol(vstup,&error,10);

Tak nakonec se mi to povedlo, po chvíli hledání jsem našel co jsem potřeboval, podmínku jsem tam dal tuhle i

f (*error == '\0') break;

To Anonymní uživatel :
Tak po prvé ten ampersand(&) vo funkcii scanf() zmaž, vstup už sám o sebe je ukazovateľ na začiatok pola znakov tak netreba ešte referenciu, preto tam máš asi tú chybu

Po druhé ten reťazec musíš najprv overiť či sa v ňom nachádzajú iba čísla a potom až použiť atoi() teda

 int dobryVstup=1;

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

	if(!(vstup[i]=>'0' && vstup[i]<='9')) dobryVstup=0;



if(dobryVstup) nacitaneCislo = atoi(vstup);