Nie działający "cin"

Question

Siemka mam problem z programem, w którym gdy program powinien poczekać na wpisanie tekstu leci dalej. Wydaje mi się, że jest to problem z cinem.
 

#include <iostream>
#include <cstdlib>
#include <ctime>
#include <string>
using namespace std;
int main()
{
    int a,b;
    string c;
    cout<<"By zaczac lowic wybierz numerek od 1 do 10:";
    cin>>a;
    if(a==1)
    {
        cout<<"Wylowiles szczupaka gratulacje"<<endl;
    }
    else
    {
    if(a==2){
        cout<<"Wylowiles lososia gratulacje"<<endl;
    }
    else
    {
    if(a==3){
        cout<<"Wylowiles suma gratulacje"<<endl;
    }
    else
    {
    if(a==4){
        cout<<"Wylowiles makrele gratulacje"<<endl;
    }
    else
    {
    if(a==5){
        cout<<"Wylowiles halibuta gratulacje!"<<endl;
    }
    else
    {
    if(a==6){
        cout<<"Wylowiles dorsza gratulacje!"<<endl;
    }
    else
    {
    if(a==7){
        cout<<"Wylowiles pstraga gratulacje!"<<endl;
    }
    else
    {
    if(a==8){
        cout<<"Wylowiles okonia gratulacje!"<<endl;
    }
    else
    {
    if(a==9){
        cout<<"Wylowiles tunczyka gratulacje!"<<endl;
    }
    else
    {
    if(a==10){
        cout<<"Wylowiles starego buta szkoda :("<<endl;
    }
    }}}}}}}}}

    cout<<"Czy chcesz sprawdzic jej wage?:"<<endl;
    cout<<"By dowiedziec sie jaka jest waga wpisz TAK :";
    cin>>b;
    srand(time(NULL));
    int liczba2 =rand()%20+19;
    cout<<"waga="<<liczba2<<"g"<<endl;
    if(liczba2>20 && liczba2<=25)
    {
        cout<<"Wartosc wynosi 6 PLN:"<<endl;
        cout<<"Jezeli chcesz ja sprzedac wpisz TAK lub NIE kiedy chcesz ja zachowac:"<<endl;
        cin>>c;
    }
    else
    {
    if(liczba2>25 && liczba2<=30){
        cout<<"Wartosc wynosi 10 PLN"<<endl;
        cout<<"Jezeli chcesz ja sprzedac wpisz TAK lub NIE kiedy chcesz ja zachowac:"<<endl;
        cin>>c;
    }
    else
    {
    if(liczba2>30 && liczba2<=35){
        cout<<"Wartosc wynosi 13 PLN"<<endl;
        cout<<"Jezeli chcesz ja sprzedac wpisz TAK lub NIE kiedy chcesz ja zachowac :"<<endl;
        cin>>c;
    }
    else
    {
    if(liczba2>35 && liczba2<=40){
        cout<<"Wartosc wynosi 16 PLN"<<endl;
        cout<<"Jezeli chcesz ja sprzedac wpisz TAK lub NIE kiedy chcesz ja zachowac :"<<endl;
        cin>>c;

    }
    }}}
    if(c=="TAK")
    {
        cout<<endl<<"Gratulacje udalo Cie sie sprzedac rybe! :)"<<endl;
    }
    else
    {
    if(c=="NIE")
        cout<<endl<<"Zachowales rybe dla siebie :)"<<endl;
    }


 return 0;

}

 

Comments

    vector<string> fishing = { "szczupaka", "lososia", "suma",
                               "makrele", "halibuta", "dorsza",
                               "pstraga", "okonia", "tunczyka" };

    cout<<"By zaczac lowic wybierz numerek od 1 do 10:";
    cin>>a;

    if (a>=1 && a<10)
       cout<<"Wylowiles "<<fishing[a-1]<<" gratulacje!"<<endl;
    else if (a==10)
       cout<<"Wylowiles starego buta szkoda :("<<endl;

 

Answer 1

int a,b;

    cout<<"By dowiedziec sie jaka jest waga wpisz TAK :";
    cin>>b;

jeśli wpisujesz "TAK" tutaj, to wczytanie do inta się nie uda i cin wejdzie w stan błędu. `b` nie powinno być stringiem?

A na boku, takie potworki:

    if(a==1)
    {
        cout<<"Wylowiles szczupaka gratulacje"<<endl;
    }
    else
    {
    if(a==2){
        cout<<"Wylowiles lososia gratulacje"<<endl;
    }
    ...    
    }}}}}}}}}

Można napisać bez klamr :D

    if(a==1)
        cout<<"Wylowiles szczupaka gratulacje"<<endl;
    else if(a==2)
        cout<<"Wylowiles lososia gratulacje"<<endl;
    else if(a==3)
    ...

 

Answer 2

Każesz podać użytkownikowi napis a próbujesz wczytać liczbę. W związku z tym ustawiana jest flaga błędu i strumień staje się niezdatny do użytkowania.