Question

我不确定我在这里做错了什么，或者是否需要修复......

我有一个自定义的Dictionary包装类，这里是一段必要的代码片段。

public int Count
{
    get
    {
        Contract.Ensures(Contract.Result<int>() >= 0);

        return InternalDictionary.Count;
    }
}

public bool ContainsKey(TKey key)
{
    //This contract was suggested by the warning message, if I remove it
    //I still get the same warning...
    Contract.Ensures(!Contract.Result<bool>() || Count > 0);

    return InternalDictionary.ContainsKey(key);
}

我为ContainsKey添加行的唯一原因是因为我收到了以下警告消息（仍然如此）：Codecontracts: ensures unproven: !Contract.Result<bool>() || @this.Count > 0。我可以删除此行，但仍然可以获得相同的问题！

我该怎么做才能摆脱这些问题？

更新

我也尝试过（按照建议）......

public Boolean ContainsKey(TKey key)
{
    Contract.Requires(Count == 0 || InternalDictionary.ContainsKey(key));
    Contract.Ensures(!Contract.Result<bool>() || Count > 0);

    return InternalDictionary.ContainsKey(key);
}

警告5方法 'My.Collections.Generic.ReadOnlyDictionary 2.ContainsKey(type parameter.TKey)' implements interface method 'System.Collections.Generic.IDictionary 2.ContainsKey（type parameter.TKey）'，因此无法添加需要。

Answer 1

“我有一个自定义的字典包装类” - 实现IDictionary<TKey, TValue>。接口方法可以指定契约，实现它们的类方法必须符合契约。在这种情况下，IDictionary<TKey, TValue>.ContainsKey(TKey)有你要问的合同：

Contract.Ensures(!Contract.Result<bool>() || this.Count > 0);

逻辑上，!a || b可以被视为a ===> b（a隐含b），使用它，我们可以将其翻译为英语：

If ContainsKey() returns true, the dictionary must not be empty.

这是一个非常明智的要求。空字典不得声称包含密钥。此是您需要证明的。

这是一个示例DictionaryWrapper类，它添加Contract.Ensures以承诺Count的实现细节等于innerDictionary.Count是其他方法可以依赖的硬保证。它会向Contract.Ensures添加类似的ContainsKey，以便IDictionary<TKey, TValue>.TryGetValue合同也可以验证。

public class DictionaryWrapper<TKey, TValue> : IDictionary<TKey, TValue>
{
    IDictionary<TKey, TValue> innerDictionary;

    public DictionaryWrapper(IDictionary<TKey, TValue> innerDictionary)
    {
        Contract.Requires<ArgumentNullException>(innerDictionary != null);
        this.innerDictionary = innerDictionary;
    }

    [ContractInvariantMethod]
    private void Invariant()
    {
        Contract.Invariant(innerDictionary != null);
    }

    public void Add(TKey key, TValue value)
    {
        innerDictionary.Add(key, value);
    }

    public bool ContainsKey(TKey key)
    {
        Contract.Ensures(Contract.Result<bool>() == innerDictionary.ContainsKey(key));
        return innerDictionary.ContainsKey(key);
    }

    public ICollection<TKey> Keys
    {
        get
        {
            return innerDictionary.Keys;
        }
    }

    public bool Remove(TKey key)
    {
        return innerDictionary.Remove(key);
    }

    public bool TryGetValue(TKey key, out TValue value)
    {
        return innerDictionary.TryGetValue(key, out value);
    }

    public ICollection<TValue> Values
    {
        get
        {
            return innerDictionary.Values;
        }
    }

    public TValue this[TKey key]
    {
        get
        {
            return innerDictionary[key];
        }
        set
        {
            innerDictionary[key] = value;
        }
    }

    public void Add(KeyValuePair<TKey, TValue> item)
    {
        innerDictionary.Add(item);
    }

    public void Clear()
    {
        innerDictionary.Clear();
    }

    public bool Contains(KeyValuePair<TKey, TValue> item)
    {
        return innerDictionary.Contains(item);
    }

    public void CopyTo(KeyValuePair<TKey, TValue>[] array, int arrayIndex)
    {
        innerDictionary.CopyTo(array, arrayIndex);
    }

    public int Count
    {
        get
        {
            Contract.Ensures(Contract.Result<int>() == innerDictionary.Count);
            return innerDictionary.Count;
        }
    }

    public bool IsReadOnly
    {
        get
        {
            return innerDictionary.IsReadOnly;
        }
    }

    public bool Remove(KeyValuePair<TKey, TValue> item)
    {
        return innerDictionary.Remove(item);
    }

    public IEnumerator<KeyValuePair<TKey, TValue>> GetEnumerator()
    {
        return innerDictionary.GetEnumerator();
    }

    IEnumerator IEnumerable.GetEnumerator()
    {
        return innerDictionary.GetEnumerator();
    }
}

Answer 2

坦率地说，我不明白合同的要点。合同是

 Contract.Ensures(!Contract.Result<bool>() || Count > 0);

你想说什么？您既不能保证字典包含密钥，也不能保证字典包含任何值。所以这个合同不能总是得到满足。这就是验证者告诉你的：它不能证明你承诺这是真的这个陈述是真的。

您最好确保返回值为true或返回值为false且Count大于零或等于零但这样的合同有什么意义呢？来电者已经知道了这一点。

鉴于此，我根本不会在这里打扰合同。

代码合同：确保未经证实的＆amp;需要未经证实

2 个答案: