Add element to null (empty) List Property

Question

I got a problem.

The problem is that I try to ad an object to a list of this objects. This list is a property, no error, but when I run it fails at this point, becouse: "NullReferenceException". Sounds logical, becouse the Property of the list is "null", but I cant declare a property, can I?

Her is some Code snipped:

class Maps
{
    protected virtual List<Ant> AllAntsAtMap { get; set; }

[...]

class Quadrangle : Maps
{
    protected override List<Ant> AllAntsAtMap { get; set; }

    public override void AddAntToMap(Ant ant)
    {
        AllAntsAtMap.Add(ant);  //Error here
    }
    public override void AddAntsToMap(List<Ant> ants)
    {
        foreach (Ant ant in ants)
        {
            AddAntToMap(ant);
        }
    }

[...]

Any reason why the property is virtual and then overridden in a derived class? You really only need to declare it once. Overriding adds no benefit here. — siride, Sep 10 '15 at 14:39

Heinzi · Accepted Answer · 2020-11-06T07:48:23.030

Add element to null (empty) List Property

null and an empty list are two different things: Adding an element to an empty list works fine, but if your property is null (as all reference-type properties are initially null), you need to initialize it with an empty list first.

You could use an auto-property initializer for that (see Kędrzu's answer), or you could manually initialize the list in the constructor:

class Maps
{
    public Maps()
    {
        AllAntsAtMap = new List<Ant>();
    }

    ...
}

(Since the property is declared in the superclass Maps, I'd do the initialization there rather than in the subclass Quadrangle.)

score 15 · Answer 2 · answered Sep 10 '15 at 14:39

15

It is much simpler in C# 6:

protected List<Ant> AllAntsAtMap { get; set; } = new List<Ant>();

answered Sep 10 '15 at 14:39

Kędrzu

2,385
13
22

1

Cool, but they probably aren't using C# 6. – siride Sep 10 '15 at 14:39
4

How this assumption is this related to question? – Kędrzu Sep 10 '15 at 14:41
5 years later: Both methods work in .NET Core 3.1 (This and init in the ctor) – Joe Johnston Nov 05 '20 at 21:57

score 3 · Answer 3 · edited Jan 04 '18 at 09:21

3

You should initialize AllAntsAtMap before usage. You can use the constructor for that:

public Quadrangle()
{
    AllAntsAtMap = new List<Ant>();
}

edited Jan 04 '18 at 09:21

John C

3,052
3
34
47

answered Sep 10 '15 at 14:37

burning_LEGION

13,246
8
40
52

score 2 · Answer 4 · answered Sep 10 '15 at 14:39

2

You need to instantiate the list before adding:

this.AllAntsAtMap = new List<AllAntsAtMap>();

answered Sep 10 '15 at 14:39

David

209
4
17

Guffa · Answer 5 · 2015-09-10T14:49:19.150

Initialise the list in the constructor.

class Maps
{
    protected virtual List<Ant> AllAntsAtMap { get; set; }

    public Maps()
    {
        AllAntsAtMap = new List<Ant>();
    }

}

You don't need to override the list property in Quadrangle unless you specifically need a separate list from the base class. If you do have a separate list in the child class then you need to initialise that list in the child class constructor.

If the base class shouldn't have an implementation of the list property, then you should make it abstract instead of virtual, and initialise it in the child class where the property is implemented.

Add element to null (empty) List Property

5 Answers5