Entity framework NullReferenceException

Question

Why would this code generate null exception:

Public Function MessageCount(ByVal mId As Long) As Integer
    Dim messages As List(Of InboxMessage) = Nothing
    Using ctx As New UFCWEntities.UFCWEntities

        Dim allMessageIds = ctx.InboxLinks.Where(Function(o) o.MemberId = mId).ToList()
        For Each i As InboxLink In allMessageIds
            messages.Add(ctx.InboxMessages.FirstOrDefault(Function(o) o.InboxMessageId = i.InboxMessageId))
        Next

    End Using

    If Not IsNothing(messages) Then
        Return messages.Count()
    End If

    Return 0

End Function

Exception is generated when trying to do messages.Add

Almost all cases of NullReferenceException are the same. Please see "What is a NullReferenceException in .NET?" for some hints. — John Saunders
– John Saunders, Commented Mar 19, 2014 at 20:06

Brian Driscoll · Accepted Answer · 2014-03-19 19:00:19Z

1

You're getting this error because ctx.InboxMessages.FirstOrDefault(...) is returning null.

You should be null checking first, e.g.:

For Each i As InboxLink In allMessageIds
    Dim firstMsg = ctx.InboxMessages.FirstOrDefault(...)
    If Not IsNothing(firstMsg) Then
        messages.Add(firstMsg)
    End If
Next

answered Mar 19, 2014 at 19:00

Brian Driscoll

19.7k3 gold badges51 silver badges66 bronze badges

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

1 Comment

Ashley Medway Over a year ago

To give more context, There is no item in InboxMessages which matches i.InboxMessageId.

Jim Wooley · Accepted Answer · 2014-03-19 20:02:36Z

0

You aren't instantiating your messages variable thus when you try to Add to it, it can't because messages is still Nothing. Change the first line to

Dim messages As New List(Of InboxMessage)

That being said, you may want to consider thinking about the whole operation as a set based operation. You shouldn't need to hydrate objects if all you want to do is filter them down and count the filtered results. From the code you supplied, you should be able to simplify this to:

Using ctx As New UFCWEntities.UFCWEntities
   return ctx.InboxLinks.Count(function(o) o.MemberId = mId)
End Using

answered Mar 19, 2014 at 20:02

Jim Wooley

10.4k1 gold badge27 silver badges44 bronze badges

Collectives™ on Stack Overflow

Entity framework NullReferenceException

2 Answers 2

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related