C# initializer null pointer

Question

Currently doing some homework on C# basics. I'm getting a null pointer on this line and can't figure out what's caussing it.:

List<Project> projecten = new List<Project>{
    new Project {
        name = "project a",
        deelnemers =
        {
            new Person { name = "Ed" },
            new Person { name = "Mike" },
        }
    },
    new Project {
        name = "project b",
        deelnemers = {
            new Person {name = "Max" },
            new Person {name = "Peter" },
        }
    }
};

the person and project classes are defined correctly I think:

public class Project
{
    public string name { get; set; }
    public List<Person> deelnemers {get; set;}
}

public class Person
{
    public string name { set; get; }
}

It's probably something stupid but I don't see it.

The complete file is this:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Collections.Generic;

namespace ConsoleApplication2
{
    class Program
    {
        static void Main(string[] args)
        {

            // Maak een collection mbv  initializers van minimaal 3 projecten met meerdere projectleden

            List<Project> projecten = new List<Project>{
                new Project {
                    name = "project a",
                    deelnemers =
                    {
                        new Person { name = "Xanvier" },
                        new Person { name = "Jantje" },
                    }
                },
                new Project {
                    name = "project b",
                    deelnemers = {
                        new Person {name = "Pietje" },
                        new Person {name = "Keesje" },
                    }
                }
            };
            List<Int16> p = new List<Int16> { 1,2,3 };
            //var projectje = new Project{ name = "project a" };

        }
    }

    public class Project
    {
        public string name { get; set; }
        public List<Person> deelnemers {get; set;}
    }

    public class Person
    {
        public string name { get; set; }
    }
}

I don't think the error is where you think it is - can you share a full, runnable (but minimal) example that demonstrates the problem? — Ant P
– Ant P, Commented Sep 19, 2016 at 11:00
@CodeCaster, this is the line where it's caussing (the first code block is one line) Can you clarify why it's not compiling? — Ravenix
– Ravenix, Commented Sep 19, 2016 at 11:03
The duplicate question shouldn't have been marked as such. The OP knows what a NullReferenceException is. deelnemers = { new Person ... } will compile. The problem is, it does not initialize your list. It will only call Add. You need to say deelnamers = new List<Person> { ... } — Dennis_E
– Dennis_E, Commented Sep 19, 2016 at 11:04
@CodeCaster I literally just compiled it. It throws an exception on runtime. It compiles fine. — ThePerplexedOne
– ThePerplexedOne, Commented Sep 19, 2016 at 11:07
I answered a similar question once: stackoverflow.com/questions/38369967 Maybe THAT should have been be the one marked as duplicate? — Dennis_E
– Dennis_E, Commented Sep 19, 2016 at 11:11

Dennis_E · Accepted Answer · 2016-09-19 11:16:09Z

1

This part is a collection initializer:

deelnemers =
    {
        new Person { name = "Ed" },
        new Person { name = "Mike" },
    }

This is perhaps one of the most confusing forms of syntax in C#. It does not initialize your list. It only calls Add on it. You need to initialize the List as well:

deelnemers = new List<Person> {
    new Person { name = "Xanvier"},
    //
}

Or, you can initialize the List in the constructor of Project. That way, your original code will work as expected.

answered Sep 19, 2016 at 11:16

Dennis_E

8,90426 silver badges32 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Ravenix · Accepted Answer · 2016-09-19 11:15:45Z

0

The solution is, like Dennis_E said in a comment: deelnemers = { new Person ... } will compile. The problem is, it does not initialize your list. It will only call Add. You need to say:

deelnemers = new List<Person> { ...

So the problem was that I forgot to initialize the list.

answered Sep 19, 2016 at 11:15

Ravenix

1,0304 gold badges16 silver badges42 bronze badges

Comments

Mohamed Jihed Eddine Gam · Accepted Answer · 2016-09-20 11:06:22Z

-1

try this List projecten = new List{ new Project { name = "projecta", deelnemers = new List{

                {new Person { name = "Ed" }},
                {new Person { name = "Mike" }}
                }
            },
        new Project {
            name = "projectb",
            deelnemers = new List<Person>{
                {new Person {name = "Max" }},
                {new Person {name = "Peter" }}
            }
        }
        };

edited Sep 20, 2016 at 11:06

answered Sep 19, 2016 at 11:08

Mohamed Jihed Eddine Gam

631 silver badge12 bronze badges

1 Comment

CodeCaster Over a year ago

"Try this" is not a good answer. Explain what you changed and why.

Collectives™ on Stack Overflow

C# initializer null pointer

3 Answers 3

Comments

Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related