java - 检查对象的身份和 Sonar 问题

Question

我们正在使用 Sonar 检查我们代码的质量，Sonar 找到了比较对象身份的代码，如下所示:

if (cellOfInterest == currentCell) { … }

Sonar 发现这种身份检查非常奇特，足以将其称为关键，并建议用相等性检查(使用 .equals() 代替)。身份检查，因此其背后的基本原理，通常不是本意。

然而，在我们的例子中，我们遍历 Cell 列表并在每次迭代 (currentCell) 中检查我们是否正在处理我们已经拥有的特殊单元格 ( cellOfInterest).

我想知道除了我们的模式之外，是否还有其他模式很常见，并且仅通过使用不同的设计就可以避免这个问题。或者您建议采取什么解决方案来避免在上述情况下使用身份检查？

我们考虑用如上所述的相等性检查替换身份检查，但它似乎不适用于我们的情况，因为其他单元格也可能“相等”但不完全相同。

欢迎所有想法!

Answer 1

如果身份是您所需要的，那么它就是您所需要的。这个警告是有道理的，因为它通常是一个错误，但在这种情况下它不是。

例如，IdentityHashMap (适用于身份与平等)在其 javadoc 中有这个:

This class is not a general-purpose Map implementation! [...] This class is designed for use only in the rare cases wherein reference-equality semantics are required.

所以它很少有用，但也有它的用处。你处于类似的位置。当然，its internal code完全符合您的预期，它使用 == 获取 key 。

底线:我不明白为什么你需要让代码更复杂只是因为一些静态代码分析工具说它可能是一个问题 - 这样的工具应该帮助你，而不是强制你进入一些奇怪的构造，基本上会做同样的事情。为了清楚起见，我将解释为什么在评论中使用 ==，将其标记为误报并继续。

如果你真的想删除警告，我能想到的唯一选择是使用 equals 并将 equals 方法更改为:

• 基于身份的默认Object#equals
• 一些唯一标识单元格的实现，可能基于一些唯一的 ID 或坐标？