I used getline to an array which it's size is determine by using pointer.. can't~ y?

I am a newbie here. Trying to do my assignment.. Here is my code.

Code:

---

# include <iostream>
# include <ctime>
# include <string>

using namespace std;

int main ()
{
   
    int num_player;
    string name[100];
       
    cout << endl << "\tWelcome to game centre. I am Joe, your game instructor.\n"
        << "\t\tSnake and Ladder" << endl << endl
        << " Please enter the number of player [maximum 4 players]:" ;
    cin >> num_player;
    cout<< endl;



    cout << " Enter the " << num_player << " player's name: " << endl;
    for ( int j= 1; j <= num_player ; j++ )
    {
        cout << j << ". " ;
        getline(cin,name[j]);
    }
   

    cout << endl;
    system("PAUSE");
    return 0;
}

---

The output I get have a problem.
System don't allow me to input for name[1];
I don't understand why~
Is it the getline problem?
can someone fix it for me? T^T

You need to remove the newline left behind by inputting the number of players.

Code:

---

    cin >> num_player;
    cin.ignore();  //!! add this

---

oh i see~~
thx~~

for ( int j= 1; j <= num_player ; j++ )
Loop is wrong. It must go from 0 to num_player - 1 becuase C++ arrays are 0-indexed.
Because you're using a pre-allocated array, you must be extra careful not to let num_player exceed 100.
Or you could use a vector to dynamically allocate the space for the array:

std::vector name(num_player);

If you insist on using a fixed-array, then, get rid of C-arrays and use a C++ array:
std::array<string, 100> name;
Don't forget to include <array>.
And don't forget to check that your number of players is in range!

Don't forget that using array templates requires C++11 compliance, which varies by compiler.

You mean std::array? That has existed since TR1 (in the std::tr1 namespace). So yes, you will need a compiler that supports C++11 or TR1 to use it. Most compilers today ought to support it.

I feel like you take it for granted though. My compiler is pretty new and still the only way I get C++11 is by using switches. Other people are possibly using something even older than I am.

mingw32-gcc.exe (TDM-2 mingw32) 4.4.1

I'm saying that you should mention C++11 that's all, because it's not the default.

Quote:

---

Originally Posted by whiteflags

I'm saying that you should mention C++11 that's all, because it's not the default.

---

Yes, you are right. I do tend to forget and take it for granted :(