Walidacja wejścia - Jak to usprawnić?

Question

Witam. W moim przypadku w tym wejściu musi być koniecznie liczba nieujemna, oraz oczywiście nie może być innym znakiem niż liczba. Musi to być również liczba całkowita, czyli nie ma mowy o czymś takim jak 5.2. Moja walidacja nie obejmuje również przypadku gdy ktoś wpisze np. 5w. Napisałem walidację liczby ujemnej i znaku, ale z liczbą po przecinku i przykładowym 5w nie mogę sobie poradzić. Proszę więc was o pomoc.

 

Mój kod: 

int ile_elementow;

    cin>>ile_elementow;

    do
    {
        if (ile_elementow<0)
        {
            cout<<"Ta liczba nie moze byc mniejsza od zera, wprowadz ja ponowownie!"<<endl;
            cin>>ile_elementow;
        }

        while(cin.fail())
        {
            cout<<"To nie jest liczba, wprowadz liczbe ponownie!"<<endl;
            cin.clear();
            cin.ignore(256, '\n');
            cin>>ile_elementow;
        }
    } while (ile_elementow<0);

 

Answer 1

Jeśli masz takie konkretne wymogi co do podanej liczby, to ja proponuję tak:

Wczytaj liczbę getline() do jakiegoś string. I przekonwertuj ją do int za pomocą funkcji stoi() (C++11). Funkcja ta konwertuje do int, więc jeśli napotka gdzieś kropkę albo inny znak niebędący cyfrą (poza minusem lub plusem), to wyrzuci wyjątek invalid_argument. Jedyne co Ci pozostanie, to sprawdzić czy liczba jest ujemna. Taka walidacja uwzględni wszystkie możliwe przypadki. Żadna nie-liczba się nie przemknie :-P

To był mój pomysł na rozwiązanie problemu. Może ktoś ma jeszcze inny. Pozdrawiam :-)

Comments

A co do mojego częściowego rozwiązania dorzuciłbyś jakieś zastrzeżenia? Chciałbym wiedzieć czy miałem dobry tok myślenia podczas pisania tego :)

---

Sposób jest dobrze pomyślany, ale problemem jest tutaj cin, który jak sam słusznie zauważyłeś nie pozwoli Ci obsłużyć sytuacji, gdy ktoś poda 5w. cin wczyta 5, a literkę w pozostawi w strumieniu. To znaczy, że kolejny cin jaki wystąpi w kodzie wczyta z automatu tę literę.

Tok myślenia jak najbardziej poprawny. Tylko ten cin tutaj jest problemem :-)

---

Moje rozwiązanie: http://wklej.to/Amo5u

---

Bardzo ciekawe rozwiązanie z tą pętlą wyciągającą pozostałe znaki z linijki. Jednak tutaj byłby problem z odróżnieniem czy użytkownik podał nieprawidłową liczbę czy może mniejszą od zera.

Twój program też nie uwzględnia przypadku, przed którym chciał się ustrzec autor. Mianowicie, wpisanie do konsoli: 123abc powinno być zasygnalizowane błędem. U Ciebie funkcja wczyta 123, a pozostałe znaki odrzuci.

Ciekawe jest też działanie warunku  !(std::cin>>ret) bo operator strumienia >> zwraca obiekt klasy istream. Z takiego zapisania warunku wynikałoby, że jeśli wczytanie się nie powiedzie, to operator zwraca NULL. Pewnie tak jest. Szkoda tylko, że nigdzie nie piszą o tym w dokumentacji na cplusplus.com 

---

Co do w warunku w while. Kiedy wczytywanie danych za pomocą '>>' się nie powiedzie to ustawiana jest flaga badbit a wtedy metoda fail zwraca true. Więc ten warunek działa jak: "cin.fail()" albo krócej: "!cin". Ale twoje rozwiązanie jest lepsze biorąc pod uwagę wymagania autora tematu.