C++ Array Help

I am writing a program that prompts the user for the name and age of a group of people. The group's size is unknown initially, so the program must keep asking the user if they want to enter more data. It has been a while since I have worked with C++ so I am trying to figure out how to take the user inputs of name and ages into an array until the user wants to stop imputing information.

I know this is incorrect but maybe you guys could help steer me in the right direction.

Code:

char name[15];
int age[3];
char answer;
int i;

while(answer=='y'||'Y')
{
cout << "Enter name: ";
cin >> name[i];
cout << "Age: ";
cin >> age[i];
         
cout << "Would you like to add another person? (Y/N)";
cin >> answer;

}

Data Structures
vector - C++ Reference
Combine these two.

You have to initialize "answer" to 'y' or 'Y' before entering the loop.

Every time I try to run this it asks me for the name and once I hit enter It keeps running. It does not let me enter a age or ask if I want to run this program again.

Code:

int main()
{


char name[15];
int age[3];
char answer = 'y'||'Y';
int i = 0;

while(answer=='y'||'Y')
{

cout << "Enter name: ";
cin >> name[i];
cout <<endl;

cout << "Enter Age: ";
cin >> age[i];
cout <<endl;

cout << "Would you like to add another person? (Y/N)";
cin >> answer;
cout <<endl;

}

I found a piece of code using vectors that works but I do not know how to implement it so It will prompt a user to enter a name and age until the user is done entering data.

Code:

const int NUM_PERSONS = 5;

    vector<string> names(NUM_PERSONS);
    vector<int> ages(NUM_PERSONS);

    for (int i = 0; i < names.size(); i++)
    {
        cout << "Enter name: ";
        getline (cin, names[i]);

        cout << "Age: ";
        cin >> ages[i];
        cin.get();  
	}

Code:

char answer = 'y'||'Y';

HUH??? While this compiles, it's really the same as saying

Code:

char answer = 'y';

So change that.

Code:

while(answer=='y'||'Y')

You can't do that. Each comparison must be separate.

Code:

while(answer == 'y'|| answer == 'Y')

Shouldn't you be making sure you don't receive more entries that you have space for?

You also are leaving a newline in the input buffer. Add a

Code:

cin.ignore()

after you "add another" question gets answered.

You do this:

Code:

std::wstring Name;
std::vector<std::wstring> Names;
std::getline(std::wcin, Name);
Names.push_back(Name);
// Repeat lines 3 to 4

Originally Posted by rags_to_riches

Code:

char answer = 'y'||'Y';

HUH??? While this compiles, it's really the same as saying

Code:

char answer = 'y';

Actually it is the same as:

Code:

char answer = true;

Originally Posted by Elysia

You do this:

Code:

std::wstring Name;
std::vector<std::wstring> Names;
std::getline(std::wcin, Name);
Names.push_back(Name);
// Repeat lines 1 to 2

Could you explain to me what is happening here? I am confused

What are you confused about? Rather than forcing us to write complete tutorials that you probably won't read but once, will you explain what you do understand? It's nice to know if you are even trying.

Originally Posted by kmdv

Actually it is the same as:

Code:

char answer = true;

Oooo...good point!

OK here is what I have so far and as of right now it works. Except I have one question.

Code:

string names[15];
int ages[3];
char answer = 'y';


while(answer == 'y'|| answer == 'Y')
{
	 for (int i = 0; i < 3; i++)
    {
	cout << "Enter name: ";
	cin >> names[i];
    cout <<endl;


	cout << "Enter Age: ";
    cin >> ages[i];
    cout <<endl;


	cout << "Would you like to add another person? (Y/N)";
	cin >> answer;
	cout <<endl;

	
	 }
}
	 for(int x = 0; x < 3; x++)
  {
    cout << "Name: " << names[x] << "   Age: ";
    cout << ages[x] << endl;
  }

Code:

 for (int i = 0; i < 3; i++)

I just set this to 3 so I could run the program but I want to know how I would do this if I didn't have a set number of Names and Ages? So when I run this program 1 time and input 10 names and ages, and the next time i only want to input 3 or 4? I tried names.size but it did not work.

OK, so first I showed you an example of how to do it properly. If you don't understand it, ask a specific question about what you don't understand.
As for the second question, ask yourself a logical question. If you were to take a list of names, how would you know when to stop? How would a computer know when to stop?
Formulate the logic, make a flowchart, translate to code.

Originally Posted by Elysia

You do this:

Code:

std::wstring Name;
std::vector<std::wstring> Names;
std::getline(std::wcin, Name);
Names.push_back(Name);
// Repeat lines 3 to 4

@Whiteflags - This is what I was confused about. I have never worked with anything like this before so I do not know what it means. I am not trying to have you do this for me. I am simply asking for help to guide me in the right direction which everyone has been doing so far.

The only reason you would need a counting loop like

Code:

for (int i = 0; i < 3; i++)

is because you are using arrays which have a fixed size.

You later posted vector code, which is good because it gets more memory on its own if it needs it. But, incorrectly in this case, you used it so that you accessed elements like this:

Code:

        cout << "Enter name: ";
        getline (cin, names[i]);

        cout << "Age: ";
        cin >> ages[i];
        cin.get();

Not much better than arrays. Once you go over the NUM_PERSONS amount, you'll have the same errors arrays would get you.

The vector class has a push_back( ) method. Read the actual input into appropriate variables first, then

Code:

names.push_back( some_name );
ages.push_back( some_age );

That's all you have to do to get the array to extend itself. Now you just keep looping until the user says no, and you have a couple vectors filled with everything the end user typed.