zabezpieczenie przed blednymi danymi

Question

Cześć, chce zabezpieczyć swój program przed wprowadzeniem błędnych danych. Przed wprowadzeniem liter zabezpieczyłem to tak 

if (!(cin>>metal>>krysztal>>deuter))
    {
        cerr<<"Nie poprawne dane! Program zostanie zakonczony.";
        exit(0);
    }

Teraz chce zabezpieczyć to jeszcze przed liczbami ujemnymi. Deklaruje unsigned int i po wprowadzeniu ujemnej liczby program się wysypuje. Jak zrobić, żeby pokazał się ten sam komunikat co przy literach? Wiem, że jest biblioteka cctype, ale nie wiem jak mogłaby mi pomóc. Mógłbym to zrobić na if'ach ale to się mija z celem :D 

Może ktoś mi podpowie w jaki sposób to zrobić :D 

Answer 1

Choćby tak:

If (!(...)||(liczba<0))

 

Comments

Dzięki, jest to jakieś rozwiązanie, ale chyba nie najbardziej optymalne ;/

W każdym razie dla tego programu na pewno wystarczające, może ktoś wpadnie na inny pomysł :D

Answer 2

Można i tak... Jest oczywiście jeszcze kilka innych sposobów:

#include <iostream>
#include <limits>
#include <type_traits>

// Pasuje do: bool, char, char16_t, char32_t, wchar_t, short, int, long, long long...
template<typename T>
typename std::enable_if<std::is_integral<T>::value, T>::type
readValue(const std::string& msg) {
    T value;
    for(;;) {
        std::cout << msg;
        std::cin >> value;
        if(!std::cin) {
            std::cerr << "Dane nieprawidłowe.\n"
                << "Spróbuj jeszcze raz.\n";
            std::cin.clear();
            std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
        } else {
            break;
        }
    }
    return value;
}

// Pasuje do: float, double, ... 
template<typename T>
typename std::enable_if<std::is_floating_point<T>::value, T>::type
readValue(const std::string& msg) {
    T value;
    for(;;) {
        std::cout << msg;
        std::cin >> value;
        if(!std::cin) {
            std::cerr << "Dane nieprawidłowe.\n"
                << "Spróbuj jeszcze raz.\n";
            std::cin.clear();
            std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
        } else {
            break;
        }
    }
    return value;
}

int main() {
    auto value1 = readValue<unsigned>("Wprowadź dane bez znaku: ");
    std::cout << "Wprowadzono i przyjęto: " << value1 << '\n';
    auto value2 = readValue<int>("Wprowadź dane ze znakiem: ");
    std::cout << "Wprowadzono i przyjęto: " << value2 << '\n';
    auto value3 = readValue<float>("Wprowadź zmiennoprzecinkowe: ");
    std::cout << "Wprowadzono i przyjęto: " << value3 << '\n';
}

Testów std::is_* jest w <type_traits> wiele... 

Comments

in:
-20
out:
Wprowadź dane bez znaku: Wprowadzono i przyjęto: 4294967276

mokrowski kompilowałeś za pomocą clang? 

---

clang 6.0

Jak ktoś będzie chciał, dopasuje i do gcc tylko trzeba użyć nieco innych predykatów z <type_traits>

---

Dziękuje :)

Answer 3

Możesz sprawić, czy w czasie wczytywania został podany minus, jeżeli tak zostanie ustawiona flaga błędu:

#include <iostream>

std::istream& minus_not_good(std::istream& in) 
{
	if ((in >> std::ws).peek() == std::char_traits<char>::to_int_type('-')) {
		in.setstate(std::ios_base::failbit);
	}
	return in;
}

void readUnsigned(unsigned int& value) 
{
	std::cin >> minus_not_good >> value;
}

int main()
{
	unsigned int metal{}, krysztal{}, deuter{};
	readUnsigned(metal);
	readUnsigned(krysztal);
	readUnsigned(deuter);
	std::cout << metal << " " << krysztal << " " << deuter << "\n";

	if (std::cin.fail())
	{
		std::cerr << "Niepoprawne dane! Program zostanie zakonczony. \n";
		std::exit(0);
	}
}

Ewentualnie jakoś przeciążyć operator>> dla unsigned int, ale to nie wiem jak zrobić. 

Comments

Dziękuje :)