Question

我正在迭代两个集合并检查两个集合是否都包含 相同的元素。我不能使用Java 8.

编辑1年后：

我在问题中创建了方法来检查两个Collections是否包含相同的元素，而没有考虑我将两个Collection实现传递给方法的事实。

但Collection并不确定元素的排序方式。我正在迭代收藏品。因此，Collection的某些实现可以按随机顺序保存元素，同时包含相同的元素。

两个系列都包含元素，具有可比性和内容如果所有元素都返回x.compareTo(y)为0，则定义为相等。

两个值被定义为不同，如果其中一个为空，而不是另一个。我想找到一种优雅的方法来比较无效和防止对最终compareTo()进行空检查。

我目前的实施：

    public static <T extends Comparable<T>> boolean isSame(@Nullable Collection<T> a, @Nullable Collection<T> b) {

    if (a == null || b == null) {
        return (a == null && b == null);
    }

    if (a.size() != b.size()) {
        return false;
    }
    Iterator<T> aIt = a.iterator();
    Iterator<T> bIt = b.iterator();
    while (aIt.hasNext()) {
        T aValue = aIt.next();
        T bValue = bIt.next();
        if (aValue == null || bValue == null) {
            if (aValue == null ^ bValue == null) {
                return false;
            }
            //both null, don't compare, continue looping...
        } else if (aValue.compareTo(bValue) != 0) {
            return false;
        }
    }
    return true;
}

如果两个值都为null，我想继续while循环，因为那样定义为相等。

但我正在努力解决这个问题：

if (aValue == null || bValue == null) {
        if (aValue == null ^ bValue == null) {
            return false;
        }
}

问题：

是否有更优雅和可读的方法来比较nullity，如果两者都不为null，则进行进一步比较，如果只有一个为null，则返回false，如果两个值都为null，则继续循环？

Answer 1

以下序列应该运作良好：

if(aValue == null && bValue == null) continue; // both null; continue
if(aValue == null || bValue == null) return false; // any null; return false
if(aValue.compareTo(bValue) != 0) { // both non-null; compare
    return false;
}

Answer 2

在Java8中，您可以构建一个Comparator，它将以创建额外对象为代价来替换比较序列（您需要决定是否关心它）：

Comparator<T> cmp = Comparator.nullsLast(Comparator.naturalOrder());

比较法将为您处理空比较（因为您假设两个null相等）：

while (aIt.hasNext()) {
    T aValue = aIt.next();
    T bValue = bIt.next();
    if (cmp.compare(aValue, bValue) != 0) {
      return false;
    }
}

以优雅的方式检查两个参数为null

2 个答案: