Sprawdzanie poprawności wpisanych danych

Question

Witam, chce sprawdzić czy tekst wpisany przez użytkownika w konsoli jest liczbą. Napisałem taki program:

#include <iostream>
using namespace std;

int main()
{
    int naturalna;
    cout << "Podaj liczbe naturalna:" << endl;
    while(!(cin >> naturalna && naturalna >= 0))
    {
        cin.clear();
        cin.sync();
    }
    cout << "Podana liczba to: " << naturalna;
    return 0;
}

I tutaj wszystko działa, wypisuje liczbę dopiero gdy jest większa lub równa 0. Problemem jest jednak to, że gdy wpisze do konsoli np. 3873wjdd to utnie mi litery a stwierdzi, że poprawnie wpisano dane i liczba to 3873. Jak zaradzić czemuś takiemu?

Answer 1

Jeżeli chcesz sprawdzić poprawność absolutnie całego input'u, to nie ma wyjścia – musisz wczytać to jako tekst i sprawdzać literka po literce czy wszystkie podane znaki to są cyfry.
Jedynym wyjątkiem może być to, że przed liczbą może stać '+' (plus) lub '-' (minus). Ale Ty chcesz tylko liczby nautralne, więc tych przypadków nie rozpatrzę.

Kod zawsze uczy więcej niż tysiąc słów, więc:

include <iostream>
#include <string>
using namespace std;

int main()
{
	bool liczba = true;
	string naturalna;
	cout << "Podaj liczbe naturalna: ";
	cin >> naturalna;

	for( int i = 0; i < naturalna.length(); i++ )
	{
		if( !isdigit( naturalna[ i ] ) )
			liczba = false;
	}

	if( liczba )
		cout << "Podana liczba to: " << naturalna << endl;
	else
		cout << "Nie podano liczby naturalnej!" << endl;

	return 0;
}

Jeżeli będziesz miał potrzebę potraktować podaną liczbę jako liczbę a nie tekst, np. gdy chcesz wykonać na niej jakieś działanie, to music konwertować string na int. Standard C++11 zawiera kilka nowych funkcji w bibliotece <string>, które umożliwiają konwertowanie w różne strony. My chcemy string na int, więc funkcja, która to wykona nazywa się:
stoi( string ). W nawiasie podajemy string, a funkcja zwróci int. Nazwa funkcji mówi sama za siebie: stoi string-to-int.
Polecam poczytać: http://www.cplusplus.com/reference/string/stoi/

;-)

Comments

Ładny warunek z użyciem isdigit, znam tą funkcje. Tylko jeszcze chciałbym to zamknąć w jednej pętli, jakbyś mógł w swoim programie to uwzględnić, czyli niech się pętla wykonuje dopóki nie zostanie wpisana liczba naturalna. :)

---

Jasne. Wysyłam kod.

#include <iostream>
#include <string>
using namespace std;

int main()
{
	bool czy_liczba;
	string liczba;
	do
	{
		czy_liczba = true;
		cout << "Podaj liczbe naturalna: ";
		cin >> liczba;

		for( int i = 0; i < liczba.length(); i++ )
		{
			if( !isdigit( liczba[ i ] ) )
				czy_liczba = false;
		}
		if( !czy_liczba )
			cout << "Nie podano liczby naturalnej!" << endl;

	} while( !czy_liczba );

	int naturalna = stoi( liczba );
	
	cout << "Podana liczba to: " << naturalna << endl;
		
	return 0;
}

Zauważyłem jednak problem w rozwiązaniu wcześniej. Ten kod konwertuje liczbę na int przed wypisaniem i jest to konieczne! W przeciwnym wypadku po podaniu liczby: 0000001, program wypisze 0000001. Natomiast w powyższym przykładzie rozwiązałem ten problem konwertując string na int o czym pisałem wcześniej

---

Ślicznie dziękuje. :) Tylko jeszcze takie coś mi wyrzuca:

cpp|26|error: cannot convert 'std::string {aka std::basic_string<char>}' to 'const char*' for argument '1' to 'int atoi(const char*)'|

W linijce 26 pokazuj taki błąd. Dodałem bibliotekę <stdlib.h> ale nie rozwiązało to problemu.

 

/////

Zamieniłem stoi na atoi i stąd ten błąd. Jeśli jest stoi to pokazuje że not declared in this scope.

---

W takim razie nie piszesz w nowym standardzie C++11. Musisz to przeboleć :-P

" 'int atoi(const char*)'"

Błąd polega na tym, że funkcja "atoi" przyjmuje jako argument tablicę znaków (const char*).
Jeżeli więc w swoim programie wywołujesz gdzieś tę funkcję "atoi" i przekazujesz jej string, to wywołaj ją tak: 
atoi( liczba.c_str() );

Metoda (funkcja) c_str() zwraca wartość w postaci łancucha znaków const char*, a właśnie taki typ przyjmuje funkcja atoi.

Aby używać funkcji atoi należy dołączyć bibliotekę <cstdlib>.

Pozdrawiam :-)

(Staraj się nie używać bibliotek z końcówką ".h", bo w C++ się z nich nie korzysta. Wytłumaczyłem to tutaj:
https://forum.pasja-informatyki.pl/84632/spotykanie-zapisu-c-c )

---

Bardzo dziękuje, super lekcja od Ciebie! :) Zawsze to troszkę więcej się wie niż przedtem. Właśnie czytam ten artykuł.

Pozdrawiam.

Answer 2

if(cin.good())