c# - SingleOrDefault() 的最佳实践

Question

我想让我的代码更好。我可以安全地将第一个示例重写为第二个示例吗？

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
}

--

编辑:

抱歉没说清楚。我的问题是直接获取 PDF 对象，而不必先使用列表。我不想检查计数是否大于 0，因为它看起来不太好。

Answer 1

是的，这看起来很安全。您还可以稍微简化您的查询:

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

SingleOrDefault 和 FirstOrDefault 之间的唯一区别是，如果找到多个匹配项，SingleOrDefault 将抛出异常，因此除非您想要此检查，否则您最好坚持使用 FirstOrDefault。