Not getting Distinct Items when using Linq to List?

Question

Here is how I am attempting to get a distinct List of items...

    var queryResults = PatientList.Distinct();
    PatientList = queryResults.ToList<SelectListItem>();

For some reason, I am not getting a distinct list here.

possible duplicate of Distinct not working with LINQ to Objects — dlev
– dlev, Commented Aug 31, 2011 at 12:53

Community · Accepted Answer · 2017-05-23 12:18:06Z

9

Use

var queryResults = PatientList.GroupBy(x=>x.Id).Select(x=>x.FirstOrDefault())
    PatientList = queryResults.ToList<SelectListItem>();

You can always try

 PatientList = PatientList.GroupBy(x=>x.Id).Select(x=>x.FirstOrDefault()).ToList<SelectListItem>();

It will give you the distinct results based off whatever you group by

Check out http://blog.jordanterrell.com/post/LINQ-Distinct()-does-not-work-as-expected.aspx

Also another question for reference: Returning a Distinct IQueryable with LINQ?

edited May 23, 2017 at 12:18

CommunityBot

11 silver badge

answered Aug 31, 2011 at 12:55

Gage

7,5239 gold badges50 silver badges77 bronze badges

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

5 Comments

SoftwareSavant Over a year ago

The type argument for method yadda yadda cannot be inferred from the usage. Try specifying the type arguments explicitly.

Gage Over a year ago

@DmainEvent, Change x.Id to whatever you need to group everything by. Not sure what you mean by your comment, which argument cannot be inferred?

SoftwareSavant Over a year ago

sorry about that. I am working on a different computer right now to right this message than the one that has the code (Little lazy this morning :-) ). But I did change it to the right value... Not sure why it isn't working.

Gage Over a year ago

@DainEvent, hmm Ok well which line did you get that error on? The groupby or where your converting to a list? The groupby should work correct what are you grouping everything by? What you are doing is similar to stackoverflow.com/questions/4472369/…

SoftwareSavant Over a year ago

@Gage... I was doing my usual and being an idiot. I forgot to put () at the end of FirstOrDefault. It works now. Thanks a ton.

Thomas Levesque · Accepted Answer · 2011-08-31 12:53:05Z

1

Your SelectListItem class needs to override Equals and GetHashCode (and optionally implement IEquatable<SelectListItem>). Otherwise, different instances with the same values will be considered different.

Another option is to implement a IEqualityComparer<SelectListItem> and pass it as the second parameter to Distinct.

answered Aug 31, 2011 at 12:53

Thomas Levesque

294k73 gold badges639 silver badges769 bronze badges

Comments

Community · Accepted Answer · 2017-05-23 12:02:18Z

1

Not sure what kind of items your PatientList contains, but I guess you have to implement IEquatable on your custom object.

This have been aswered before here: Distinct not working with LINQ to Objects

edited May 23, 2017 at 12:02

CommunityBot

11 silver badge

answered Aug 31, 2011 at 12:51

Muttok

6716 silver badges4 bronze badges

2 Comments

Thomas Levesque Over a year ago

If you implement it in the objects being compared, that would be IEquatable, not IEqualityComparer...

SoftwareSavant Over a year ago

It is actually a List of SelectListItem's

Collectives™ on Stack Overflow

Not getting Distinct Items when using Linq to List?

3 Answers 3

5 Comments

Comments

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

5 Comments

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related