C#可空性检查：模式匹配与赋值的写法验证及优化问询

1. 原代码能否改写为目标形式？ #

完全可以。

你的第二种写法利用了C#的模式匹配+变量赋值特性，逻辑和原代码完全等价：

• 当repository.Get(employeeName)返回null时，is not EmployeeData employee条件成立，直接返回false；
• 当返回非空的EmployeeData实例时，该实例会被赋值给非空类型的employee变量，后续代码可以安全使用，编译器能正确推断其可空性，不会出现空引用警告。

这段改写后的代码是合法且正确的。

2. 关于你想要的更优实现方式 #

你尝试的as EmployeeData employee is null写法不符合C#语法，所以无法编译。这里提供几种更简洁清晰的实现方案：

方案一：使用空合并运算符+返回语句（C# 9+） #

这是最简洁的写法之一，一行完成赋值和空检查：

bool F(string employeeName) {
  EmployeeData employee = repository.Get(employeeName) ?? return false;
  // 使用非空的employee执行操作
  return true;
}

?? return false会在左侧为null时直接返回false，否则将左侧的非空实例赋值给employee。

方案二：正向模式匹配 #

如果你更倾向于正向判断（先确认非空），可以把条件反过来写：

bool F(string employeeName) {
  if (repository.Get(employeeName) is EmployeeData employee) {
    // 使用非空的employee执行操作
    return true;
  }
  return false;
}

这种写法逻辑直观，和原代码的判断顺序一致，同样能让编译器正确推断employee的非空性。

方案三：保留"判空返回"的直白逻辑 #

如果你一定要坚持"如果是null就返回false"的写法，除了你提到的is not版本，原代码本身就是逻辑直白的选择：

bool F(string employeeName) {
  var employee = repository.Get(employeeName);
  if (employee is null) return false;
  // 使用非空的employee执行操作
  return true;
}

优点是逻辑清晰，对新手友好。

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