CODEWITHSUNDEEP
c#
Mighty Ferengi
Mighty Ferengi

Reputation: 836

C# - Class.models.modelname.list.get returned null when using List.Add

I built a class:

public partial class SvcCodes
{
    public List<SvcCodeReport> Report { get; set; }
    public List<SvcCodesCheck> TestList { get; set; }
}

It uses this class:

public partial class SvcCodeReport
{
    public List<Dictionary<string, object>> ProgResults { get; set; }
    public IEnumerable<ExtraData> ExtraData { get; set; }
}

In the controller, data is pulled from two sources and assigned to the SvcCodeReport class:

var model = new SvcCodeReport()
{
  ProgResults = tasks,
  ExtraData = _context.ExtraData.Where(h => h.ServiceCode == Int32.Parse(id)
};

Then the model variable is added to the Report class:

check.Report.Add(model);

But when I run the program, I get a null object error and a line that says:

LRProgReport.Models.SvcCodes.Report.get returned null

Is this error saying that Report is null BEFORE the add or after?

If before, why is that an issue? If after, why is the model variable not being added to the list?

Upvotes: 6

Views: 14119

Answers (2)

Emre Kabaoglu
Emre Kabaoglu

Reputation: 13146

You should initialize the Report list first and you can do it in the SvcCodes constructor;

public partial class SvcCodes
{
    public List<SvcCodeReport> Report { get; set; }
    public List<SvcCodesCheck> TestList { get; set; }

    public SvcCodes()
    {
        Report = new List<SvcCodeReport>();
    }
}

Upvotes: 11

pm100
pm100

Reputation: 50190

I suspect you are falling fowl of Linq's deferred execution model. When you do x.Where() etc the code is not run then. It is only run when you finally force the data to be used. Try this instead

var model = new SvcCodeReport()
{
  ProgResults = tasks,
  ExtraData = _context.ExtraData.Where(h => h.ServiceCode == Int32.Parse(id).ToList();
};

This will force _context.ExtraData to be evaluated right there

Upvotes: 1

Related Questions