HashMap (Java) 给出错误的结果

Question

我创建了一个哈希映射，其中每个条目对应 3 个值 关键对象值（数量为两个）

我创建了一个类，我创建了其对象并将结果存储在哈希映射中 这是我下面的代码，其中我将传入的数据与哈希映射中的先前值进行比较。如果出现相同的数据，那么我只需增加该数据的计数器。我已经在 for 循环中获取了 print 语句。尽管两个字符串匹配，但我的代码仍然从未进入 if 循环以增加计数器。为什么？

 for(i=1;i<=hMap.size();i++)
 {
      String skey = Integer.toString(i);
      if(hMap.get(skey).olddata==comingdata)
      {
           hMap.get(skey).counter=  hMap.get(skey).counter+1;
      }
 }

Answer 1

您还没有提供有关所涉及类型的足够信息，但是，但我强烈怀疑这就是问题所在：

if(hMap.get(skey).olddata==comingdata)

如果olddata引用，而不是相等性code> 和 comingdata 是某种类型的引用。 （编辑：从它的声音来看，它们是字符串引用。）

我的猜测是你想要：

String skey = Integer.toString(i);
if(hMap.get(skey).olddata.equals(comingdata))
{
   hMap.get(skey).counter=  hMap.get(skey).counter+1;
}

或者更有效地避免无意义的查找：

WhateverType value = hMap.get(Integer.toString(i));
if (value.olddata.equals(comingdata))
{
    value.counter++;
}

我还建议如果你总是要按整数查找，为什么不使用 Integer 键而不是总是将整数转换为字符串？

此外，遵循 Java 命名约定是值得的，并且您应该将字段设为私有（如果尚未设为私有）。

如果这些都没有帮助，请发布更多代码。问题出现在 HashMap 中而不是出现在代码中的可能性非常小。

Answer 2

目前尚不清楚 olddata 的类型，但也许您应该使用 equals() 来比较值：

if (hMap.get(skey).olddata.equals(comingdata))

在 Java 中，== 用于比较原始数据类型的相等性或比较对象类型的同一性。如果您需要比较两个对象类型的相等性，则必须使用 equals() 方法，该方法是为所有对象定义的，因为它继承自 Object 类，请注意您还必须重写类中的 equals() 和 hashCode()，提供对该类有意义的实现。

Answer 3

在 Java 中，您不会将对象与 == 进行比较，除非您试图查看它们是否具有相同的参考值。

if (hMap.get(skey).olddata.equals(comingdata)) {
...

您也不应该像那样公开 olddata；它应该可以通过 getter 获得；例如getOldData()

Answer 4

应该是

if(hMap.get(skey).olddata.equals(comingdata))

Answer 5

您实际上是指 comingdata.equals(hMap.get(skey).olddata) 吗？此外，请注意 equals(Object) 和 hashCode() 必须正确实现。