Member 14082092 Ответов: 3

Бесконечный цикл в программе C++


я пытаюсь сделать программу которая сравнивает между двумя массивами которые они имеют рабочие дни двух сотрудников и пользователь вводит месяц и выходные дни и я сделал код ниже но это дает мне бесконечный цикл

Что я уже пробовал:

#include<iostream>
using namespace std;
int main(){
	int months,size,daysOff1,daysOff2,exit,work1,work2;
	do{
		cout<<"Please enter the month's number: "<<endl;
		cin>>months;
		
		switch(months){
			case 1:
			case 3:
			case 5:
			case 7:
			case 8:
			case 10:
			case 12: 
				size = 31;
				break;
			case 4:
			case 6:
			case 9:
			case 11: 
				size = 30;
				break;
			case 2: 
				size = 28;
				break;
			default : 
				cout<<"please enter a correct value"<<endl;
			continue;  // to Make it ask again if the user entered a wrong value
		}
		
		cout<<"Please enter the number of off days for fahad: "<<endl;
		cin>>daysOff1;
		daysOff1++;
		cout<<"Please enter the number of off days for talal: "<<endl;
		cin>>daysOff2;
		daysOff2++;
		
		int fahad[size],talal[size];
	
		for (int i=0; i < size; i++)
		{
			while( work1 < size )
			{
				if(fahad[i-1]+daysOff1 < size){
					fahad[i] = fahad[i-1] + daysOff1 + 1;
					work1 = fahad[i];
					i++;
					
				}
			}
		}
		
		for(int i=0; i < size; i++)
		{
			while (work2 < size)
			{
				if(talal[i-1]+daysOff2 < size){
				talal[i] = talal[i-1] + daysOff2 + 1;
				work2 = talal[i];
				i++;
				
				}
			}
		}
	
		for(int i=0; i<size ;i++)
		{
			for(int j=0; j < size ;j++){	
		if(fahad[i] == talal[j]){
			cout<<i+1<<endl;
		}}}
		
		cout<<"0 To exit 1 to continue";
		cin>>exit;
	}while(exit != 0);
}

Richard MacCutchan

А вы тоже постили https://www.codeproject.com/Questions/1270843/Write-a-Cplusplus-program[^] с другим счетом? Если это так, Пожалуйста, удалите дубликат учетной записи и не перепечатывайте один и тот же вопрос.

3 Ответов

Рейтинг:
2

Richard MacCutchan

for (int i=0; i < size; i++)
{
    while( work1 < size )
    {
        if(fahad[i-1]+daysOff1 < size){
            fahad[i] = fahad[i-1] + daysOff1 + 1;
            work1 = fahad[i];
            i++;
            
        }
    }
}

Первый раз вокруг этой переменной цикла i будет иметь значение 0. Но вы пытаетесь обратиться к элементу массива с помощью [i - 1] что является недопустимым и приведет к случайному значению. Это делает вполне вероятным, что вы установите fahad[i], а оттуда work1, на число большее, чем size что приведет к бесконечному циклу.

Вам нужно проверить в своих циклах, что вы не пытаетесь обращаться к элементам до начала или после конца ваших массивов.


Рейтинг:
0

OriginalGriff

Я понятия не имею, что этот код пытается сделать: он вообще не имеет большого смысла, особенно без вашего фактического домашнего вопроса - мы даже не знаем, что вы вводите, чтобы дать вам эти результаты!

Так что все будет зависеть от тебя.
К счастью, у вас есть инструмент, который поможет вам выяснить, что происходит: отладчик. Как вы его используете, зависит от вашей компиляторной системы, но быстрый поиск в Google имени вашей IDE и "отладчика" должен дать вам необходимую информацию.

Поместите точку останова в первую строку функции и запустите код через отладчик. Затем посмотрите на свой код и на свои данные и определите, что должно произойти вручную. Затем по одному шагу в каждой строке проверяйте, что то, что вы ожидали, произойдет именно так, как и произошло. Когда это не так, тогда у вас есть проблема, и вы можете вернуться назад (или запустить ее снова и посмотреть более внимательно), чтобы выяснить, почему.

Извините, но мы не можем сделать это за вас - пришло время вам освоить новый (и очень, очень полезный) навык: отладку!


Рейтинг:
0

Patrice T

Цитата:
я сделал код ниже, но он дал мне бесконечный цикл

Посмотрите, что ваш код действительно делает с отладчиком, вы увидите, где код зацикливается, и вы увидите значения переменных по мере его выполнения. Что-то не так, как вы думаете, что это должно быть.

Ваш код ведет себя не так, как вы ожидаете, или вы не понимаете, почему !

Существует почти универсальное решение: запускайте свой код на отладчике шаг за шагом, проверяйте переменные.
Отладчик здесь, чтобы показать вам, что делает ваш код, и ваша задача-сравнить с тем, что он должен делать.
В отладчике нет никакой магии, он не знает, что должен делать ваш код, он не находит ошибок, он просто помогает вам, показывая, что происходит. Когда код не делает того, что ожидается, вы близки к ошибке.
Чтобы увидеть, что делает ваш код: просто установите точку останова и посмотрите, как работает ваш код, отладчик позволит вам выполнять строки 1 на 1 и проверять переменные по мере их выполнения.

Отладчик - Википедия, свободная энциклопедия[^]

Освоение отладки в Visual Studio 2010 - руководство для начинающих[^]
Базовая отладка с помощью Visual Studio 2010 - YouTube[^]

1.11 — отладка программы (пошаговое выполнение и останова) | выучить C++[^]

Отладчик здесь только для того, чтобы показать вам, что делает ваш код, и ваша задача-сравнить его с тем, что он должен делать.