DistinctBy aber ignorieren null/leer

Question

ich eine Liste der Filme, und ich brauche sie mit einer anderen Liste zu verschmelzen und vervielfältigen.

Ich bin mit Jon Skeet DistinctBy(m => m.SomeUniqueMovieProperty) dies zu erreichen, und es funktioniert OK. Außer, wir fanden bald heraus, dass es Fälle geben würde, in denen 10-20% der Filme (in jeder Liste) diese Eigenschaft nicht ausgefüllt haben, was DistinctBy dazu veranlasst, sie in einen glücklichen Film zu stürzen.

Dies ist ein Problem, wollen wir alle diese Filme halten, die für diese Eigenschaft keinen Wert haben. Anfangs dachte ich daran, diese Filme aus jeder Sammlung zu extrahieren, zu duplizieren und dann wieder zusammenzufügen, gibt es eine kürzere Lösung für dieses Problem?

Answer 1

verketten die Ergebnisse DistinctBy() mit den Ergebnissen der Where([null or empty]).

var nullMovies = allMovies.Where(m=>string.IsNullOrEmpty(m.SomeUniqueMovieProperty)); 

var distinctNonNullMovies = allMovies.Where(m => !string.IsNullOrEmpty(m.SomeUniqueMovieProperty)).DistinctBy(m => m.SomeUniqueMovieProperty); 

var result = nullMovies.Concat(distinctNonNullMovies); 

Answer 2

Wenn Sie all die Nullen einschließen möchten Sie die Null-Eigenschaft mit etwas ersetzen müssen, die einzigartig ist, wenn es null ist. Unter der Annahme, dass die Eigenschaft eine Zeichenfolge ist, wird Guid für diesen Job gut funktionieren.

.DistinctBy(m => m.SomeUniqueMovieProperty ?? Guid.NewGuid().ToString()) 

Jedes Mal, wenn es eine Eigenschaft mit einem Nullwert trifft, wird es mit einem zufälligen neuen GUID-Wert gefüllt.

---

Wenn Sie auch leer Titel nicht haben, um die Abfrage erhalten entfernt Änderung

.DistinctBy(m => String.IsNullOrEmpty(m.SomeUniqueMovieProperty) ? Guid.NewGuid().ToString() : m.SomeUniqueMovieProperty) 

---

Eine weitere Option ist Ihre eigene DistinctBy machen, die die Art und Weise Sie wollen verhält. Dies ist eine optimierte Version der original source, die nur den Filter anwendet, wenn shouldApplyFilter Wahr zurückgibt, Kommentare werden auch der Kürze halber entfernt.

static partial class MoreEnumerable 
{ 
    public static IEnumerable<TSource> ConditionalDistinctBy<TSource, TKey>(this IEnumerable<TSource> source, 
     Func<TSource, TKey> keySelector, Func<TKey, bool> shouldApplyFilter) 
    { 
     return source.ConditionalDistinctBy(keySelector, shouldApplyFilter, null); 
    } 

    public static IEnumerable<TSource> ConditionalDistinctBy<TSource, TKey>(this IEnumerable<TSource> source, 
     Func<TSource, TKey> keySelector, Func<TKey, bool> shouldApplyFilter, IEqualityComparer<TKey> comparer) 
    { 
     if (source == null) throw new ArgumentNullException("source"); 
     if (keySelector == null) throw new ArgumentNullException("keySelector"); 
     if (shouldApplyFilter == null) throw new ArgumentNullException("shouldApplyFilter"); 
     return ConditionalDistinctByImpl(source, keySelector, shouldApplyFilter, comparer); 
    } 

    private static IEnumerable<TSource> ConditionalDistinctByImpl<TSource, TKey>(IEnumerable<TSource> source, 
     Func<TSource, TKey> keySelector, Func<TKey, bool> shouldApplyFilter, IEqualityComparer<TKey> comparer) 
    { 
     var knownKeys = new HashSet<TKey>(comparer); 
     foreach (var element in source) 
     { 
      var key = keySelector(element); 
      if (shouldApplyFilter(key) && knownKeys.Add(key)) 
      { 
       yield return element; 
      } 
     } 
    } 
} 

Es wäre wie

.ConditionalDistinctBy(m => m.SomeUniqueMovieProperty, s => !String.IsNullOrEmpty(s)); 

Answer 3

man konnte sie vielleicht verwendet werden, filtern auf einem Verbund verschiedene Schlüssel, wie wie folgt

movies.DistinctBy(m => String.Format({0}{1}{...},m.prop1,m.prop2,[])); 

Answer 4

Eine letzte Möglichkeit, die wahrscheinlich übertrieben ist, Sie kann IEqualityComparer implementieren und die Logik dort einfügen, wenn null als eindeutig betrachtet wird. DistinctBy hat nur in diesem Fall eine Überladung.

public class MovieComparer : IEqualityComparer<string> 
{ 

    public bool Equals(string x, string y) 
    { 
     if (x == null || y == null) 
     { 
      return false; 
     } 

     return x == y; 
    } 

    public int GetHashCode(string obj) 
    { 
     if (obj == null) 
     { 
      return 0; 
     } 
     return obj.GetHashCode(); 
    } 
} 

Answer 5

Unter der Annahme, m ‚s Equals/GetHashCode nicht außer Kraft gesetzt wird, wenn m.SomeUniqueMoviePropertynull ist, und Sie haben keine andere eindeutigen Schlüssel, können Sie m sich als eindeutigen Schlüssel verwenden.

DistinctBy(m => (object) m.SomeUniqueMovieProperty ?? m)