c# - 是否有必要使用构造函数注入(inject)检查空值？

Question

我正在使用 .NET Core 构造函数注入(inject)。 在一位同事的代码审查中，他提出了一个问题，即我是否应该检查 Controller 中注入(inject)依赖项的空值。

由于框架负责创建服务的实例，在我看来它会处理任何错误并且永远不会将空值依赖项传递给构造函数。不过我对此没有任何事实证据，所以我想知道是否有必要进行空检查。

例如，我是否应该检查以下代码中的“myService”是否为空？ (假设代码配置为使用 DI)

public class MyController
{
    private readonly IMyService _myService;

    public MyController(IMyService myService) 
    {
        _myService = myService;
    }
}

Answer 1

Is it necessary to check null values with constructor injection?

视情况而定。

• 这个内部代码是否由您和(可能)一些队友在(幸运的)代码审查环境中使用？

不要。这不是必需的。该框架不允许这样做。

• 此代码是否在公共(public)库中，由多人使用或实际上没有遵循依赖注入(inject)？

那就去做吧。手动实例化会在某处导致 NullReferenceException，而这些很难追踪。

也就是说，使用这样的东西:

public MyController(IMyService myService) 
{
    if (myService == null)
    {
        throw new ArgumentNullException(nameof(myService));
    }

    _myService = myService;
}

是一种非常便宜的检查，如果有人出于某种原因通过了 null，则更容易追踪。

---

更好的是，正如@ScottFraley 提到的，使用更新的 C# 版本，上面的代码可读性更强:

public MyController(IMyService myService) 
{
    _myService = myService ?? throw new ArgumentNullException(nameof(myService));
}