Loop not functioning

[byu1212]

I have everything set ready to go except one thing and I think I need another set of eyes to solve this one. The loop will not exit even though I have countered it within in the loop. I have made comments next to the potential problems. Please take a look and let me know your thoughts.

void DayOfTheWeek::getDay()
{
string Day[7] = {"sun","mon","tues","wed","thur","fri","sat"};
string Days[7] = {"Sunday","Monday","Tuesday","Wednesday","Thursday","Friday","Saturday"};
string choice;
int dayFound = 0;
int x = 0;
int y = 0;
bool dayCount = false;

cout << "\n What day are you looking for? " << endl << endl;
cout << " sun - Sunday\n";
cout << " mon - Monday\n";
cout << " tues - Tuesday\n";
cout << " wed - Wednesday\n";
cout << " thur - Thursday\n";
cout << " fri - Friday\n";
cout << " sat - Saturday\n";
cin >> choice;

for (x;x< 7;x++)
{
if(Day[x] == choice)
{
dayFound = x;
dayCount = true;
break;
}
}

while(y < 7 && dayCount == false) //here is problem
{
cout << "Invalid entry. Please try again";
cout << "\n What day are you looking for? " << endl << endl;
cout << " sun - Sunday\n";
cout << " mon - Monday\n";
cout << " tues - Tuesday\n";
cout << " wed - Wednesday\n";
cout << " thur - Thursday\n";
cout << " fri - Friday\n";
cout << " sat - Saturday\n";
cin >> choice;

for (x;x< 7;x++)
{
if(choice == Day[x])
{
dayFound = x;
dayCount = true;
break;
}

} //dayCount does not stay true and re-enters loop??? thoughts??
y++;
}

if(x < 7)
{
day = Days[dayFound];
}
}

[firedraco]

Use [code][/code] tags next time...I'm posting this so I can see the code better:

void DayOfTheWeek::getDay()
{
	string Day[7] = {"sun","mon","tues","wed","thur","fri","sat"};
	string Days[7] = {"Sunday","Monday","Tuesday","Wednesday","Thursday","Friday","Saturday"};
	string choice;
	int dayFound = 0;
	int x = 0;
	int y = 0;
	bool dayCount = false;

	cout << "\n What day are you looking for? " << endl << endl;
	cout << " sun - Sunday\n";
	cout << " mon - Monday\n";
	cout << " tues - Tuesday\n";
	cout << " wed - Wednesday\n";
	cout << " thur - Thursday\n";
	cout << " fri - Friday\n";
	cout << " sat - Saturday\n";
	cin >> choice;

	for (x;x< 7;x++)
	{
		if(Day[x] == choice)
		{
			dayFound = x;
			dayCount = true;
			break;
		}
	}

	while(y < 7 && dayCount == false) //here is problem
	{
		cout << "Invalid entry. Please try again";
		cout << "\n What day are you looking for? " << endl << endl;
		cout << " sun - Sunday\n";
		cout << " mon - Monday\n";
		cout << " tues - Tuesday\n";
		cout << " wed - Wednesday\n";
		cout << " thur - Thursday\n";
		cout << " fri - Friday\n";
		cout << " sat - Saturday\n";
		cin >> choice;

		for (x;x< 7;x++)
		{
			if(choice == Day[x])
			{
				dayFound = x;
				dayCount = true;
				break;
			}

		} //dayCount does not stay true and re-enters loop??? thoughts??
		y++;
	}

	if(x < 7)
	{
		day = Days[dayFound];
	}
}

[firedraco]

Hm, it looks like it should work...are you sure that dayCount is being set to true? Try debugging and stepping through it.

[helios]

You don't set x back to zero on your second loop, but let me restructure your code to avoid repetition:

do{
	cout << "\n What day are you looking for? " << endl << endl;
	cout << " sun - Sunday\n";
	cout << " mon - Monday\n";
	cout << " tues - Tuesday\n";
	cout << " wed - Wednesday\n";
	cout << " thur - Thursday\n";
	cout << " fri - Friday\n";
	cout << " sat - Saturday\n";
	//Don't use std::cin >>. Trust me, std::getline() will save you many headaches.
	std::getline(std::cin,choice);

	for (x=0;x<7 && !dayCount;x++){
		if(Day[x] == choice){
			dayFound = x;
			dayCount = true;
		}
	}
}while (!dayCount);

By the way, did you know you can do this?

for (int a=0;a<10;a++)
	std::cout <<a<<std::endl;
for (int a=0;a<10;a++)
	std::cout <<a<<std::endl;

[byu1212]

Wow! A little miscue such as setting x = 0 was the problem. I can't thank you all enough for the assistance. That one hiccup was holding everything back. Also, Helios thanks for the example on the for loops; that is definitely an easier way than me writing everything out. Again, thank you and have a great day!