(C++) Does not stop on std::cin and goes into infinity loop? - c++

Question

I have a wird problem. Im using Visual Studio 2012, thats my code: (something is in polish but i hope you will understand how it works).
#include <stdio.h>
#include <stdlib.h>
#include <iostream>
#include <queue>
using namespace std;
#pragma warning (disable: 4996);
struct strona
{
int zawartosc;
int gdzie;
};
void Wyswietl(int tab[], int dlugosc)
{
cout<<"Tablica wyglada tak"<<endl;
for(int i=0;i<dlugosc;i++)
{
cout<<tab[i]<<"\t";
}
cout<<endl;
}
int main()
{
int ileStron=3;
int *tablicaStron, *tablicaBitowOdniesienia;
tablicaStron=new int[ileStron];
tablicaBitowOdniesienia=new int[ileStron];
queue <strona> kolejka;
char opcja='a';
while(opcja!='k')
{
cout<<"(D)odawac (K)oniec (W)yswietl";
cin>>opcja;//DONT STOP THE PROGRAM!
if(opcja=='D'|opcja=='d')
{
strona tymczas;
cout<<"Podaj co dodać do kolejki";
cin>>tymczas.zawartosc;
int licznik=0;
if(kolejka.size()<ileStron)
{
tymczas.gdzie=kolejka.size();
kolejka.push(tymczas);
tablicaStron[tymczas.gdzie]=tymczas.zawartosc;
}
else if(kolejka.size()==ileStron)
{
cout<<"bang bang";
int czyJest=0;
int licznikfora=0;
for(int i=0;i<ileStron;i++)//sprawdza czy wpisywana strona nie istnieje przypadkiem w tablicy stron
{
if(tablicaStron[i]==tymczas.zawartosc)
{
czyJest=1;
}
licznikfora++;
}
cout<<"czyJest ma wartosc "<<czyJest<<" a licznik fora "<<licznikfora<<endl;
if(czyJest==0)
{
tymczas.gdzie=kolejka.front().gdzie;
kolejka.pop();//TUTAJ SIE BEDZIE ZAPISYWAC DO PAMIECI WIRTUALNEJ
kolejka.push(tymczas);
tablicaStron[tymczas.gdzie]=tymczas.zawartosc;
}
else if(czyJest==1)
{
cout<<"to co chcesz dodac juz jest w pamieci";
}
}
else
{
cout<<"rozmiar kolejki sie nie zgadza";
}
}
else if(opcja=='W'|opcja=='w')
{
Wyswietl(tablicaStron,ileStron);
cout<<endl;
cout<<"pierwszy element w kol: "<<kolejka.front().zawartosc<<"|"<<kolejka.front().gdzie<<" "
<<"ostatni element w kol: "<<kolejka.back().zawartosc<<"|"<<kolejka.back().gdzie<<endl;
}
}
system("pause");
return 0;
}
The problem is that after choosing option (d) - just type d and press enter, then type any few letters and the program should show you
(D)odawac (K)oniec (W)yswietl
but it is starting to loop to infinity...
What is the problem?

Answer 1

I think problem is with this statement:
cin>>tymczas.zawartosc;
At this point if you give a char or string input, program blows up.
to make your program work properly:
start the program and give only one input d, let the control hit this point : cin>>tymczas.zawartosc;
now input a number here, as it is int type. now control hits first cin.
The problem is with using cin to read int but user inputs a char or string. Behaviour is undefined in this case.
This link should resolve confusion: C++ character to int