C# .NET Framework 4.8中冗余字符串求值的性能优化问询

你的性能损耗根源在于C#的传值求值语义：方法调用时，所有参数表达式会在进入方法体前完全计算完成，哪怕方法内部根本不会使用这个参数。对于Logger.MenuWrite($"Passed: {Cmd.Observable} and {Cmd.Dist}")这类调用，只要你写了这个代码，不管Logger.MenuItems是true还是false，字符串插值的逻辑（包括访问属性、拼接字符串、分配内存）都会先执行，这就是开销的来源。

至于你尝试的AggressiveInlining效果有限，是因为.NET Framework 4.8的RyuJIT编译器对条件分支下的无用代码优化能力不足：即使Logger的方法被内联，JIT也很难判断字符串插值操作没有副作用（比如属性getter可能隐含逻辑），因此不会主动消除这个求值过程。委托方案性能更差，大概率是因为你用的Action<string>仍然需要先求值字符串再传递，反而多了一层委托调用的开销。

1. 延迟求值：改用Func<string>作为参数 #

这是最适合运行时动态开关的方案，核心是把字符串的生成逻辑延迟到日志开关打开时才执行：

public static class Logger
{
    public static bool MenuItems { get; set; }

    // 改为接受Func<string>，仅当开关开启时才执行字符串生成
    public static void MenuWrite(Func<string> msgFactory)
    {
        if (MenuItems)
        {
            string msg = msgFactory.Invoke();
            // 执行日志记录逻辑
        }
    }
}

调用方式改为：

Logger.MenuWrite(() => $"Passed: {Cmd.Observable} and {Cmd.Dist}");

当MenuItems为false时，Lambda表达式不会被执行，完全避免了字符串插值的开销，仅存在极微小的委托引用传递成本，规模扩大后优势会非常明显。

2. 编译时开关：预处理器指令 #

如果日志开关是编译阶段确定的（比如仅Debug模式开启），可以用预处理器指令直接在编译时移除无用的日志调用：

// 在项目属性中定义LOG_MENU_ITEMS常量（Debug模式下）
#if LOG_MENU_ITEMS
Logger.MenuWrite($"Passed: {Cmd.Observable} and {Cmd.Dist}");
#endif

这种方式在开关关闭时完全没有运行时开销，但缺点是开关无法在运行时动态调整。

3. 封装条件判断逻辑 #

如果不想重复写if (Logger.MenuItems)，可以封装一个静态工具方法统一处理：

public static class LoggerUtils
{
    public static void LogMenu(Func<string> msgFactory)
    {
        if (Logger.MenuItems)
        {
            Logger.MenuWrite(msgFactory.Invoke());
        }
    }
}

调用时直接用LoggerUtils.LogMenu(() => $"xxx")，既避免了手动内联的代码冗余，又保留了延迟求值的性能优势。

• 缓存静态开关字段：如果日志开关不会频繁变化，可以将静态字段的值缓存到局部变量，减少静态字段访问的线程同步开销：

  bool isMenuLogEnabled = Logger.MenuItems;
  if (isMenuLogEnabled)
  {
      Logger.MenuWrite($"xxx");
  }
  

• 避免不必要的装箱：如果字符串插值中涉及值类型（比如int、DateTime），可以考虑直接调用ToString()指定格式，减少隐式装箱的开销。

内容的提问来源于stack exchange，提问作者tanr