修改Action委托表达式报错：匿名函数无法返回值求解决方案

错误原因 #

报错Anonymous function converted to a void returning delegate cannot return a value的核心原因是：Action<T>是无返回值委托，你在try块中用return methodCall.Compile().Invoke(m);试图返回一个void类型的结果，但匿名函数被转换成void返回的委托，不允许使用return语句返回值。

修复方案 #

方案1：快速修复编译错误（直接执行委托）

如果不需要保留表达式树的结构，只是想要编译通过并执行监控逻辑，只需去掉return，直接调用原方法即可：

private Expression<Action<T>> GenerateMethodCallWithMonitoring<T>(string recurringJobId, Expression<Action<T>> methodCall)
{
    return m =>
    {
        var transaction = Agent.Tracer.StartTransaction(recurringJobId, ApiConstants.ActionExec);
        try
        {
            methodCall.Compile().Invoke(m); // 移除return，直接调用无返回值方法
        }
        finally
        {
            transaction?.End();
        }
    };
}

方案2：构建完整表达式树（保留表达式结构）

如果需要后续对表达式树进行解析、转换等操作，不能提前编译委托，需要手动构建表达式节点来组合监控逻辑和原方法调用：

private Expression<Action<T>> GenerateMethodCallWithMonitoring<T>(string recurringJobId, Expression<Action<T>> methodCall)
{
    // 获取原表达式的参数m
    var parameter = methodCall.Parameters[0];
    
    // 声明transaction变量
    var transactionVar = Expression.Variable(typeof(ITransaction), "transaction");
    
    // 构建启动事务的表达式
    var startTransactionExpr = Expression.Assign(
        transactionVar,
        Expression.Call(
            typeof(Agent).GetProperty(nameof(Agent.Tracer)).GetGetMethod(),
            nameof(ITracer.StartTransaction),
            Type.EmptyTypes,
            Expression.Constant(recurringJobId),
            Expression.Constant(ApiConstants.ActionExec)
        )
    );
    
    // 构建原方法调用的表达式
    var invokeMethodExpr = Expression.Invoke(methodCall, parameter);
    
    // 构建事务结束的表达式（包含null检查）
    var endTransactionExpr = Expression.Call(transactionVar, nameof(ITransaction.End));
    var safeEndTransactionExpr = Expression.Condition(
        Expression.NotEqual(transactionVar, Expression.Constant(null)),
        endTransactionExpr,
        Expression.Empty()
    );
    
    // 构建try-finally块
    var tryFinallyBlock = Expression.TryFinally(
        Expression.Block(invokeMethodExpr),
        Expression.Block(safeEndTransactionExpr)
    );
    
    // 组合所有表达式到一个块中
    var finalBlock = Expression.Block(
        new[] { transactionVar },
        startTransactionExpr,
        tryFinallyBlock
    );
    
    // 返回最终的Lambda表达式
    return Expression.Lambda<Action<T>>(finalBlock, parameter);
}

这种方式不会提前编译原方法委托，而是将监控逻辑和原方法调用都作为表达式节点组合在一起，适合需要保持表达式树完整性的场景。

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