ReSharper 是否会警告字典键？

Question

我们最近遇到一个问题，我们调用字典来检索值，并期望键存在。它并没有导致流程中断。

ReSharper 是否具有检查此问题的功能，就像我听说它可以检查空对象一样？

这里有一个例子来支持我所说的内容：

Dictionary<String, Entity> allEntities = 
      new Dictionary<String, Entity>(SringComparer.OrdinalIgnoreCase);

allEntities.AddMany(db.GetAllEntities());

Entity thisEntity = allEntities[entityID]; 
      // <-- error here as EntityID isn't in all entities...

我希望 ReSharper 能够说我没有像这样检查字典：

if (allEntities.ContainsKey(entityID))
    ...

---

仅供参考，我没有 ReSharper，但这将是另一件事添加到业务案例中以供所有开发人员使用。

Answer 1

即使它可以（我认为它不会），它所做的只是警告您键可能不在字典中 - 就像它警告您一个对象可能为空。您仍然有责任添加代码来检查您是否认为有必要。

我认为，如果这样的功能在每次访问字典时都会向您发出警告，那么它将是一种负担，而不是一种祝福。然后，您可能会争辩说，您希望它就所有其他类型的集合访问或可能引发的任何异常发出警告。这里要传达的信息是，您应该更多地依赖单元测试来捕获此类问题，而不是静态分析工具

编辑：

由于 ReSharper 没有此功能，如果您确实需要它，您可以考虑使用新的功能自己编写它Roslyn API。请参阅这篇文章< /a> 有关如何编写代码分析器的示例

Answer 2

我目前已经安装了 ReSharper 6，至少在我的设置中，它不会警告我在访问其值之前使用 ContainsKey 检查 Dictionary。

Answer 3

Resharper是编译时使用的工具。它怎么可能知道在运行时哪些值将被放入字典中？

在对它进行任何操作之前，您可以检查字典以查看它是否具有您想要的值。

Entity thisEntity;
if (allEntities.TryGetValue(entityID, out thisEntity)){
    //DoStuff with thisEntity
}

或者仅使用 if (allEntities.ContainsKey(entityID)){}