HAA0502显式新引用类型分配

问题描述

https://marketplace.visualstudio.com/items?itemName=MukulSabharwal.ClrHeapAllocationAnalyzer

其中一种方法如下所示

执行#1

public void ConfigureServices(IServiceCollection services) {

services.AddSingleton<IPocoDynamo>(serviceProvider => {
    var pocoDynamo = new PocoDynamo(serviceProvider.GetRequieredService<IAmazonDynamoDB>());
    pocoDynamo.SomeMethod();
    return pocoDynamo;
});

}

例2

public async Task<EventTO> AddEvent(EventTO eventObj)
{
  try
    {       
      throw new Exception("Error!");
    }
 catch (Exception ex)
 {
   Logger.Log(ex, eventObj);
   return null;
  }
}

我在整个应用程序中都在使用DI。但无论分析器在哪里找到new关键字，都会警告为

HAA0502显式新引用类型分配

无论在哪里使用Lambda表达式，它都是警告(如示例1所示)

Warning HAA0301 Heap allocation of closure Captures:

导致此问题的原因&；如何解决此问题？

谢谢！

推荐答案

堆分配分析器用于标记代码执行的所有分配。这不是您希望始终处于打开状态的内容：请考虑以下愚蠢的代码

        public static string MyToString(object? o)
        {
            if (o == null)
                throw new ArgumentNullException(nameof(o)); // HAA0502 here

            return o.ToString() ?? string.Empty;
        }

分析器将在标记的行上以警告的形式发出HAA0502作为信息，告诉您正在分配新实例。现在，很明显在这种情况下您在做什么，这是一个微不足道的警告，但是分析器的目的是帮助您发现可能会使您的代码变慢的讨厌分配。

现在考虑以下愚蠢的代码：

        public static void Test1()
        {
            for (int i = 0; i < 100; i++)
            {
                var a = i + 1;

                var action = new Action(
                    () =>   // HAA0301 Heap allocation of closure Capture: a
                    {
                        Console.WriteLine(a);
                    }
                );

                action();
            }
        }

除了将在new Action(上标记的HAA0502(因为我们正在创建新对象)之外，lambda上还有一个额外的警告：HAA0301。这就是分析器变得更有用的原因：这里分析器告诉您，运行时将创建一个包含捕获变量的新对象a。如果您不熟悉这一点，您可能会认为在该代码中会转换成类似以下内容(仅用于解释目的)：

        private sealed class Temp1
        {
            public int Value1 { get; }

            public Temp1(int value1)
            {
                Value1 = value1;
            }

            public void Method1()
            {
                Console.WriteLine(Value1);
            }
        }

        public static void Test1()
        {
            for (int i = 0; i < 100; i++)
            {
                var a = i + 1;

                var t = new Temp1(a);
                t.Method1();
            }
        }

在后一段代码中，很明显，您在每次迭代中都在分配一个对象。

您可能会遇到的主要问题是：分配对象有问题吗？在99.9%的情况下，这不是问题，您可以轻松编写可读、精确和简洁的代码，而无需处理低层细节。但是如果您遇到性能问题(即剩余的0.01%)，分析器可能会非常方便，因为它在您或代表您的编译器正在分配某些内容的一张照片中显示或编译器代表您或代表您的编译器正在分配某些内容。分配对象需要将来的垃圾回收器周期来回收内存。

关于您的代码，您正在使用工厂模式通过DI初始化服务：该代码只运行一次。因此，您分配新对象也就不足为奇了。因此，您可以安全地隐藏这部分代码上的警告。您可以使用IDE来让生成抑制代码。这就是我建议禁用分析器并仅在查找性能问题时启用它的原因。