Remove duplicate rows mvc

Question

I have this method

Meeting is a class
Attendees is an ICollection in Meeting

Class

public partial class Meeting
    {

        public Meeting()
        {
            this.Attendees = new List<Attendees>();
        }

public virtual ICollection<Attendees> Attendees{ get; set; }
[...]

Method Controller

private void RemoveRowsDuplicated(Meeting model)
        {
            if (model.Attendees != null)
            {
                foreach (var item in model.Attendees.GroupBy(x => x.UserName).Select(y => y.Last()))
                {
                    context.Attendees.Remove(item);
                }
            }
        }

The objective is remove duplicate Attendees with the same username in the table.
But the current method it deletes all records and keeps the duplicate
Where am I going wrong?

I believe, it was already answered here: stackoverflow.com/questions/1606679/… — drcolombo
– drcolombo, Commented Jan 14, 2016 at 15:50

Mitklantekutli · Accepted Answer · 2016-01-14 15:43:07Z

3

Correct version of your method will look like this:

private static void RemoveRowsDuplicated(Meeting model)
        {
            if (model.Attendees != null)
            {
                var duplicates = new List<Attendees>();
                foreach (var item in model.Attendees.GroupBy(x => x.UserName).Where(x=>x.Count()>1))
                {
                    duplicates.AddRange(item.Skip(1));
                }
                duplicates.ForEach(x=>context.Attendees.Remove(x));
            }
        }

edited Jan 14, 2016 at 15:43

answered Jan 14, 2016 at 15:37

Mitklantekutli

4101 gold badge3 silver badges16 bronze badges

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

Comments

jamdownian · Accepted Answer · 2016-01-14 15:43:55Z

0

You can try writing raw SQL and invoking via EF and return Attendees objects in a list.

var query = "Select * from Attendees group by username";
var attendeesList = dbContext.Database.SqlQuery<Attendees>(query).ToList<Attendees>();

answered Jan 14, 2016 at 15:43

jamdownian

4203 silver badges11 bronze badges

Comments

AntonE · Accepted Answer · 2016-01-14 15:47:12Z

-1

As I can see you grouped elements by name and remove last item. So you remove unique elements. Like this

 private void RemoveRowsDuplicated(Meeting model)
        {
             if (model.Attendees != null)
             {
                var temporaryAtendees = new List<Attendees>();
                foreach(var item in model.Attendees)
                {
                    if (temporaryAtendees.Contains(item))
                    {
                        context.Attendees.Remove(item);
                    }
                    else
                    {
                        temporaryAtendees.Add(item);
                    }
                }
                }
            }

edited Jan 14, 2016 at 15:47

answered Jan 14, 2016 at 15:26

AntonE

6826 silver badges13 bronze badges

1 Comment

AntonE Over a year ago

Yes I see now you have other question and my answer not about this

Collectives™ on Stack Overflow

Remove duplicate rows mvc

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