Jak usunąć duplikaty z kolekcji za pomocą IEqualityComparer, LinQ Wyraźny

Question

Jestem w stanie usunąć duplikaty z kolekcji, mam realizowane IEqualityComparer dla pracownika klasy nadal nie jestem coraz wyjście

static void Main(string[] args) 
    { 
     List<Employe> Employeecollection = new List<Employe>(); 

     Employeecollection.Add(new Employe("abc","def")); 
     Employeecollection.Add(new Employe("lmn","def")); 
     Employeecollection.Add(new Employe("abc", "def")); 

     IEnumerable<Employe> coll = Employeecollection.Distinct(new Employe()); 

     foreach (Employe item in coll) 
     { 
      Console.WriteLine(item.fName + " " + item.lName); 
     } 

    } 

poniższa jest implementacja klasy pracownik, tutaj realizowane IEqualityComparer

class Employe : IEqualityComparer<Employe> 
{ 
    public string fName { get; set; } 
    public string lName { get; set; } 

    public Employe() 
    { 

    } 

    public Employe(string firstName, string LastName) 
    { 
     this.fName = firstName; 
     this.lName = LastName; 
    } 

    #region IEqualityComparer<pcf> Members 

    public bool Equals(Employe x, Employe y) 
    { 
     if (x.fName == y.fName && x.lName == y.lName) 
     { 
      return true; 
     } 

     return false; 
    } 

    public int GetHashCode(Employe obj) 
    { 
     return obj.GetHashCode(); 
    } 

    #endregion 
} 

Answer 1

Zapomnij IEqualityComparer i po prostu używać LINQ bezpośrednio:

EmployeeCollection.GroupBy(x => new{x.fName, x.lName}).Select(g => g.First()); 

Answer 2

trzeba zastąpić GetHashCode metodę w swoim pracownikiem. Nie zrobiłeś tego. Jednym z przykładów dobrej metody haszowania podano poniżej: (generowane przez ReSharper)

public override int GetHashCode() 
{ 
    return ((this.fName != null ? this.fName.GetHashCode() : 0) * 397)^(this.lName != null ? this.lName.GetHashCode() : 0); 
} 

teraz po Distinct jest wywoływana, wydruki pętli foreach:

abc def 
lmn def 

W twoim przypadku dzwonisz klasa obiektu GetHashCode, który nic nie wie o polach wewnętrznych.

Jedna prosta uwaga, MoreLINQ zawiera DistinctBy metodę rozszerzenia, który pozwala zrobić:

IEnumerable<Employe> coll = 
Employeecollection.DistinctBy(employee => new {employee.fName, employee.lName}); 

anonimowych obiekty mają prawidłową realizację zarówno GetHashCode i Equals metod.

Answer 3

Oto dobry tutorial

public int GetHashCode(Employe obj) 
    { 
     return obj.fname.GetHashCode()^obj.lname.GetHashCode(); 
    } 

Answer 4

Realizacja hashcode nie jest poprawna:

public override int GetHashCode() 
{ 
    return 13 * fName.GetHashCode() + 7 * lName.GetHashCode(); 
} 

Answer 5

public int GetHashCode(Employe obj) 
{ 
    return obj.GetHashCode(); 
} 

Do tego metoda, zwróć hashcode właściwości, które porównujesz dla równości, zamiast samego obiektu. Porównywanie kodu kreskowego obiektów zawsze będzie false, więc twoja lista nigdy nie będzie filtrowana dla duplikatów.

Answer 6

Wygląda na to, że porównuje się przez odniesienie zamiast treści, dlatego funkcja porównania nie działa.

Zmień to, aby użyć .Equals() zamiast == i powinno działać. przykład poniżej:

#region IEqualityComparer<pcf> Members 

public bool Equals(Employe x, Employe y) 
{ 
    if (x.fName.Equals(y.fName) && x.lName.Equals(y.lName)) 
    { 
     return true; 
    } 

    return false; 
} 

public int GetHashCode(Employe obj) 
{ 
    return obj.GetHashCode(); 
} 

#endregion