происходит зацикливание если вводить не число

[Anakin Skyywalker]

Решил задачу "про лифт в двухэтажном доме, смысл ее в том что чел садится на 1 этаже и едет на 2, дальше вводится параметр на каком этаже садится другой человек и если лифт находится на этом же этаже человек садится в него и едет на другой.Если же лифт на другом этаже в данный момент, то сначало перемещается лифт на нужный этаж а потом уже в него садится человек.
с клавиатуры вводится этаж 1 или 2 , другое цифровое значение выводит "Нет такого этажа".Все бы не чего но если вводить букву или символ происходит зацикливание.Помогите понять почему оно происходит. Приму и критику к своему коду.

  #include<iostream>
#include<conio.h>
using namespace std;
int main()
{
	setlocale(LC_ALL, "Russian");
	
	int people,lift=1;
	
	cout << "Введите на каком этаже человек:  ";
	for (int i = 0; i = i + 1;)
	
	{
		cin >> people;
		if (people == 1)
		{  
			if (lift == 1)
			{
				cout << " человек сел в лифт на 1 этаже" << endl;
				lift=2;
				
			}
			else
			{
				cout << "Лифт переместился на 1 этаж" << endl;
				cout << " человек сел в лифт на 1 этаже" << endl;
				lift = 2;
			}
			cout << "Лифт переместился на " << lift << " этаж" << endl;
		}
		
		
		
		else if (people==2)
		{
			if (lift == 2)
			{
				cout << " человек сел в лифт на 2 этаже" << endl;
				lift = 1;

			}
			else 
			{
				cout << "Лифт переместился на 2 этаж" << endl;
				cout << " человек сел в лифт на 2 этаже" << endl;
				lift = 1;
			}
			cout << "Лифт переместился на " << lift << " этаж" << endl;
		
		}

		else if (people !=1 && people !=2)
		{
			cout << "Нет такого этажа!!!" << endl;
		}
		cout << "Введите на каком этаже человек: " ;
	}
	_getch();
	return 0;
}

[Gekan]

Сам новичок, но насколько понимаю, то это стандартное поведение cin. Просто необходима дополнительная обработка, что-то вроде:

do 
{
    cin.clear();
    cin.sync();
    cin>>people;
} while ( cin.fail() );

Этот код вставить вместо

    cin>>people;

Но, по-моему, тут есть много другого, что можно подправить.
Например, вечный цикл можно записать так:

while (true)
{

}

[DakotaZ]

Такое поведение неудивительно, поскольку Вы пытаетесь прочитать в переменную число, а возвращается буква, поток принимает в таком случае ошибочный статус и дальнейшая работа с ним не представляется возможной.

Что бы исправить эту ситуацию, Вам нужно перед командой cin >> people; вписать сброс статуса потока, это делает функция cin.clear().
Однако, сброс состояния не поможет, потому что при следующей итерации чтение cin >> people; опять столкнется с прошлой проблемой, ведь команда cin.clear() лишь сбрасывает статус потока, но не очищает его.
Соответственно, нужно очистить поток, это делает команда cin.sync().

Но, следует сказать, что это решение является лишь "заглушкой" и не приемлемо в маломальски профессиональных кодах.

А заместо использования cin.sync() лучше использовать такую команду, подключив библиотеку <limits>:
cin.ignore(numeric_limits<streamsiz e>::max(), '\n');

Либо такую:
cin.ignore(cin.rdbuf()->in_avail());

Можно переписать Вашу программу вот так, более эстетично и эффективно:

#include <iostream>

int floors = 10, start_point = 1;
int elevator_cfloor, target_floor;
void move_elevator(int to);
int main()
{
	setlocale(LC_ALL,"");
	
	std::cout << "Запуск программы Лифт 2.0 : Нажмите в любой момент сочетание CTRL + C что бы завершить работу программы." << std::endl;
	std::cout << "Этажей доступно : " << floors << std::endl;
	elevator_cfloor = start_point;//set elevator start point
	for(;;)
	{
		std::cout << "Лифт сейчас на : " << elevator_cfloor << " этаже." << std::endl;
		std::cout << "Лифт готов к работе, на каком Вы этаже? : ";
		std::cin.sync();
		std::cin.clear();
		std::cin >> target_floor;
		if(target_floor > floors || target_floor <= 0)
		{
			std::cout << "Извините, но " << target_floor << " этажа у нас нет." << std::endl;
			continue;
		}
		if(target_floor != elevator_cfloor)
		{
			std::cout << "Окей, отправляю к Вам лифт." << std::endl;
			move_elevator(target_floor);
		}
		else
			std::cout << "Вам повезло, лифт находится на Вашем этаже, заходите." << std::endl;
		do{
			std::cout << "Теперь Вы в лифте, куда едем? : ";
			std::cin.sync();
			std::cin.clear();
			std::cin >> target_floor;
			if(target_floor > floors || target_floor <= 0)
				std::cout << "Извините, но " << target_floor << " этажа у нас нет." << std::endl;
			} while(target_floor > floors || target_floor <= 0);
		std::cout << "Лифт направлен на " << target_floor << " этаж, приятного путешествия." << std::endl;
		move_elevator(target_floor);
		std::cout << "Вы достигли " << target_floor << " этажа, лифт теперь свободен, продолжаем работу." << std::endl;
	}
}

void move_elevator(int to)
{
	int it = to-elevator_cfloor;
	if(it < 0)
		it = -it;
	for(int i = 0; i < it; i++)
	{
		if(elevator_cfloor < to)
			elevator_cfloor++;
		else if(elevator_cfloor > to)
			elevator_cfloor--;
		std::cout << "Лифт доехал до " << elevator_cfloor << " этажа." << std::endl;
	}
}