StyleCop问题需要解析 - 包含一个调用链,该调用链导致调用由类定义的虚方法

时间:2013-10-25 09:32:50

标签: c# stylecop

我一直用StyleCop清理我的项目文件,我有以下无法解决的问题

PasswordPolicyCollection.cs(13):CA2214:Microsoft.Usage:'PasswordPolicyCollection.PasswordPolicyCollection()'包含一个调用链,该调用链导致调用该类定义的虚方法。检查以下调用堆栈是否存在意外后果:PasswordPolicyCollection..ctor()ConfigurationElementCollection.CreateNewElement():ConfigurationElement PasswordPolicyCollection.Add(PasswordPolicy):Void ConfigurationElementCollection.BaseAdd(ConfigurationElement):Void

以下是代码:

using System.Configuration;

public class PasswordPolicyCollection : ConfigurationElementCollection
{
    public PasswordPolicyCollection()
    {
        var passwordPolicy = (PasswordPolicy)this.CreateNewElement();
        this.Add(passwordPolicy);
    }

    public override ConfigurationElementCollectionType CollectionType
    {
        get
        {
            return ConfigurationElementCollectionType.AddRemoveClearMap;
        }
    }

    public new PasswordPolicy this[string name]
    {
        get
        {
            return (PasswordPolicy)BaseGet(name);
        }
    }

    public PasswordPolicy this[int index]
    {
        get
        {
            return (PasswordPolicy)BaseGet(index);
        }

        set
        {
            if (BaseGet(index) != null)
            {
                BaseRemoveAt(index);
            }

            BaseAdd(index, value);
        }
    }

    public void Add(PasswordPolicy passwordPolicy)
    {
        this.BaseAdd(passwordPolicy);
    }

    public int Indexof(PasswordPolicy policy)
    {
        return BaseIndexOf(policy);
    }

    public void Remove(PasswordPolicy url)
    {
        if (BaseIndexOf(url) >= 0)
            BaseRemove(url.Name);
    }

    public void RemoveAt(int index)
    {
        BaseRemoveAt(index);
    }

    public void Remove(string name)
    {
        BaseRemove(name);
    }

    public void Clear()
    {
        BaseClear();
    }

    protected override void BaseAdd(ConfigurationElement policy)
    {
        BaseAdd(policy, false);
    }

    protected override sealed ConfigurationElement CreateNewElement()
    {
        return new PasswordPolicy();
    }

    protected override object GetElementKey(ConfigurationElement policy)
    {
        return ((PasswordPolicy)policy).Name;
    }
}

有什么想法吗?

2 个答案:

答案 0 :(得分:4)

在构造函数中,您调用Add方法。在此方法中,您可以调用非密封的BaseAdd方法,这就是您收到警告的原因。 要摆脱此代码分析警告,您必须密封BaseAdd方法

protected override sealed void BaseAdd(ConfigurationElement policy)
{
    BaseAdd(policy, false);
}

答案 1 :(得分:3)

实际上,这不是一个stylecop警告,而是一个代码分析(FxCop)警告。

您正在构造函数中调用虚方法(CreateNewElement已覆盖)。通常你不应该这样做,因为正如构造函数在C#中的工作原理一样,将首先调用基础构造函数(ConfigurationElementCollection.ctor),但它将执行最派生类的CreateNewElement,即一个你已经被覆盖了。

请参阅Virtual member call in a constructor