ArrayList 中对 Collection.contains 方法的可疑调用

Question

我收到警告，watchStore.contains(s) 是对 java.util.Collection#contains 的可疑调用。我该如何解决？我想使用contains() 来查找具有匹配序列号的特定对象。

public Watch findWatchBySerialNumber(long srch) {
    long s = srch;
    Watch watch = null;

    for(int i = 0; i < watchStore.size(); i++) {
        watch = watchStore.get(i);
            if(watchStore.contains(s)) {
                System.out.print("item found");
                return watch;
            }
    } 
    System.out.print("item not found");
    return null; // watch is not found.
}

Answer 1

您可以使用 org.apache.commons.lang.ArrayUtils 包中的 contains 方法。

检查值是否在给定的数组中。

如果传入一个空数组，该方法返回false。

参数： array 要搜索的数组 valueToFind 要查找的值 回报： 如果数组包含对象，则为 true

long [] imageHashes= {12l,13l,14l,15l}; System.out.println(ArrayUtils.contains(imageHashes, 13l));

Answer 2

请问我该如何解决。我想使用 contains() 来查找 具有匹配序列号的特定对象。

在这种情况下，覆盖 Watch 的 equals() 以使用 serialNumber 字段进行比较。

然后添加接受序列号的构造函数。

public class Watch {
    private final long serialNumber;

    public Watch(long serialNumber) {
        this.serialNumber = serialNumber;
    }

    @Override
    public boolean equals(Object obj) {
        return obj == this ||
            (obj instanceof Watch && ((Watch)obj).serialNumber == serialNumber);
    }

    @Override
    public int hashCode() {
        return (int)serialNumber;
    }
}

将if(watchStore.contains(s)){ 替换为if(watchStore.contains(watchToFind)){，其中Watch watchToFind = new Watch(s);

Answer 3

假设Watch 是类，watchStore 是List<Watch>，并且serialNo 存在字段Watch...

public Optional<Watch> findWatchBySerialNumber(long serial) {
    return watchStore.stream()
                     .filter(w -> w.getSerialNo() == serial)
                     .findFirst();
}

如果您没有使用 Java 8，则代码很接近，但有点危险，因为您有机会返回 null。如果你可以使用 Guava 的Optional，这里会是更好的选择。

public Watch findWatchBySerialNumber(long serial) {
    for(Watch w : watchStore) {
        if(w.getSerialNo() == serial) {
            return w;
        }
     }
     return null;
}

---

您的contains 不起作用，因为您的列表不包含Longs，它包含Watchs。这也是编译器认为它可疑的原因。 contains 接受 Object，但如果您要查找的内容与列表中的内容没有可比的 equals，它将返回 false。

在这种情况下，您必须遍历整个集合才能找到它，尤其是因为您正在寻找这些对象的特定属性，而不是特定的、易于提供的值。