扩展方法中的 ArgumentNullException 或 NullReferenceException？

Question

在空实例上调用扩展方法时（扩展方法不允许），您认为抛出的最佳异常类型是什么？由于扩展方法只不过是静态方法，您可能会认为它应该是 ArgumentNullException，但另一方面，它们的使用方式与实例方法类似，因此使用 NullReferenceException 可能更自然。让我们看下面的例子：

public static string ToInvariantString(this IFormattable value, string format)
{
    return value.ToString(format, CultureInfo.InvariantCulture);
}

这样如果value参数为null，就会抛出NullReferenceException。

另一个例子是：

public static string ToInvariantString(this IFormattable value, string format)
{
    if (value == null) throw new ArgumentNullException("value");
    return value.ToString(format, CultureInfo.InvariantCulture);
}

编辑： 在一些答案中，您指出扩展方法可以像静态方法一样被调用，在这些情况下，空引用异常将是错误的，这是一个很好的观点，实际上也是我关心的问题之一，不知道为什么我忘记了首先在问题中提到这一点。

也有人指出抛出 NullReferenceException 是错误的，是的，确实如此。这就是为什么我不抛出它，我只是通过不保护该方法让它发生（让 CLR 抛出它）。

我认为我赞成 ArgumentNullException （这就是我到目前为止所使用的），但我仍然认为至少有空间反对 NullReferenceException，因为在大多数要使用该方法的地方它似乎更自然。

Answer 1

一般来说，包括例外情况，您应该将扩展方法视为普通的静态方法。在这种情况下，您应该抛出 ArgumentNullException。

出于多种原因，在这里抛出 NullReferenceException 是一个坏主意

• 空引用实际上并未发生，因此看到一个 NullReferenceException 是违反直觉的
• 抛出 NullReferenceException 并导致 NullReferenceException 发生会产生明显不同的异常（查看差异的一种方法是错误代码）。 CLR 引发的许多异常都是如此。

请参阅何时可以捕获 StackOverflowException （我就这个主题发表过一篇文章）。

• 调用扩展方法就像调用常规方法一样是完全合法的。在这种情况下，我当然不会排除 NullReferenceException，而是排除 ArgumentNullException。

Answer 2

除了所有其他答案（都很好）之外，我认为为了一致性，值得看看微软所做的事情......并且据我所知，Enumerable 中的扩展方法都抛出 ArgumentNullException 。

Answer 3

由于扩展方法可以在 C# 2.0 中使用，并且可以像静态方法一样调用它们（您不必将它们用作扩展方法），因此您应该使用 ArgumentNullException。

仅仅因为它们看起来类似于类型上的方法，并不意味着它们就是这样，或者总是像这样被调用。

Answer 4

从用户的角度来看，该方法的外观和行为都类似于实例方法，因此如果我是他们，我会期望看到 NullReferenceException。

也就是说，我建议在代码中显式地抛出一个或另一个，而不是像第一个示例中那样“碰巧”抛出一个。

Answer 5

参数空异常。不需要像调用实例方法一样调用扩展方法。您可以像调用普通方法一样调用它们。在这种情况下，NullReferenceException 将是完全错误的。

Answer 6

更让人困惑的是，微软同时抛出了 ArgumentNullExceptions 和 NullReferenceExceptions。

这个抛出隐式 NullReferenceException 的示例来自 Roslyn (src\Workspaces\CSharp\Portable\Extensions\StringExtensions.cs)：

internal static class StringExtensions
{
    public static string EscapeIdentifier(
        this string identifier,
        bool isQueryContext = false)
    {
        var nullIndex = identifier.IndexOf('\0');
        if (nullIndex >= 0)
        {
            identifier = identifier.Substring(0, nullIndex);
        }

        var needsEscaping = SyntaxFacts.GetKeywordKind(identifier) != SyntaxKind.None;

        // Check if we need to escape this contextual keyword
        needsEscaping = needsEscaping || (isQueryContext && SyntaxFacts.IsQueryContextualKeyword(SyntaxFacts.GetContextualKeywordKind(identifier)));

        return needsEscaping ? "@" + identifier : identifier;
    }

此扩展方法从 . NET Framework (System.Core/System/Linq/Enumerable.cs)：

public static IEnumerable<TResult> Select<TSource, TResult>(this IEnumerable<TSource> source, Func<TSource, TResult> selector) {
            if (source == null) throw Error.ArgumentNull("source");
            if (selector == null) throw Error.ArgumentNull("selector");
            if (source is Iterator<TSource>) return ((Iterator<TSource>)source).Select(selector);
            if (source is TSource[]) return new WhereSelectArrayIterator<TSource, TResult>((TSource[])source, null, selector);
            if (source is List<TSource>) return new WhereSelectListIterator<TSource, TResult>((List<TSource>)source, null, selector);
            return new WhereSelectEnumerableIterator<TSource, TResult>(source, null, selector);


   }

正如上面的评论中提到的，我建议将扩展方法完全作为实例方法来实现，因此如果 NullReferenceException >此-参数为null。如果有人不恰当地调用我的扩展方法，他们可以随意这样做，但也必须预料到不恰当的行为（NullReferenceException 而不是 ArgumentNullException）。但是，如果他们按预期调用该方法，他们也应该获得预期的行为：完全一致的体验。

//Instance method
string foo = null;
foo.Trim();

它们

//Extension method
string foo = null;
foo.Right(10); 

看起来很相似，行为也应该相似，程序员甚至不需要知道它是实例还是扩展方法。