用于规避三元运算符的NotNull泛型扩展方法实现咨询

首先，你的这个NotNull扩展方法思路挺不错的——用扩展方法封装重复的空值判断逻辑，替代零散的三元运算符，确实能提升代码的可读性和复用性。不过从实现细节和实际使用场景来看，这里还有一些可以优化的点，以及需要注意的潜在问题：

一、潜在问题 #

• 值类型默认值的歧义风险：当Tresult是值类型（比如int、DateTime）时，default(Tresult)会返回该值类型的默认值（比如0、DateTime.MinValue）。但很多业务场景中，这些默认值本身可能是合法的业务值，这时候调用者就无法区分返回结果是“对象为空时的默认标识”还是“真实的业务值”，容易引发逻辑错误。
• 对值类型Tin的无效支持：当前方法没有泛型约束，若Tin是值类型，a != null的判断永远为true（因为值类型不能为null），这时候这个方法对值类型的调用完全是多余的，还会造成不必要的代码执行。
• 委托为空的未处理风险：如果传入的whenNotNull委托是null，调用whenNotNull(a)会直接抛出NullReferenceException，当前代码没有做任何防御性处理，会导致意外的运行时错误。

二、优化方向 #

1. 增加泛型约束，明确适用场景 #

给Tin加上class约束，限定方法只作用于引用类型（毕竟值类型不需要空判断），避免无效调用：

public static class NotNullExtensionMethod {
    public static Tresult? NotNull<Tin, Tresult>(this Tin? a, Func<Tin, Tresult> whenNotNull) 
        where Tin : class 
        where Tresult : notnull {
        if (whenNotNull == null)
            throw new ArgumentNullException(nameof(whenNotNull), "执行委托不能为空");
            
        return a != null ? whenNotNull(a) : default;
    }
}

这里还利用了C# 8+的可空引用类型语法，让a的可空语义更明确，同时约束Tresult为非空类型，返回Tresult?也更符合“可能为空”的逻辑。

2. 处理委托为空的情况 #

在方法开头先判断whenNotNull是否为null，抛出更明确的ArgumentNullException，比直接抛出NullReferenceException更易于调试和定位问题。

3. 提供值类型的专属重载（可选） #

如果需要支持值类型的空判断（比如可空值类型int?），可以单独加一个重载：

public static Tresult? NotNull<Tin, Tresult>(this Tin? a, Func<Tin, Tresult> whenNotNull) 
    where Tin : struct 
    where Tresult : notnull {
    if (whenNotNull == null)
        throw new ArgumentNullException(nameof(whenNotNull));
        
    return a.HasValue ? whenNotNull(a.Value) : default;
}

这样可空值类型的空判断逻辑会更准确。

4. 优化方法命名，提升语义清晰度 #

NotNull这个名字其实有点歧义，它既可以理解为“确保对象非空”，也可以理解为“当对象非空时执行逻辑”。改成WhenNotNull或者IfNotNull会更直观，让调用者一眼就能明白方法的作用：

foo.OptionalField.WhenNotNull(r => r.SomeStringField);

5. 添加无返回值的重载（可选） #

如果有些场景只是想在对象非空时执行操作，不需要返回值，可以增加一个Action版本的重载：

public static void WhenNotNull<Tin>(this Tin? a, Action<Tin> whenNotNull) where Tin : class {
    if (a != null && whenNotNull != null)
        whenNotNull(a);
}

用法示例：foo.OptionalField.WhenNotNull(r => r.DoSomething());

总结 #

如果你的主要使用场景是处理引用类型的空值，并且项目基于C# 8及以上版本，优化后的版本会更健壮、语义更清晰，也能避免不少潜在的运行时问题。如果需要兼容旧版本C#，可以去掉可空引用类型的语法，保留泛型约束和委托空判断的逻辑。

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