IEnumerable是否是？更换不好的设计？

Question

我创建了以下扩展方法。 这是一个糟糕的设计吗？我应该为 ICollection 执行此操作吗？

public static IEnumerable<TSource> Replace<TSource>(this IEnumerable<TSource> source, IEnumerable<TSource> newItems)
{
    return source.Except(newItems).Union(newItems);
}

public static IEnumerable<TSource> Replace<TSource>(this IEnumerable<TSource> source, IEnumerable<TSource> newItems, IEqualityComparer<TSource> comparer)
{
    return source.Except(newItems, comparer).Union(newItems, comparer);
}

更新：我认为命名有点错误。我想要这个函数做的是添加并覆盖。 更新：向 Union 添加了比较器。

Answer 1

您拥有的简化逻辑操作是

 return newItems.Union(source);
 return newItems.Union(source, comparer);

当您采用 source.Except(newItems).Union(newItems) 时，您将采用 source 中的所有不同项目，但 中的任何项目除外code>newItems，然后添加 newItems 中的所有不同项目。这就是 Union 所做的！获取 newItems 的所有不同项目，并将来自 source 的尚不存在的不同项目添加到其中。

您可以使用不同的名称AddWithOverwrite、AddWithReplace等来调用它，这些名称将错误（没有任何内容被添加到任何内容中，源和 newItems 都没有以任何方式修改），但操作本身不必像代码那样复杂。

有一些权衡。通过上述方法，所有 newItems 都将出现在 source 之前。计数器是您已经丢失了替换商品的订单。

Answer 2

不。

如果您查看 LINQ 中的原始扩展方法，它们都使用 IEnumerable 接口。如果您使用通用集合，这应该足够了。但是，您的 Replace 方法缺少要替换的成员的参数。你应该这样做：

public static IEnumerable<TSource> Replace<TSource>(this IEnumerable<TSource> source, IEnumerable<TSource> oldItems, IEnumerable<TSource> newItems)
{
    return source.Except(oldItems).Union(newItems);
}


public static IEnumerable<TSource> Replace<TSource>(this IEnumerable<TSource> source, IEnumerable<TSource> oldItems, IEnumerable<TSource> newItems, IEqualityComparer<TSource> comparer)
{
    return source.Except(oldItems, comparer).Union(newItems);
}

更新

这是我可以想出的 OverWrite 方法：

    public static IEnumerable<TSource> OverWrite<TSource, TSelectItem>(this IEnumerable<TSource> source, IEnumerable<TSource> newItems, Func<TSource, TSelectItem> selectProperty) where TSource : class
    {
        IEnumerable<TSource> result = source;

        if (newItems != null)
        {
            result = source.Select(s => newItems.FirstOrDefault(n => EqualityComparer<TSelectItem>.Default.Equals(selectProperty(s), selectProperty(n))) ?? s);
        }

        return result;
    }

    public static IEnumerable<TSource> OverWrite<TSource, TSelectItem>(this IEnumerable<TSource> source, IEnumerable<TSource> newItems, Func<TSource, TSelectItem> selectProperty,IEqualityComparer<TSelectItem> propertyComparer) where TSource : class
    {
        IEnumerable<TSource> result = source;

        if (newItems != null)
        {
            result = source.Select(s => newItems.FirstOrDefault(n => propertyComparer.Equals(selectProperty(s), selectProperty(n))) ?? s);
        }

        return result;
    }

你可以像这样使用它：

someObjects.OverWrite(newObjects, item => item.ID);
someObjects.OverWrite(newObjects, item => item.ID, new PropertyComparer());