for loop not letting me input variable data

I have a for loop set up to name the elements of an array. I think I am using it wrong. Here is my code.

Code:

---

/* Declared variables outside of for loop */
string Heroes[9000];
int h_cost[9000];
string h_name;
int h_min;
int num_heroes;


/*For loop to set up names and cost of heroes*/
for (int h = 1; h <= num_heroes; ++h){
                          cout <<"What unit for Hero" <<h <<"?" <<endl;
                          string h_name = Heroes[h];
                          cin>> h_name;
                          cout <<"How many civs to buy this hero?" <<endl;
                          cin>> h_cost[h];
                          if (Heroes[h] == "Minerals"){
                                        cout <<"How many minerals awarded?" <<endl;
                                        cin>> h_min;
                                        }
                          }

---

In essence I want the program to loop for as many heroes I have. The ouput should be similar to this....

Code:

---

What unit for Hero1?
Heroes name // <---value input by user
How many civs to buy this hero?
Hero cost    // <---value input by user
What unit for Hero2?
Heroes name // <---value input by user
How many civs to buy this hero?
Hero cost // <---value input by user

---

Instead I get this....

Code:

---

What unit for Hero1?
Heroes name // <---value input by user
What unit for Hero2?
How many civs to buy this hero?
What unit for Hero3?
How many civs to buy this hero?
What unit for Hero3?
How many civs to buy this hero?
What unit for Hero4?

---

The problem is that its only letting me input the name for hero1 and just displays the rest of the questions without letting me input what I want them to be. Why isn't it letting me input values for my other heroes and costs?

Do not know what to tell you, I compiled your code and it let me input the values for every "Hero".

Does the name of the first hero have a space in it? When using operator>> you can only read in a single word.

Oh, ok thats my problem. How would I allow input to be able to use spaces without messing up the program?

std::getline

You would have to use getline(cin, h_name); to get the name.

Using getline leads to another issue since you will want to use operator>> to read in the h_cost int value. The operator>> leaves a newline in the stream from when the user hits enter after typing the answer. The getline function reads that newline and stops and so the second time through your loop the name will show up empty.

A simple solution is to call cin.ignore() after every time you call cin >> to read in something. This will ignore the newline. Do not call ignore() after calls to getline, since getline() discards the newline on its own.

Quote:

---

Originally Posted by Daved

You would have to use getline(cin, h_name); to get the name.

Using getline leads to another issue since you will want to use operator>> to read in the h_cost int value. The operator>> leaves a newline in the stream from when the user hits enter after typing the answer. The getline function reads that newline and stops and so the second time through your loop the name will show up empty.

A simple solution is to call cin.ignore() after every time you call cin >> to read in something. This will ignore the newline. Do not call ignore() after calls to getline, since getline() discards the newline on its own.

---

Thanks, but doing it that way created another problem. Using this code:

Code:

---

for (int h = 1; h <= num_heroes; ++h){
                          cout <<"What unit for Hero" <<h <<"?" <<endl;
                          string h_name = Heroes[h];
                          getline(cin, h_name);
                         
                          cout <<"How many civs to buy this hero?" <<endl;
                          //int num_civs = h_cost[h];
                          cin >> h_cost[h];
                          cin.ignore();
                          if (Heroes[h] == "Minerals"){
                                        cout <<"How many minerals awarded?" <<endl;
                                        cin>> h_min;
                                        }
                          }

---

This does not allow me to input value for hero1.

EDIT: Nevermind, I got it. I had to enter in another cin.ignore(); after the >>operator I used before I initialised the loop. Thanks Daved!!