Zadanie Domowe lekcja 17

Witam! Nie udaje mi się zabezpieczyć zmiennej przed wprowadzeniem niepoprawnej liczby (Mam na myśli znaku : a,b,c, . , / itd. )
Nie mam pojęcia co robię źle, że program nie chce działać poprawnie. Proszę o pomoc
Z góry dziękuje! :)


Oto mój kod



#include <iostream>
using namespace std;
int wczytajLiczbe()
{
   int liczba;
   bool prawda = cin.good();
   do
   {
       cin >> liczba;
       cout << "Udalo sie? " << prawda << endl;
       if(prawda == 0)
       {
           cout << "Wprowadz poprawna liczbe" << endl;
       }

   }while(prawda != 1);
   return liczba;
}

int main()
{

    std::cout << "Podaj liczbe: ";
    int liczba = wczytajLiczbe();

    std::cout << "Podales liczbe: " << liczba << std::endl;
    return 0;
}

Nie udaje mi się zabezpieczyć zmiennej przed wprowadzeniem niepoprawnej liczby (Mam na myśli znaku : a,b,c, . , / itd. )

1. Ustawiasz zmienną prawda przed jakimkolwiek wczytywaniem.
Zrób to po wczytaniu danych.

2. Do czyszczenia strumienia należy użyć std::cin.clear() oraz std::cin.ignore().

Też sobie ostatnio wróciłem do programowania i w sumie szło nieźle aż do funkcji. Podobny błąd u mnie.


Nie za bardzo rozumiem co masz na myśli, @karambaHZP

C/C++ int wczytajLiczbe() {     int liczba;     bool prawda;     prawda = std::cin.good(); // sprawdzenie strumienia     do     {         std::cin >> liczba; // a dopiero tutaj wczytywanie         prawda; // to nie wiem co robi?         std::cin.clear(); // to powinno wyć w ifie         std::cin.ignore(); // to też, poza tym jest niekompletne i pobierze tylko jeden zalegający znak         if( prawda = !1 ) // tutaj masz przypisanie zamiast porównania, warunek zawsze false         {             std::cout << "Wprowadz poprawna liczbe ";         }     } while( prawda = !1 ); // to samo co w ifie         return liczba; }

Nie za bardzo rozumiem co masz na myśli

Patrząc na kody, to raczej ma na myśli kolejność wykonywania operacji.

edit: powinno być tak.
1. próba wczytania danych
2. sprawdzenie stanu strumienia i zapisanie go w zmiennej .
3. użycie zmiennej  w warunku, lub bezpośrednie użycie metody cin.good() w warunku.
4. jeśli wczytanie się nie powiodło, oczyść strumień i ponów próbę wczytywania.

Aaaa, no tak... spacja przypadkowa naciśnięta przed negacją. Dziwne, że tego nie zauważyłem.

spacja przypadkowa naciśnięta przed negacją.

Coś tu ściemniasz. W tym przypadku wyjdzie nieznajomość operatora .

Tak, bo to jednak nie spacja.
Błąd [ cpp [ /cpp
Wkleje kod na czysto

#include <iostream>
int wczytajLiczbe()
{
int liczba;
do
{
std::cin >> liczba;
std::cin.good();
if(std::cin.good()!=1)
{
std::cout << "Wprowadz poprawna liczbe ";
}
}while (std::cin.good()!=1);
return liczba;
}

int main()
{
    std::cout << "Podaj liczbe: ";
    int liczba = wczytajLiczbe();
    std::cout << "Podales liczbe: " << liczba << std::endl;
    return 0;
}

Nie wiem co zrobić żeby pomimo nie spełnienia warunku while przestawało wykonywać czynność z if i zataczało pętle (powtarzało cały kod od początku)

@BluePotato zerknij: