DistinctBy但忽略null / empty

时间:2016-06-20 19:54:08

标签: c# linq morelinq

我有一个电影列表,我需要将它们与另一个列表合并并复制。

我正在使用Jon Skeet的DistinctBy(m => m.SomeUniqueMovieProperty)来实现这一目标,而且运行正常。除此之外,我们很快发现有些情况下,10-20%的电影(在任何一个列表中)都没有填写此属性,导致DistinctBy将它们合并为1部幸运电影。

这是一个问题,我们希望保留所有那些没有此属性值的电影。最初我想从每个集合中提取这些电影,复制,然后再次合并它们,这个问题是否有更短的解决方案?

5 个答案:

答案 0 :(得分:3)

DistinctBy()的结果与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);

答案 1 :(得分:3)

如果要包含所有空值,则需要将null属性替换为null时的唯一值。假设该属性是一个字符串,Guid将很好地完成这项工作。

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

只要它遇到一个空值的属性,它就会被一个随机的新guid值填充。

如果您还想删除空标题,请将查询更改为

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

另一个选择是让你自己的DistinctBy按照你想要的方式行事。这是original source的经过调整的版本,仅在shouldApplyFilter返回true时才应用过滤器,为简洁起见,也会删除注释。

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;
            }
        }
    }
}

它会像

一样使用
.ConditionalDistinctBy(m => m.SomeUniqueMovieProperty, s => !String.IsNullOrEmpty(s));

答案 2 :(得分:2)

也许您可以在复合的不同键上过滤它们,如下所示

p

答案 3 :(得分:2)

假设m的{​​{1}} / Equals未被覆盖,如果GetHashCodem.SomeUniqueMovieProperty并且您没有任何其他唯一密钥,您可以使用null本身作为唯一键。

m

答案 4 :(得分:1)

最后一种方法,可能是矫枉过正,你可以实现IEqualityComparer并将逻辑放在那里,如果null被认为是唯一的。对于这种情况,DistinctBy有一个过载。

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();
    }
}