IAsyncEnumerable扩展方法返回类型加async是否有价值及写法对比

1. 执行逻辑的触发时机区别 #

最本质的差异是参数校验与前置逻辑的执行时机：

• 第一种（本地函数拆分写法）：外层方法是普通同步方法，你调用Concat的瞬间，外层的所有逻辑（包括参数校验）都会立刻执行，只有返回的Core本地函数内部的迭代逻辑会延迟到首次枚举时触发。
  你可以直接在外层方法中添加参数校验，一旦传入null等非法参数，调用方法时立刻抛出异常，符合常规LINQ方法的行为预期，不会把错误延迟到枚举环节才暴露，大幅降低调试成本。
  示例参数校验写法：

  public static IAsyncEnumerable<TSource> Concat<TSource>(this IAsyncEnumerable<TSource> source, IAsyncEnumerable<TSource> sequence) {
      // 调用时立刻校验
      if (source == null) throw new ArgumentNullException(nameof(source));
      if (sequence == null) throw new ArgumentNullException(nameof(sequence));
      return Core(source, sequence);
      static async IAsyncEnumerable<TSource> Core(IAsyncEnumerable<TSource> source, IAsyncEnumerable<TSource> sequence) {
          await foreach (var item in source) yield return item;
          await foreach (var item in sequence) yield return item;
      }
  }
  

• 第二种（直接带async的写法）：整个方法被async修饰，又包含yield return，会被编译器整体编译为延迟执行的状态机，所有逻辑（包括你写在方法开头的参数校验）都会延迟到第一次await foreach枚举的时候才执行。如果传入非法参数，调用方法时不会有任何报错，只有真正用到序列的时候才会抛出异常，非常容易把问题埋到业务运行阶段才暴露。

2. 后续迁移成本差异 #

你目标是后续切换到System.Linq.Async，第一种写法的签名和官方库完全一致，后续只要删除你自行实现的扩展方法即可，业务侧的调用代码不需要做任何修改。而第二种写法加了Async后缀，后续迁移时需要批量修改所有调用处的方法名，额外增加了迁移成本。

C#9 版本的选型建议 #

直接选择第一种拆分本地函数的写法即可：

• C#9 完全支持静态本地函数语法，没有兼容性问题
• 符合官方System.Linq.Async的实现规范，迁移成本最低
• 即时参数校验的行为符合开发者对LINQ方法的普遍预期，避免隐性BUG

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