C#中代码拆分时如何消除冗余空值检查？

当你把大型方法拆分成多个小方法后，重复的空值检查确实繁琐。以下是几种比自定义VerifiedPerson更简洁的解决思路：

1. 直接传递已验证的非空属性 #

最直接的方式是在主方法里完成一次空值检查，然后把非空的Name属性直接传给子方法，而非整个Person对象。子方法参数类型设为非空的string，完全不需要再做空值判断：

private class Person
{
    public string? Name { get; set; }
}

// 拆分后的代码
private void Do(Person person)
{
    if (person.Name is null)
        return;
    
    // 直接传递已验证的非空Name
    WriteIfNameIsJohn(person.Name);
    WriteNameLength(person.Name);
}

// 子方法接收非空string，无需检查
private void WriteNameLength(string name)
{
    Console.WriteLine(name.Length);
}

private void WriteIfNameIsJohn(string name)
{
    Console.WriteLine(name.Contains("John"));
}

这个方案完全利用编译器的空值跟踪特性，没有额外类型开销，简单高效。

2. 使用NotNullWhen属性辅助编译器推断 #

如果必须传递整个Person对象，可以借助C#的[NotNullWhen]特性，让编译器识别：当某个验证方法返回true时，Person的Name属性是非空的。子方法只需调用一次验证方法，编译器就能自动推断空值状态，不用重复写if判断：

using System.Diagnostics.CodeAnalysis;

private class Person
{
    public string? Name { get; set; }
}

// 定义验证方法，用NotNullWhen标记空值推断规则
private bool HasValidName([NotNullWhen(true)] Person? person)
{
    return person?.Name is not null;
}

private void Do(Person person)
{
    if (!HasValidName(person))
        return;
    
    WriteIfNameIsJohn(person);
    WriteNameLength(person);
}

private void WriteNameLength(Person person)
{
    // 调用验证方法后，编译器自动识别person.Name非空
    if (HasValidName(person))
        Console.WriteLine(person.Name.Length);
}

private void WriteIfNameIsJohn(Person person)
{
    if (HasValidName(person))
        Console.WriteLine(person.Name.Contains("John"));
}

这种方式把空值逻辑封装成可复用的验证方法，同时让编译器正确跟踪空值状态，避免重复代码。

3. 用局部函数封装子逻辑（适合内部复用场景） #

如果拆分后的子方法只在主方法内部使用，可以用局部函数代替独立方法。局部函数能共享主方法的变量和空值状态，编译器会自动识别Name已被验证，无需重复检查：

private class Person
{
    public string? Name { get; set; }
}

private void Do(Person person)
{
    if (person.Name is null)
        return;
    
    // 局部函数直接使用已验证的person.Name
    void WriteNameLength()
    {
        Console.WriteLine(person.Name.Length);
    }

    void WriteIfNameIsJohn()
    {
        Console.WriteLine(person.Name.Contains("John"));
    }

    WriteIfNameIsJohn();
    WriteNameLength();
}

这个方案适合不需要将方法提取到外部的场景，既拆分了逻辑，又完全避免了重复检查。

4. 从根源避免空值（可选） #

如果业务场景允许，调整Person类的设计，让Name在构造时就保证非空，从根源上消除空值检查需求：

// 构造时强制传入非空Name，避免后续空值问题
private class Person
{
    public string Name { get; }

    public Person(string name)
    {
        Name = name ?? throw new ArgumentNullException(nameof(name));
    }
}

// 拆分后的方法无需任何空值检查
private void Do(Person person)
{
    WriteIfNameIsJohn(person);
    WriteNameLength(person);
}

private void WriteNameLength(Person person)
{
    Console.WriteLine(person.Name.Length);
}

private void WriteIfNameIsJohn(Person person)
{
    Console.WriteLine(person.Name.Contains("John"));
}

注意这个方案只适用于Name不允许为空的业务场景，若必须支持空Name，还是用前面几种方案更合适。

内容的提问来源于stack exchange，提问作者stone-w4tch3r