6

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

我有一个自定义 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.IDictionary2.ContainsKey(type parameter.TKey)”，因此无法添加 Requires。

4

2 回答 2

5

“我有一个自定义 Dictionary 包装类” - 实现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();
    }
}
于 2012-02-28T21:28:44.847 回答
1

坦率地说，我不明白合同的意义。合同是

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

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

最好你能确保返回值是true或者返回值是false并且Count大于零或等于零但是这样的合约有什么意义呢？调用者已经知道这一点。

鉴于此，我根本不会为这里的合同而烦恼。

于 2011-09-14T16:57:24.503 回答