.net - 为什么 NotImplementedException 存在？

Question

锁定。这个问题及其答案是locked因为这个问题是题外话，但具有历史意义。它目前不接受新的答案或互动。








这真的，真的让我感到不安，所以我希望有人能给我一个合理的理由来解释为什么事情是这样。

未实现异常。 你在拉我的腿，对吧？

不，我不会通过说“等一下，该方法已实现 - 它抛出 NotImplementedException”来对此进行廉价攻击。是的，没错，您必须实现抛出 NotImplementedException 的方法(与 C++ 中的纯虚函数调用不同 - 现在说得通了!)。虽然这很可笑，但在我的脑海中还有一个更严重的问题。

我只是想知道，在 NotImplementedException 的存在下，任何人怎么能用 .Net 做任何事情？您是否希望用 try catch 块包装每个抽象方法调用以防止可能未实现的方法？如果你捕捉到这样的异常，你到底该怎么处理它？？

我认为没有办法测试一个方法是否在不调用它的情况下实际实现。由于调用它可能会产生副作用，因此我无法预先进行所有检查，然后再运行我的算法。我必须运行我的算法，捕获 NotImplementedExceptions 以及一些如何将我的应用程序回滚到某种正常状态的方法。

这很疯狂。疯狂的。疯狂的。所以问题是:为什么存在 NotImplementedException ?

作为先发制人的打击，我不希望任何人回应，“因为设计师需要将其放入自动生成的代码中。”这太可怕了。在您提供实现之前，我宁愿不编译自动生成的代码。例如，自动生成的实现可以是“throw NotImplementedException;”其中 NotImplementedException 未定义!

有没有人曾经捕获并处理过 NotImplementedException？你有没有在你的代码中留下 NotImplementedException ？如果是这样，这是否代表一个定时炸弹(即，您不小心将其留在那里)，或设计缺陷(该方法不应该被实现并且永远不会被调用)？

我也非常怀疑 NotSupportedException ......不支持？什么？如果它不受支持，为什么它是您界面的一部分？ Microsoft 的任何人都可以拼写不正确的继承吗？但是，如果我不会因为这个问题受到太多虐待，我可能会为此提出另一个问题。

附加信息:

This是关于这个主题的有趣读物。

似乎与 Brad Abrams 有很强的一致性“NotImplementedException 是针对尚未实现的功能，但确实应该(并且将会实现)。就像您在构建类时可能开始使用的功能一样，在那里获取所有抛出 NotImplementedException 的方法，然后用真实的方法将它们清除代码…”

来自 Jared Parsons 的评论非常弱，可能应该被忽略: NotImplementedException:当一个类型由于任何其他原因没有实现方法时抛出这个异常。

MSDN在这个主题上甚至更弱，只是说明“未实现请求的方法或操作时抛出的异常。”

Answer 1

我觉得有一种情况很有用:TDD。

我编写我的测试，然后我创建 stub 以便测试编译。这些 stub 除了 throw new NotImplementedException(); 什么也不做.这样，无论如何，默认情况下测试都会失败。如果我使用了一些虚拟返回值，它可能会产生误报。现在所有测试都编译并失败，因为没有实现，我处理这些 stub 。

因为我从不使用 NotImplementedException在任何其他情况下，否 NotImplementedException将永远传递给发布代码，因为它总是会使某些测试失败。

你不需要到处捕获它。好的 API 会记录抛出的异常。这些是你应该寻找的。

编辑:我写了一个 FxCop 规则来找到它们。

这是代码:

using System;
using Microsoft.FxCop.Sdk;

/// <summary>
/// An FxCop rule to ensure no <see cref="NotImplementedException"/> is
/// left behind on production code.
/// </summary>
internal class DoNotRaiseNotImplementedException : BaseIntrospectionRule
{
    private TypeNode _notImplementedException;
    private Member _currentMember;

    public DoNotRaiseNotImplementedException()
        : base("DoNotRaiseNotImplementedException",
               // The following string must be the assembly name (here
               // Bevonn.CodeAnalysis) followed by a dot and then the
               // metadata file name without the xml extension (here
               // DesignRules). See the note at the end for more details.
               "Bevonn.CodeAnalysis.DesignRules",
               typeof (DoNotRaiseNotImplementedException).Assembly) { }

    public override void BeforeAnalysis()
    {
        base.BeforeAnalysis();
        _notImplementedException = FrameworkAssemblies.Mscorlib.GetType(
            Identifier.For("System"),
            Identifier.For("NotImplementedException"));
    }

    public override ProblemCollection Check(Member member)
    {
        var method = member as Method;
        if (method != null)
        {
            _currentMember = member;
            VisitStatements(method.Body.Statements);
        }
        return Problems;
    }

    public override void VisitThrow(ThrowNode throwInstruction)
    {
        if (throwInstruction.Expression != null &&
            throwInstruction.Expression.Type.IsAssignableTo(_notImplementedException))
        {
            var problem = new Problem(
                GetResolution(),
                throwInstruction.SourceContext,
                _currentMember.Name.Name);
            Problems.Add(problem);
        }
    }
}

这是规则元数据:

<?xml version="1.0" encoding="utf-8" ?>
<Rules FriendlyName="Bevonn Design Rules">
  <Rule TypeName="DoNotRaiseNotImplementedException" Category="Bevonn.Design" CheckId="BCA0001">
    <Name>Do not raise NotImplementedException</Name>
    <Description>NotImplementedException should not be used in production code.</Description>
    <Url>http://stackoverflow.com/questions/410719/notimplementedexception-are-they-kidding-me</Url>
    <Resolution>Implement the method or property accessor.</Resolution>
    <MessageLevel Certainty="100">CriticalError</MessageLevel>
    <Email></Email>
    <FixCategories>NonBreaking</FixCategories>
    <Owner></Owner>
  </Rule>
</Rules>

要构建它，您需要:

引用 Microsoft.FxCop.Sdk.dll和 Microsoft.Cci.dll

将元数据放在名为 DesignRules.xml 的文件中并将其作为嵌入式资源添加到您的程序集中

为您的程序集命名 Bevonn.CodeAnalysis .如果要为元数据或程序集文件使用不同的名称，请确保相应地将第二个参数更改为基本构造函数。

然后只需将生成的程序集添加到您的 FxCop 规则中，并从您宝贵的代码中删除那些该死的异常。在某些极端情况下，当抛出一个 NotImplementedException 时它不会报告 NotImplementedException 但我真的认为如果你真的在写这样的克苏尔代码，你是绝望的。对于正常使用，即 throw new NotImplementedException(); ，它有效，这才是最重要的。