抑制或避免警告 CA2214

Question

我有一种情况，我根据收到的数据创建了一个名为 EntryEvent 的对象。必须解析该数据。基本事件应该开始解析通过构造函数接收并提供给对象的数据。子类型知道如何解析该特定数据集。现在，在编译所述代码时，我收到警告 CA2214，它包含对虚拟方法的调用链。虽然产生不可预见的后果可能很糟糕，但我不知道如何获得所需的行为：解析接收到的事件，而无需从外部调用额外的“解析”方法。

有问题的代码是：

public abstract class BaseEvent
{
    protected BaseEvent(object stuff)
    {
        this.ParseEvent();
    }

    protected abstract void ParseEvent();
}

public class EntryEvent : BaseEvent
{
    public EntryEvent( object stuff )
        : base( stuff )
    {
    }

    protected override void ParseEvent()
    {
        // Parse event
    }
}

Answer 1

根据 MSDN（重点是我的）：

调用虚拟方法时，直到运行时才会选择执行该方法的实际类型。当构造函数调用虚方法时，可能调用该方法的实例的构造函数没有执行。

所以在我看来，你有这些选择（至少）：

1) 不要禁用该警告，而是为您的特定类抑制该消息，以记录其预期行为（假设您特别注意处理这种情况）。如果它仅限于在一个非常受控的环境中的少数类，那还不错（毕竟......警告不是错误，它们可能会被忽略）。

2) 从基类构造函数中删除该虚拟方法调用，但保留abstract 方法声明。开发人员必须实现这样的方法并在构造函数中调用它，他们需要将他们的类标记为sealed。最后在类/方法文档中的某处添加该方法必须在其构造函数中调用，并且它们的类必须是 sealed 才能这样做。 他们可以忘记该调用，但您可以添加（对于DEBUG 构建）访问属性或方法时的检查（例如，作为类接口的一部分，强制设置特定标志）。如果他们忘记设置标志或忘记调用方法，则会引发异常（“此对象尚未构建，必须在派生类构造函数中调用 ParseEvent()。”）。

我不太喜欢这种方法，因为它增加了额外的复杂性，但是如果你的类层次结构太大（然后你觉得你不能使用#1）或延迟初始化（在#3中描述）不适用那么它可能是一个工作的解决方案。我还考虑更改设计以引入一个工厂方法，该方法将为每个完全构造的对象调用 ParseEvent()。

3) 稍微改变您的设计：将解析推迟到需要的时候。例如：

public abstract class BaseEvent
{
    public DateTime TimeStamp
    {
        get
        {
            if (_timestamp == null)
                ParseEvent();

            return _timestamp.Value;
        }
        protected set { _timestamp = value; }
    }

    protected BaseEvent(object stuff)
    {
    }

    protected abstract void ParseEvent();

    private DateTime? _timestamp;
}

最后一个示例仅用于说明目的，您可能希望使用Lazy<T> 以更简洁、清晰和线程安全的方式执行相同的任务。当然实际上你会有更多的字段/属性，并且可能解析会一次性提供所有值（然后你只需要一个标志，不需要每个字段上的Nullable/特殊值）这是我更喜欢的方法即使它更冗长。