Expression> 类型的嵌套类型的替代方案

Question

我有一个调用服务时使用的函数。在调用服务之前，它将创建一个日志条目：

protected TResult CallService<TService, TResult>(TService service,
    Expression<Func<TService, TResult>> functionSelector)
{
    Logger.LogServiceCall(service, functionSelector);
    return functionSelector.Compile()(service);
}

Visual Studio 2010 代码分析器在以下消息中通知我不应使用嵌套类型：

CA1006：Microsoft.Design：考虑 设计在哪里 'ServiceManager.CallService(TService, 表达式>)' 不嵌套泛型类型 '表达式>'。

虽然我可以简单地为此条目创建抑制规则，但是否存在替代方案可以防止显示此类警告？

Answer 1

在这种情况下我会抑制它，因为调用者不必处理嵌套泛型，他只是传递一个 lambda 表达式，这很容易使用。

CA 不会对 lambda 表达式设置例外。有时抑制它比编写奇怪的代码更好。

Answer 2

老实说，大多数时候我都会禁止该规则。虽然我可以理解某些嵌套类型的构造是可以避免的，但情况往往并非如此；您通常希望将其留给调用站点，因为您无法保证调用站点将希望以相同的方式实例化嵌套泛型类型。

这是我觉得有点专横的规则之一；我基本上同意其中的大多数，但不同意这一点。

Answer 3

像您这样的方法在 Linq 中广泛使用，例如：

public static IQueryable<TSource> Where<TSource>(this IQueryable<TSource> source, 
    Expression<Func<TSource, bool>> predicate)

另一种方法是声明委托类型来替换嵌套的 Func，但这同样可能会让更有经验的开发人员感到困惑谁习惯于使用表达式树。

微软显然对嵌套泛型表达式类型的 CA1006 做出了例外，我们也应该如此。

Answer 4

您可以使用 SuppressMessageAttribute 抑制消息警告。

[System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Design","CA1006:<rule name>")]
protected TResult CallService<...Snip...