Thread: Problems with getline

I could be wrong, but I think you need a cin.ignore() after you get the Y/N for a pool.

It probably didn't solve the problem because you didn't do it, or at least didn't do it properly.

Case in point:

Code:

#include <iostream>

int main()
{
	std::string s1, s2;
	int num;
	char yn;
	
	for(int i=0;i<3;i++)
	{
		std::cout << "Enter string 1: ";
		getline(std::cin, s1, '\n');
		std::cout << "Enter string 2: ";
		getline(std::cin, s2, '\n');
		std::cout << "Enter int 1: ";
		std::cin >> num;
		do
		{
			std::cout << "Enter [Y/N]: ";
			std::cin >> yn;
			std::cin.ignore();  /* Important */
		}
		while((yn != 'Y') && (yn != 'N'));
	}
	
	return 0;
}

First iteration of output without the std::cin.ignore():

Enter string 1: s1
Enter string 2: s2
Enter int 1: 1
Enter [Y/N]: Y
Enter string 1: Enter string 2: S2

First iteration of output with the std::cin.ignore():

Enter string 1: s1
Enter string 2: s2
Enter int 1: 1
Enter [Y/N]: Y
Enter string 1: S1

Because when you use std::cin to read a char, it reads a char and leaves the '\n' you entered in the stream. Then when getline() is called, it reads until it hits the '\n', which it does right away.

Example:

Code:

cout<<"Please indicate of the garden has a pool"<<endl;
cin>>gardensP[i].Pool;

When this happens, and you type "Y" and press enter, this is the input stream:

Code:

'Y', '\n'

std::cin will read the 'Y' char. So the input stream now looks like this:

Code:

'\n'

You then come back to this in the loop:

Code:

cout<<"Please enter the address of garden "<<i+1<<":"<<endl;
getline(cin,gardensP[i].Address,'\n');

And getline() reads the '\n' in the stream and decides it reached the end of what you wanted.

Placing std::cin.ignore() after reading the 'Y' char will read and discard the '\n' char.