C＃7ローカルメソッドで新しい汎用パラメータを再導入することをお勧めしますか？

Question

私はC＃7の新機能、特にローカルメソッドを試していました。私はLinq Where Operatorを書いた。

イテレータブロックをローカルメソッドとして実装しました（実際に、ローカルメソッドは非同期メソッドとイテレータの完璧なソリューションであるという記事を読んでいます）。

これらの2つの実装に違いがあるのか​​、どちらが最適ではないのだろうか？

最初の実装：ここ

は私が...パラメータのためのローカル方法、新しい名前のための新しいジェネリック型パラメータを導入

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

第2の実施：

新しい汎用パラメータはありません。新しいpはありませんローカルメソッドは囲むメソッドの変数を取り込むことができるので、arameters。

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

2番目の実装が優れています。主な違いは、第二の実装が自分自身を繰り返してからあなたを解放し、暗黙的にそのパラメータを捕獲することである：

• パラメータの型を指定して、
• 指定するパラメータ名、および
• する関数に引数を渡すとき。

繰り返しを避けることは非常に重要なプログラミングの慣行なので、2番目の実装を優先する必要があります。