メソッド内に "ToListAsync（）"を含むIEnumerableを返す

Question

現在、私は他のコードを使用しています。がある

public async MyMethod Task<IEnumerable<MyResult>> FindResults() 
{ 
// some code here 
var ret = await repository.BusinessObjects.ToListAsync() 

return ret; 

} 

私質問：私はこのようなものを見たとき、これが初めてではないではありません、それはMyMethodは戻り型でIEnumerableをを使用して逆の結果が既に列挙されているので、それはですこのように保つのが良い？ リストを使用する方が良いでしょうか？

（私はそれがまた、非非同期操作に適用されると思いますが、私はこれに私は、私は念のために働くのコードでそれを見る方法を表示することにしました）

Answer 1

技術的に言えば、このようにコードを書くことに問題はありません。上記の具体的なコード例では、List<>インスタンスを返してもメソッドの戻り値の型がIEnumerable<>の場合は（技術的に）問題ありません。パフォーマンスヒットはなく、実際に返されるタイプはまだList<>です。

これは良いことではないと言われています。 とが可能な最も汎用的/抽象的な型を受け入れるように、メソッドを構造化する必要があるという設計ガイドラインがあります（の制限内で、各メソッドのパラメータとしてオブジェクトを使用しないでください） ）。

参考資料として、Vladimir KhorikovによるReturn the most specific type, accept the most generic typeと題されたこの記事を参照してください。これはまた、以前に頼まれました。Is it better to return the most specific or most general type from an action method?を参照してください。

これは、第1の署名List<MyResult>が実際に返される型であると仮定して、次の2つの署名が良い練習になることを意味します。第2の例では、更新を行っている間は入力。

public async Task<List<MyResult>> FindResultsAsync(); 
public async Task UpdateAsync(IEnumerable<MyResult> itemsForUpdate); 

Answer 2

これはあなたのユースケースに依存します。 IEnumerableは優れた抽象化を提供しますが、使用に応じてパフォーマンスコストが発生する可能性があります。

リストを非同期で作成し、列挙型のコンシューマとして（潜在的に）繰り返し使用するのは、一度繰り返すことが本当です。

私は個人的に（私の使用の場合）、あなたがパフォーマンスに重要でないパスでのみ示したパターンを使用します。結果はすでに列挙されていて、それがこのようにそれを維持する方が良いですので、初期化

Answer 3

はMyMethod戻り値の型にIEnumerableを使用することは逆効果ではないですか？ Listを使用する方が良いでしょうか？

どのように違いがあるのか​​、それが問題なのか分かりません。通常、この問題は、EFクエリのような遅延コレクションを返すときに発生し、複数の反復が複数のデータベース呼び出しを引き起こします。

Listを返しているので、複数の繰り返しが問題になることはありません。 をに変更してListを返すようにしたい場合は、複数回繰り返しても安全ですが、実際には何も変更されません。

より基本型を返すことに他利点があります。戻り値の型がListまたはIEnumerable doesnのであるかどうかが、複数の列挙についてのあなたの懸念の文脈で（例えば、それはあなたがList以外の異なるコレクション型を返すことができます）問題ありません。