在C＃7本地方法中重新引入新的通用参数是否是一种好习惯？

Question

我正在尝试C＃7的新功能，尤其是本地方法。 我写了Linq Where运算符。

我将迭代器块实现为本地方法（实际上，我读过一些文章说，本地方法是异步方法和迭代器的理想解决方案）。

我想知道这两种实现之间是否有区别，如果没有，那是最好的？

第一次执行：

在这里，我为本地方法引入了新的泛型类型参数，为参数引入了新名称...

public static  IEnumerable<TSource> Where<TSource> (this IEnumerable<TSource> source,  Func<TSource, bool> predicate) {
    if(source == null) throw new ArgumentNullException(nameof(source));
    if(predicate == null) throw new ArgumentNullException(nameof(predicate));
    return WhereIterator(source,  predicate);

    IEnumerable<TSequence> WhereIterator<TSequence> (IEnumerable<TSequence> localSource,  Func<TSequence, bool>  localPredicat) {
        foreach(TSequence item in localSource) {
            if(localPredicat(item)) {
                yield return item;
            }
        }
    }
}

第二实施：

没有新的通用参数，也没有新参数，因为本地方法可以捕获封闭方法的变量。

public static  IEnumerable<TSource> Where<TSource> (this IEnumerable<TSource> source,  Func<TSource, bool> predicate) {
    if(source == null) throw new ArgumentNullException(nameof(source));
    if(predicate == null) throw new ArgumentNullException(nameof(predicate));
    return  WhereIterator();

    IEnumerable<TSource> WhereIterator () {
        foreach(TSource item in source) {
            if(predicate(item))
                yield return item;
            }
        }
    }
}

Answer 1

您的第二个实现更好。 主要区别在于第二种实现隐式捕获其参数，使您不必重复自己的工作：

• 指定参数类型时，
• 指定参数名称时，以及
• 将参数传递给函数时。

避免重复是非常重要的编程习惯，因此您应该选择第二种实现。