SingleOrDefault（）のベストプラクティス

Question

私のコードをより良くしたいです。最初の例を2番目の例に安全に書き換えることはできますか？

IQueryable<PDF> pdfList = (from pdfobj in pdfDataContext.PDFs 
          where pdfobj.Id == pdfId 
          select pdfobj); 
if (pdfList.Count() > 0) 
{ 
    PDF pdfOldWay = pdfList.FirstOrDefault(); 
    pdfOldWay. // do something. (pdfOldWay can't be null here...) 
} 

-

PDF pdfNewWay = (from pdfobj in pdfDataContext.PDFs 
       where pdfobj.Id == pdfId 
       select pdfobj).SingleOrDefault(); 
if (pdfNewWay != null) 
{ 
    // do something 
} 

-

EDIT：明確にされていないため

申し訳ありません。私の問題は、最初にリストを使用することなくPDFオブジェクトを直接取得することです。私はカウントが0より大きく、ちょうど良く見えないので、チェックをしたくない。

Answer 1

はい、安全です。あなたはまた、少しあなたのクエリを簡素化することができます：

PDF pdfNewWay = pdfDataContext.PDFs.SingleOrDefault(p => p.Id == pdfId); 
if (pdfNewWay != null) 
{ 
    // do something 
} 

SingleOrDefaultとFirstOrDefaultの唯一の違いは、複数の一致が見つかった場合は、このチェックをしない限り、あなたが同様に固執するようSingleOrDefaultは、例外がスローされますということですFirstOrDefault。

Answer 2

はい、同じ結果が得られますが、問題はありません。

Answer 3

SingleOrDefaultは2つ以上の項目がある場合に例外をスローするため、FirstOrDefaultを使用してください。それはあなたのために大丈夫ですか？

一方、いくつかのidに対してpdfobjectが1つしかないことを保証したい場合は、SignleOrDefaultを使用するよりも優れています。

Answer 4

常に0または1の行があることが保証されている場合は、SingleOrDefaultが最適なソリューションです。