Ochrona programu przed znakami.

Question

Witam. Mam problem. Zrobiłem program do losowania liczb, ale nie wiem jak go zabezpieczyć tak aby użytkownik nie wpisywał znaków tylko same liczby. Prosiłbym o jakąś radę. Z góry dzięki.

#include <iostream>
#include <cstdio>
#include <windows.h>
#include <clocale>
#include <cstring>
#include <time.h>

using namespace std;

struct RRecordData
{
    int ID;
    int Number;
};

int RandRange( int min, int max )
{
    return min + rand() % (1 + max - min);
}

void GetBalls(int *numbers, int *mini, int *maxi)
{

    printf( "Podaj ilosc kulek do wylosowania: " );
    scanf( "%d", numbers);

    printf( "Podaj najmniejsza wartosc losowanej kulki: " );
    scanf( "%d", mini);

    printf( "Podaj najwieksza wartosc losowanej kulki: " );
    scanf( "%d", maxi);

}

void GetRandomNumbers(int *num, int *mini, int *maxi)
{
    RRecordData * Losowanie;
    Losowanie = new RRecordData[ *num + 1];

    srand( time( NULL ) );

    for( int i = 1; i <= *num; i++ )
    {
        Losowanie[ i ].ID = i;
        Losowanie[ i ].Number = RandRange( *mini, *maxi );
    }

    printf( "----------------\n" );
    printf( "|  ID  | Number |\n" );
    printf( "-----------------\n" );

    for( int j = 1; j<=*num; j++ )
        printf( "| %4d | %4d   |\n", Losowanie[ j ].ID, Losowanie[ j ].Number );

    delete[] Losowanie;

}

int main()
{
    int mini, maxi;
    int numbers;

    GetBalls(&mini, &maxi, &numbers);

    GetRandomNumbers(&mini, &maxi, &numbers);

    return 0;
}

 

Answer 1

Ochrona przed wpisywaniem liter to drobnostka. Biblioteka <cctype> ma szereg funkcji sprawdzających czy dany znak jest cyfrą, czy literą (czy jeszcze znakiem pisanym, warto z nich korzystać tak myślę, bo z reguły powinny być bardziej przenośne niż to co sobie sami napiszemy) →http://www.cplusplus.com/reference/cctype/isalpha/
Polecam też pilnować czy liczby nie są wprowadzane w złej kolejności (najpierw maksymalna, później minimalna). Nie są wprowadzane liczby ujemne (np. ujemna liczba kul do wylosowania → wprowadzić unsigned).

Comments

Dziękuje za pomoc. Spróbuje skorzystać z tych funkcji.

Pozdrawiam

---

Prosiłbym o opinie. Zrobilem takie coś. 

  while(numbers < 0 || mini > maxi)
    {
        printf("Podano niepoprawne wartosci. Sprobuj jeszcze raz.\n\n");
        GetBalls(&numbers, &mini, &maxi);
    }

Warunek, który sprawdza czy ilość kulek nie jest ujemna i czy minimalna liczba nie jest większa od maksymalnej. Aby chronić program przed wprowadzaniem znaków przez użytkownika użyłem atoi. Nie mogłem wpaść na żaden inny pomysł. 

void GetBalls(int *numbers, int *mini, int *maxi)
{
    unsigned int bufferlength = 50;
    char tablica[bufferlength];
    bool NumberTrue = true;
    bool NumberTrue1 = true;
    bool NumberTrue2 = true;

    while(NumberTrue==true)
    {
        printf( "Podaj ilosc kulek do wylosowania: ");
        fgets(tablica, bufferlength, stdin);

        *numbers = atoi(tablica);

        if(*numbers==0)
            printf("Podano nieprawidlowa wartosc. Sproboj ponownie.\n\n");
        else
            NumberTrue=false;
    }

    while(NumberTrue1==true)
    {
        printf( "Podaj minimalna wartosc wylosowanej kulki: ");
        fgets(tablica, bufferlength, stdin);

        *mini= atoi(tablica);

        if(*mini==0)
            printf("Podano nieprawidlowa wartosc. Sproboj ponownie.\n\n");
        else
            NumberTrue1=false;
    }

    while(NumberTrue2==true)
    {
        printf( "Podaj maksymalna wartosc wylosowanej kulki: ");
        fgets(tablica, bufferlength, stdin);

        *maxi = atoi(tablica);

        if(*maxi==0)
            printf("Podano nieprawidlowa wartosc. Sproboj ponownie.\n\n");
        else
            NumberTrue2=false;
    }

}

 

---

Wydaje się okej.

PS w głównym kodzie masz błąd:

linia 35: void GetRandomNumbers(int *num, int *mini, int *maxi)

A w funkcji main przekazujesz parametry na odwrót:

linia 66: GetRandomNumbers(&mini, &maxi, &numbers);