非null对象调用ToString()会返回null吗？VS报CS8600警告求解

问题代码 #

string returnValue;
var registryValue = key.GetValue("something") ?? throw new InvalidOperationException();
returnValue = registryValue.ToString();

VS提示CS8600警告：“Converting null literal or possible null value to non-nullable type.”，疑问点如下：

1. 是否存在非null对象调用object.ToString()返回null的情况？
2. 这是否是IntelliSense未识别到此处不会出现null？
3. 能否安全使用returnValue = registryValue.ToString()!;，还是必须处理可能的null？

解答 #

1. 非null对象调用object.ToString()可能返回null吗？ #

• .NET基类object的默认ToString()实现返回类型的完全限定名，不会返回null。但任何子类都可以重写ToString()方法，故意返回null。例如：

  public class MaliciousClass
  {
      public override string ToString()
      {
          return null;
      }
  }
  

  如果key.GetValue("something")返回该类的实例，即使对象本身非null，ToString()也会返回null。不过这种情况在正规代码中非常罕见。

2. 为什么会触发CS8600警告？ #

• 编译时registryValue的类型是object，C#的可空分析系统仅基于编译时类型判断。虽然你通过?? throw确保了registryValue非null，但系统无法预知该对象的实际类型是否重写了ToString()并返回null。因此系统将ToString()的返回值判定为string?（可空字符串），赋值给非空的string returnValue时就触发了警告。

3. 能否安全使用!运算符，还是必须处理null？ #

• 分两种情况判断：
  • 若你确定key.GetValue("something")返回的对象（比如.NET内置类型：string、int、DateTime等）的ToString()绝对不会返回null，那么使用returnValue = registryValue.ToString()!;是安全的，这相当于告诉编译器“我确认此处不会为null”。
  • 若不确定实际类型，或担心存在自定义类型恶意返回null，建议显式处理null，例如：

    returnValue = registryValue.ToString() ?? string.Empty;
    

    或根据业务逻辑做其他null处理。

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