NotImplementedException

NotImplementedException

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








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

NotImplementedException。 你在拉我的腿是吧?

不,我不会通过说“等一下,该方法已实现 - 它抛出 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 在这个主题上甚至更弱,仅说明“未实现请求的方法或操作时抛出的异常”。

最佳答案

我觉得有一种情况很有用: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.dllMicrosoft.Cci.dll
  • 将元数据放入名为 DesignRules.xml 的文件中,并将其作为嵌入资源添加到您的程序集中
  • 将您的程序集命名为 Bevonn.CodeAnalysis 。如果要为元数据或程序集文件使用不同的名称,请确保相应地将第二个参数更改为基本构造函数。

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

    关于.net - 为什么 NotImplementedException 存在?,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/410719/

    10-13 04:25