尝试使用流畅的表达式进行捕捉
本文关键字:表达式 | 更新日期: 2023-09-27 18:35:06
此 LINQ 查询表达式失败,并显示 Win32Exception "访问被拒绝":
Process.GetProcesses().Select(p => p.MainModule.FileName)
这在IOException"设备未就绪"中失败:
DriveInfo.GetDrives().Select(d => d.VolumeLabel)
过滤掉无法访问的对象并避免异常的最佳方法是什么?
编写一个扩展方法!
void Main()
{
var volumeLabels =
DriveInfo
.GetDrives()
.SelectSafe(dr => dr.VolumeLabel);
}
// Define other methods and classes here
public static class LinqExtensions
{
public static IEnumerable<T2> SelectSafe<T,T2>(this IEnumerable<T> source, Func<T,T2> selector)
{
foreach (var item in source)
{
T2 value = default(T2);
try
{
value = selector(item);
}
catch
{
continue;
}
yield return value;
}
}
}
通过这种方式,您可以自定义所需的任何行为,并且不必创建笨重和笨拙的where子句,这样您甚至可以在出现异常时让它返回替代值。
基于注释的更新:此解决方案不适用于常见枚举器。它确实基于问题示例中使用的枚举器工作。因此,它不是一个通用的解决方案。因为它是作为通用解决方案编写的,所以我建议不要使用它(为了简单)。我将保留此答案以丰富知识库。
另一种扩展方法解决方案。为什么我更喜欢它而不是现有的解决方案?
- 我们只想跳过导致异常的元素。这是我们的 LINQ 扩展的唯一关注点。
- 此实现不会混合
Select
和try/catch
的关注点。 - 在需要时,我们仍然可以使用现有的 LINQ 方法,例如
Select
。 - 它是可重用的:它允许在 LINQ 查询中多次使用。
- 它遵循 linq 命名约定:我们实际上跳过类似于
Skip
和SkipWhile
方法。
用法:
var result = DriveInfo
.GetDrives()
.Select(d => d.VolumeLabel)
.SkipExceptions() // Our extension method
.ToList();
法典:
public static class EnumerableExt
{
// We use the `Skip` name because its implied behaviour equals the `Skip` and `SkipWhile` implementations
public static IEnumerable<TSource> SkipExceptions<TSource>(this IEnumerable<TSource> source)
{
// We use the enumerator to be able to catch exceptions when enumerating the source
using (var enumerator = source.GetEnumerator())
{
// We use a true loop with a break because enumerator.MoveNext can throw the Exception we need to handle
while (true)
{
var exceptionCaught = false;
var currentElement = default(TSource);
try
{
if (!enumerator.MoveNext())
{
// We've finished enumerating. Break to exit the while loop
break;
}
currentElement = enumerator.Current;
}
catch
{
// Ignore this exception and skip this item.
exceptionCaught = true;
}
// Skip this item if we caught an exception. Otherwise return the current element.
if (exceptionCaught) continue;
yield return currentElement;
}
}
}
}
你的答案是正确的。当然,您可以尝试将检查逻辑隐藏在扩展方法中。
public static IEnumerable<TElement> WhereSafe<TElement, TInner>(this IEnumerable<TElement> sequence, Func<TElement, TInner> selector)
{
foreach (var element in sequence)
{
try { selector(element); }
catch { continue; }
yield return element;
}
}
Process
.GetProcesses()
.WhereSafe(p => p.MainModule)
.Select(p => p.MainModule.FileName)
或者更好的是:
public static IEnumerable<TInner> TrySelect<TElement, TInner>(this IEnumerable<TElement> sequence, Func<TElement, TInner> selector)
{
TInner current = default(TInner);
foreach (var element in sequence)
{
try { current = selector(element); }
catch { continue; }
yield return current;
}
}
Process
.GetProcesses()
.TrySelect(p => p.MainModule.FileName)
插入一个 WHERE 过滤器(尝试访问任何对象并吸收可能的访问错误):
{ try { var x = obj.MyProp; return true; } catch { return false; } }:
第一个表达式:
Process
.GetProcesses()
.Where(p => { try { var x = p.MainModule; return true; } catch { return false; } })
.Select(p => p.MainModule.FileName)
第二个表达式:
DriveInfo
.GetDrives()
.Where(d => { try { var x = d.VolumeLabel; return true; } catch { return false; } })
.Select(d => d.VolumeLabel)
我会尝试第一种情况:
//Declare logger type
private readonly ILog _log = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType);
Process.GetProcesses()
.Where(p => {
try {
var x = p.MainModule;
return true;
}
catch(Win32Exception e2)
{ IgnoreError(); }
})
.Select(p => p.MainModule.FileName)
public static void IgnoreError(Exception e)
{
#if DEBUG
throw e2;
//Save the error track, I prefer log4net
_log.Info("Something bad happened!");
#end if
}
对于第二种情况,我宁愿使用 IF 并保存日志:
//Somewhere in the begging of your class, in a place whose name I do not care to remember ...
//Declare logger type
private readonly ILog _log = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType);
public List<string> VolumenLabels()
{
//Return the List<T>
List<string> myVolumeLabels = new List<string>();
//Getting the info
DriveInfo[] allDrives = DriveInfo.GetDrives();
foreach(DriveInfo drive in allDrives)
{
if (drive.IsReady == true)
{
myVolumeLabels.Add(drive.VolumeLabel.ToString());
}
else
{
_log.Info("Check the Drive: " + drive.Name + " the device is not ready.");
}
}
return myVolumeLabels;
}
我希望,我帮了一点忙...有好的一天!