getting error "Object reference not set to an instance of an object" when i am creating a new instance of it

Hi, sorry for nooby question

I have a static class for the players and keeping the players list (Players) and a class that holds each individual player (PlayerClass)

static public class Players {

	static public PlayerClass[] playerList;

	static public void AddPlayer(string name, Color color) {
		playerList [playerList.Length] = new PlayerClass(playerList.Length ,name, color);
		Debug.Log ("Player " + playerList[playerList.Length].Name + " has been added");
	}


}

public class PlayerClass {

	private int playerid;
	private Color playerColor;
	private string playerName;




	public PlayerClass ( int id, string name, Color color) {
		playerid = id;
		playerName = name;
		playerColor = color;
	}

	public string Name {
		get {
			return playerName;
		}
	}

	public Color PlayerColor {
		get {
			return playerColor;
		}
	}
}

I am calling the function AddPlayer with the parameters (“Neutral”,Color.white)

and then i am getting the error

NullReferenceException: Object reference not set to an instance of an object
Players.AddPlayer (System.String name, Color color) (at Assets/Scripts/Players.cs:9)
MapScript.Start () (at Assets/Scripts/MapScript.cs:16)

Btw… MapScript is where i am calling the function

As far as i know that if it isnt a monobehaviour you can just create a new instance with the new keyword.

Thanks in advance.

Btw also if you think that i am doing everything wrong in the way to manage players please make me know… i am just starting and this is the best way i could come up with

It is the playerlist that is probably null. You have to initialize it in some way before using it.

PlayerClass[] playerList = new PlayerClass;

It is a static class so you could do this:

static public PlayerClass[] playerList = new PlayerClass[someDefinedSize];

Keep in mind that arrays can not change size so you will have to know how many players you are going to add beforehand. When you do add a player you have to search and find an empty spot if you are not storing the most reason used index of the array so you know where you are.

An array of Length 5 has cells from 0 - 4, which means that this:

playerList [playerList.Length]

is trying to access cell number 5 (in my example) which does not exist, you should do this if you want to add to the last position of an array:

playerList [playerList.Length - 1]

Final Note: I would recommend using the Generic List class instead, I think it would suit your need better. It can change its size dynamically so you do not need to know beforehand how many players you are going to create.