转换为方法组 Resharper

Question

我在下面编写了一个像这样的方法：

internal static IList<EmpowerTaxView> GetEmpowerTaxViewsByLongAgencyAndAgencyTaxTypes(
   IList<EmpowerCompanyTaxData> validEmpowerCompanyTaxDatas,
   IList<EmpowerTaxView> empowerTaxViews)
{
    IList<EmpowerTaxView> result = new List<EmpowerTaxView>();

    foreach (EmpowerCompanyTaxData empowerCompanyTaxData in validEmpowerCompanyTaxDatas)
    {
         IList<EmpowerTaxView> validEmpowerTaxViews =
           GetEmpowerTaxViewsByLongAgencyAndTaxType(
             empowerCompanyTaxData, empowerTaxViews);

         validEmpowerTaxViews.ToList().ForEach(delegate(EmpowerTaxView etv)
         {
              result.Add(etv);   
         });   
    }

    return result;
}

对于这个方法，resharper 说：

validEmpowerTaxViews.ToList().ForEach(delegate(EmpowerTaxView etv)
{
    result.Add(etv);   
});

转换为方法组。这意味着什么以及应该采取什么措施来消除这种情况。

Answer 1

Resharper 的意思是，您可以通过使用方法组 Add 更简单地表达 ForEach 代码。
示例：

validEmpowerTaxViews.ToList().Foreach(result.Add);

Add 定义的方法组与 ForEach 所需的委托兼容，因此 C# 编译器将负责执行转换。 Resharper 中的默认设置是优先选择方法组而不是 lambda 和显式委托创建语句。

Answer 2

JaredPar 已经提供了正确的答案，我只是想建议该方法的更简单的实现：

internal static IList<EmpowerTaxView> GetEmpowerTaxViewsByLongAgencyAndAgencyTaxTypes(
    IList<EmpowerCompanyTaxData> validEmpowerCompanyTaxDatas,
    IList<EmpowerTaxView> empowerTaxViews)
{
    var results =
        from empowerCompanyTaxData in validEmpowerCompanyTaxDatas
        from etv in GetEmpowerTaxViewsByLongAgencyAndTaxType(
            empowerCompanyTaxData, empowerTaxViews)
        select etv;
    return results.ToList();
}

Answer 3

接受 Resharper 的建议，看看它会带来哪些变化。您随时可以撤消它们。

如果您对更改不满意并且不希望 Resharper 将来提出建议，那么您可以禁用该特定选项 - 其他选项将仍然可用。有关详细信息，请参阅此处的答案。

Resharper：变量

Answer 4

来自 Microsoft 网站 方法组是由成员查找产生的一组重载方法

您可以检查检查 MSIL 代码，如果您使用方法组，那么它只会创建一个指向该方法的新委托。
但是，如果您使用 lambda 表达式，那么它会生成一个带有匿名方法的类。结果是方法组需要的内存少于 lambda，并且 Resharper 建议您进行优化