验证C＃基类构造函数参数

Question

在VS2010测试版中运行代码分析（以前版本的FxCop）后，我收到以下警告：

在外部可见方法'Identity.Identity（WindowsIdentity）'中，在使用之前验证参数'windowsIdentity'。

构造函数是：

public Identity(WindowsIdentity windowsIdentity)
         : base(windowsIdentity.Token)
{
         init();
}

对于定义为的类：

public class Identity : WindowsIdentity

我的问题是，如何验证windowsIdentity参数？ 我应该在构造函数中验证它，并抛出异常，还是有更好的方法来调用它？

Answer 1

您可以使用静态方法验证它：

public Identity(WindowsIdentity windowsIdentity)
         : base(GetToken(windowsIdentity))
{
         init();
}

static Token GetToken(WindowsIdentity ident)
{
    if(ident == null)
        throw new ArgumentNullException("ident");

    return ident.Token;
}

（我没有费心去寻找WindowsIdentity.Token的类型，但是你明白了）

Answer 2

我相信FXCop在这里报告此错误是因为它认为在调用基类构造函数时访问windowsIdentity会遇到NullReferenceException。

为null添加验证检查的一种方法是向您的类添加一个静态私有函数，该函数可以检查WindowsIdentity参数是否为null并采取适当的操作：

private static WindowsIdentity ValidateIdentity( WindowsIdentity identity )
{
    if( identity == null )
        throw new ArgumentNullException( "identity" );
    // possibly some other validation checks here...

    return identity;        
}

public Identity(WindowsIdentity windowsIdentity)
    : base( ValidateIdentity( windowsIdentity ).Token )
{
     init();
}

另一种方法是使用三元运算符来验证参数，如：

public Identity(WindowsIdentity windowsIdentity)
    : base( windowsIdentity == null ? null : windowsIdentity.Token )
{
     init();
}

但是，你真正应该问自己的是你会做什么？ 如果您只是要抛出异常，那么让代码保持原样可能是正确的，因为如果参数为null，它将已经通过NullReferenceException 。

Answer 3

它抱怨，因为如果你将NULL作为windowsIdentity传递，那么当构造函数链接到基类时，它将抛出一个空引用异常。

处理它的最佳方式取决于您的设计。 您可以像这样检查null：

:base(windowsIdentity == null ? null : windowsIdentity.Token)

或者您可以在基类构造函数中创建另一个构造函数，该构造函数将WindowsIdentity作为参数，并让该构造函数执行验证的那一部分。 基本上有很多方法可以解决它，只需使用在你的情况下最有效的方法。

Answer 4

从C＃6.0开始，您可以使用null-coalescing运算符与null-conditional运算 符结合使用，如下所示：

public Identity(WindowsIdentity winIdentity)
    : base(winIdentity?.Token ?? throw new ArgumentNullException(nameof(winIdentity)))
{
    init();
}

Answer 5

FX警告告诉你参数不能为空，所以如果你真的需要它，你应该以某种方式验证它。 因为你在构造函数中使用它，你可能想要一个不同于null的值，所以你应该在那里验证它，因为FX警察会让你烦恼。

如果你需要一个带null的构造函数，你应该有另一个没有参数的构造函数。

如果您没有使用它或者您在另一个点验证它，则可以跳过警报。

为了避免FXcop的问题，你应该抛出ArgumentNullException。