When a nullable struct, for example, int?
or Guid?
, is passed to ArgumentNullException.ThrowIfNull
, it's boxed to an object, causing a performance penalty.
For improved performance, it's better to check the HasValue
property and manually throw an exception than to pass a nullable struct to ArgumentNullException.ThrowIfNull
.
Check for null and throw the ArgumentNullException manually.
ExampleThe following code snippet shows a violation of CA1871:
static void Print(int? value)
{
ArgumentNullException.ThrowIfNull(value);
Console.WriteLine(value.Value);
}
The following code snippet fixes the violation:
static void Print(int? value)
{
if (!value.HasValue)
{
throw new ArgumentNullException(nameof(value));
}
Console.WriteLine(value.Value);
}
When to suppress warnings
It's safe to suppress this warning if performance isn't a concern.
Suppress a warningIf you just want to suppress a single violation, add preprocessor directives to your source file to disable and then re-enable the rule.
#pragma warning disable CA1871
// The code that's violating the rule is on this line.
#pragma warning restore CA1871
To disable the rule for a file, folder, or project, set its severity to none
in the configuration file.
[*.{cs,vb}]
dotnet_diagnostic.CA1871.severity = none
For more information, see How to suppress code analysis warnings.
RetroSearch is an open source project built by @garambo | Open a GitHub Issue
Search and Browse the WWW like it's 1997 | Search results from DuckDuckGo
HTML:
3.2
| Encoding:
UTF-8
| Version:
0.7.4