空引用异常C＃get set

Question

I think that I am misunderstanding the purpose of a get; set; in C#. I have a list of doubles that I am trying to populate and I am using the following code...Both of these are in the same class and when I try to run this, I get a Null reference exception when trying to populate the list. What exactly am I misunderstanding?

public List<double> NewData
{ get; set; }

public InfoFile(String fileName, String groupName)
{

    this.group = groupName;
    test = File.ReadAllLines(fileName);
    FileInfo label = new FileInfo(fileName);
    this.name = Path.GetFileName(fileName);
    isDrawn = false;

    for (int t = 2; t < test.Length; t++)
    {
        NewData.Add(double.Parse(test[t].Substring(6, 4)));
    }
}

Answer 1

You need to initialize/instantiate your list.

public List<double> NewData
{
    get;
    set;
}

public InfoFile(String fileName, String groupName)
{
    // initialize NewData to a new instance of List<double>
    NewData = new List<double>(); 

    this.group = groupName;
    test = File.ReadAllLines(fileName);
    FileInfo label = new FileInfo(fileName);
    this.name = Path.GetFileName(fileName);
    isDrawn = false;
    for (int t = 2; t < test.Length; t++)
    {
        NewData.Add(double.Parse(test[t].Substring(6, 4)));

    }

}

Docs:

In C# 3.0 and later, auto-implemented properties make property-declaration more concise when no additional logic is required in the property accessors. They also enable client code to create objects. When you declare a property as shown in the following example, the compiler creates a private, anonymous backing field that can only be accessed through the property's get and set accessors.

http://msdn.microsoft.com/en-us/library/bb384054.aspx

---

Depending on your scenario, as stated by @EricLippert in his comment to your OP, a setter is rarely used when exposing a List/Collection. It's more common to "Lazy Load" said List/Collection:

public List<double> _newData;
public List<double> NewData
{
    get
    {
        if(_newData == null)
            _newData = new List<double>();

        return _newData;
    }
}

Answer 2

I believe you're just trying to do the following?

List<double> doubleList = new List<double>();
for (int t = 2; t < test.Length; t++)
{
    doubleList.Add(double.Parse(test[t].Substring(6, 4)));    
}

Edit: You may want to look at using a TryParse versus the Parse method depending on how confident you are in your input data.

Answer 3

Automatic properties initializes the property with the default value of that data type, and default value of all reference types are null. So basically the value of your NewData Property is null unless you assign something to it. For collections I would rather only make a get accessor as in your case too there is no need of a setter. So my property would look like:

private List<double> newData = new List<double>();

public List<double> NewData { get{return newDate;} }