函数作为参数传递与先赋值变量再传递的差异探讨

Hey Rotem, no need to apologize at all—this is a really common and practical question, so let’s break it down clearly!

编程逻辑与性能开销的差异 #

首先从核心层面说：在绝大多数日常开发场景下，这两种写法的逻辑和性能几乎没有差异。

• 逻辑上：两种写法都是先执行GetProducts()获取结果，再把这个结果传入GetAllData()。哪怕GetProducts()带有副作用（比如修改全局变量、输出日志），两种写法都会只执行一次GetProducts()，所以逻辑是完全等价的。
• 性能上：现代编译器（比如C#的JIT编译器）会做临时变量消除优化，第一种写法里的data变量几乎不会带来额外的性能开销，和第二种写法的执行效率基本一致。只有在极端的性能敏感场景（比如高频循环百万次以上）才可能有可观测的差异，但这种情况在业务开发中极少遇到。

两种写法的优缺点 #

写法一：var data = GetProducts(); GetAllData(data); #

• 优点：
  • 调试友好：可以在var data = ...这行加断点，直接查看data的具体内容，排查问题时非常直观，不用跳进GetProducts()内部就能确认返回值是否符合预期。
  • 可读性更强：如果GetProducts()的返回类型不直观，或者方法名不够清晰，临时变量data能帮读者快速理解这是一个中间数据，后续会被使用。
  • 可复用性：如果后续代码还要用到GetProducts()的结果，直接用data就行，避免重复调用方法（尤其是当GetProducts()是耗时操作或有副作用时）。
• 缺点：
  • 多了一行代码，略显冗余，如果data只被使用一次的话，会让代码看起来不够紧凑。

写法二：GetAllData(GetProducts()); #

• 优点：
  • 代码简洁紧凑：逻辑简单的情况下，一眼就能看出是把GetProducts()的结果直接传给GetAllData()，代码行数更少。
  • 无多余变量：不需要声明临时变量，代码更“干净”。
• 缺点：
  • 调试不便：没办法直接查看GetProducts()的返回值，要么得在GetProducts()内部加断点，要么得用调试器的即时窗口手动执行GetProducts()来查看结果，排查问题更麻烦。
  • 可读性下降：如果GetProducts()的返回类型复杂，或者方法名不够明确，读者需要先理清GetProducts()的返回值，才能理解GetAllData()接收的是什么，增加了理解成本。

更优雅/合适的实现方式 #

是否用Func<>或者其他方式，取决于你的具体场景：

1. 延迟执行场景（推荐） #

如果GetAllData()可以修改方法签名，让它接受一个Func<T>而不是直接接受T，那么可以改成延迟执行的写法：

// 先修改GetAllData的签名
void GetAllData(Func<Product[]> dataProvider)
{
    // 内部可以根据条件决定是否执行dataProvider()
    if (NeedData())
    {
        var data = dataProvider();
        // 处理数据
    }
}

// 调用时
GetAllData(() => GetProducts());

这种写法的好处是：可以避免不必要的GetProducts()调用（比如GetAllData()内部不需要数据时），节省性能开销；同时也保留了代码的紧凑性。

2. 复用调用逻辑场景 #

如果GetProducts()的调用逻辑需要多次复用（比如需要传递不同参数，或者和其他逻辑组合），可以把它封装成Func<>：

Func<Product[]> fetchProducts = () => GetProducts();

// 多次调用
var data1 = fetchProducts();
GetAllData(data1);
var data2 = fetchProducts();
ProcessOtherData(data2);

但如果只是单次调用，这种写法反而会增加冗余，没必要用。

3. 其他场景：保持简洁性优先 #

如果不需要延迟执行，也不需要复用中间结果，那么根据是否需要调试来选择即可：需要调试选写法一，追求简洁选写法二。另外，代码的可读性永远是第一位的，不用刻意追求花里胡哨的语法，适合场景的写法就是最优的。

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