Проверка параметра конструктора базового класса C#

Я полагаю, что FXCop сообщает об этой ошибке, потому что считает, что вы можете столкнуться с исключением NullReferenceException, открыв windowsIdentity при вызове конструктор базового класса.

Одним из способов добавления проверки на 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();
}

Но на самом деле вы должны спросить себя, что бы вы сделали? Если вы просто собираетесь сгенерировать исключение, можно оставить код как есть, поскольку он уже пройдет через 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();
}

Но что вы действительно должны спросить себя, так это что бы вы сделали? Если вы просто собираетесь сгенерировать исключение, можно оставить код как есть, поскольку он уже пройдет через 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();
}

Но что вы действительно должны спросить себя, так это что бы вы сделали? Если вы просто собираетесь сгенерировать исключение, можно оставить код как есть, поскольку он уже пройдет через NullReferenceException , если аргумент равен нулю.

Он жалуется, потому что, если вы передадите NULL как windowsIdentity, тогда, когда конструктор будет связываться с базовым классом, он выдаст исключение с нулевой ссылкой.

Лучший способ справиться с этим зависит от вашего дизайн. Вы можете проверить его на null следующим образом:

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

Или вы можете создать другой конструктор в конструкторе базового класса, который принимает WindowsIdentity в качестве параметра, и пусть этот конструктор выполняет эту часть проверки. В общем, есть множество способов справиться с этим, просто используйте то, что лучше всего работает в вашей ситуации.

FX-полицейский сообщает вам, что параметр не может быть нулевым, поэтому, если он вам действительно нужен, вы должны как-то его проверить. Поскольку вы используете его в конструкторе, вам, вероятно, нужно значение, отличное от null, поэтому вам следует проверить его там, чтобы FX-полицейский перестал вас раздражать.

Если вам нужен конструктор с null, у вас должен быть другой конструктор без параметров.

Если вы не используете его или проверяете в другой точке, вы можете пропустить предупреждение.

Чтобы избежать проблемы с FXcop, вы должны выбросить ArgumentNullException.