distintas en el marco Entidad

Question

Tengo una lista de objetos que algunos de ellos tienen el mismo ID, así que me gustaría para eliminar aquellos elementos que se duplican.

me trataron con algo como esto:

List<post> posts = postsFromDatabase.Distinct().ToList(); 

Pero esto no funciona!

Así que escribió este método a fin de evitar los duplicados:

public List<Post> PostWithOutDuplicates(List<Post> posts) 
    { 
     List<Post> postWithOutInclude = new List<Post>(); 
     var noDupes = posts.Select(x => x.Id).Distinct(); 
     if (noDupes.Count() < posts.Count) 
     { 
      foreach (int idPost in noDupes) 
      { 
       postWithOutInclude.Add(posts.Where(x => x.Id == idPost).First()); 
      } 
      return postWithOutInclude; 
     } 
     else 
     { 
      return posts; 
     } 
    } 

todas las ideas de cómo mejorar el rendimiento ??

Gracias de antemano.

Answer 1

Ésta es agradable y fácil:

List<Post> posts = posts 
.GroupBy(x => x.Id) 
.Select(x => x.FirstOrDefault()) 

Pero si desea escribir de la manera correcta, me gustaría aconsejarle a escribir así:

public class PostComparer : IEqualityComparer<Post> 
{ 
    #region IEqualityComparer<Post> Members 

    public bool Equals(Post x, Post y) 
    { 
     return x.Id.Equals(y.Id); 
    } 

    public int GetHashCode(Post obj) 
    { 
     return obj.Id.GetHashCode(); 
    } 

    #endregion 
} 

ya que le dará tienes más libertad cuando se trata de comparaciones adicionales. Después de haber escrito esta clase, puede usarla así:

List<Post> posts = postsFromDatabase.Distinct(new PostComparer()).ToList(); 

Answer 2

en lugar de .First(), tratan .FirstOrDefault()

Answer 3

Creo que escribe su propio comparador personalizado es un buen enfoque.

Aquí es un artículo de MSDN que explica el tema muy bien: http://support.microsoft.com/kb/320727

La razón por la que la Distinto no su funcionamiento que Distinct() no tiene idea acerca de cómo detemine si hay iguales, por lo que es el uso la referencia para determinarlo es el mismo "objeto". Está funcionando como si se suponía que funcionara. Todas las clases en la consulta no son el mismo objeto.

Al escribir su propio comparador (es fácil) se puede decir que Distinct() cómo hacer la comparacion para determinar si son iguales.

Editar: Si no se usa Distinct no es un problema y la situación no es frecuente, la primera respuesta de Piotr Justyna es simple y efectiva.