LINQ表达式中的异常处理

Question

我有一个简单的LINQ表达式,如:

newDocs = (from doc in allDocs
       where GetDocument(doc.Key) != null
       select doc).ToList();

问题是,GetDocument()可能会抛出异常.如何忽略GetDocument(doc.Key)== null或抛出异常的所有doc元素？

旧学校的相同代码如下:

foreach (var doc in allDocs)
            {
                try
                {
                    if (GetDocument(doc.Key) != null) newDocs.Add(doc);
                }
                catch (Exception)
                {
                    //Do nothing...
                }
            }

Answer 1

allDocs.Where(doc => {
    try {
        return GetDocument(doc.Key) != null;
    } catch {
        return false;
    }
}).ToList();

我不确定是否可以使用查询理解语法,除非通过这样的巴洛克式暴行:

newDocs = (from doc in allDocs
           where ((Predicate<Document>)(doc_ => {
               try {
                   return GetDocument(doc_.Key) != null;
               } catch {
                   return false;
               }
           }))(doc)
           select doc).ToList();

Answer 2

可以编写linq扩展名以跳过导致异常的所有元素.请参阅此stackoverflow帖子

 public static IEnumerable<T> CatchExceptions<T> (this IEnumerable<T> src, Action<Exception> action = null) {
        using (var enumerator = src.GetEnumerator()) {
            bool next = true;

            while (next) {
                try {
                    next = enumerator.MoveNext();
                } catch (Exception ex) {
                    if (action != null) {
                        action(ex);
                    }
                    continue;
                }

                if (next) {
                    yield return enumerator.Current;
                }
            }
        }
    }

例:

ienumerable.Select(e => e.something).CatchExceptions().ToArray()

ienumerable.Select(e => e.something).CatchExceptions((ex) => Logger.Log(ex, "something failed")).ToArray()

发布此处以防万一其他人首先找到此答案.

Answer 3

您可以移动整个try catch块并GetDocument调用另一个方法

Document TryGetDocument(string key)
{
         try
         {
            if (GetDocument(doc.Key) != null) 
              return doc;
         }
         catch (Exception)
         {
             return null;
         }
     return null;
}

然后在您的查询中使用此功能 -

newDocs = (from doc in allDocs
       where TryGetDocument(doc.Key) != null
       select doc).ToList();

这将使您的查询简洁易读.