在清除之前检查计数是否有任何好处？ [关闭]

Question

我看到有很多人在C＃中执行此操作...

if (someDictionary.Count > 0) {
    someDictionary.Clear();
}

我的问题是，为什么不只是.Clear它？ 这有什么用呢？ 我应该在清算前检查计数吗？还是只是没用，没有什么不同？

更重要的是，我很困惑为什么我经常看到它。

Answer 1

如果查看Dictionary.Clear（）的源代码，您会发现它会自行检查。

    public void Clear() {
        if (count > 0) {
            for (int i = 0; i < buckets.Length; i++) buckets[i] = -1;
            Array.Clear(entries, 0, count);
            freeList = -1;
            count = 0;
            freeCount = 0;
            version++;
        }
    }

因此，在外面进行检查似乎是在浪费时间。

人们为什么要这样做？ 可能是因为在不需要时不进行工作似乎很有意义，并且如果您不知道实现的目的，您可能会认为自己有所收获。

Answer 2

不，没有必要。 正如您可以从参考实现中看到的那样， Clear()方法实际上是自己完成的：

public void Clear() {
    if (count > 0) {
         /* omitted for brevity */
     }
}

Answer 3

我不会担心别人为什么会做这样的事情，为什么要这么做，有这么多方法可以使人动容，在软件中有很多不同的用例，不良做法等……我们永远也不会知道为什么会这样完成。

我们所知道的是，除非您有理由这样做，否则无需先检查计数。 在集合上调用clear将清除它，如果它已经为空，则将不执行任何操作并继续运行（即，不会引发异常）。 几乎没有影响性能，甚至没有影响，实际上，如果您知道无论如何都要清除该计数，则可能会认为首先检查计数实际上较慢。