返回null的IEnumerable函数引发参数空异常，需提前判空吗？

首先明确说：这绝对不是预期的正常情况。在.NET的设计惯例里，返回IEnumerable<T>这类集合类型的方法，标准做法是当没有数据时返回空集合（比如Enumerable.Empty<TaskNote>()），而不是null。这样调用方可以放心地链式调用ToList()、Where()等LINQ方法，不用额外处理null的情况。

你遇到的System.ArgumentNullException，就是因为ToList()方法的source参数不允许为null——当GetAll返回null时，直接调用ToList()就会触发这个异常。

关于是否必须在调用ToList()前检查null：

• 如果GetAll的实现没办法修改（比如是第三方库代码），那必须手动检查null，避免异常。可以用空合并运算符简化代码：

  IEnumerable<TaskNote> notesEnumerable = noteManager.GetAll(newTaskId);
  List<TaskNote> notes = notesEnumerable?.ToList() ?? new List<TaskNote>();
  

• 如果能修改GetAll方法，更推荐的做法是从根源解决问题，让它返回空集合而非null。比如修改方法逻辑：

  public IEnumerable<TaskNote> GetAll(int taskId)
  {
      // 你的数据查询逻辑
      var queryResult = /* 从数据库/存储中获取数据的代码 */;
      // 如果查询结果为null，返回空集合
      return queryResult ?? Enumerable.Empty<TaskNote>();
  }
  

这样后续所有调用GetAll的地方都不用再处理null，代码更简洁也更符合.NET的设计规范。

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