在搜索特定值时静默捕获 ClassCastException 是否安全？

Question

假设我正在实现一个排序集合（简单的示例 - 基于排序数组的 Set。）考虑这个（不完整的）实现：

import java.util.*;

public class SortedArraySet<E> extends AbstractSet<E> {

    @SuppressWarnings("unchecked")
public SortedArraySet(Collection<E> source, Comparator<E> comparator) {
    this.comparator = (Comparator<Object>) comparator;
    this.array = source.toArray();
    Arrays.sort(this.array, this.comparator);
}

@Override
public boolean contains(Object key) {
    return Arrays.binarySearch(array, key, comparator) >= 0;
}

    private final Object[] array;

    private final Comparator<Object> comparator;

}

现在让我们创建一组整数

Set<Integer> s = new SortedArraySet<Integer>(Arrays.asList(1, 2, 3), null);

并测试它是否包含一些特定值：

System.out.println(s.contains(2));
System.out.println(s.contains(42));
System.out.println(s.contains("42"));

上面的第三行将抛出一个ClassCastException。不是我想要的。我希望它返回 false （如 HashSet 所做的那样。）

我可以通过捕获异常并返回 false 来获得此行为：

@Override    
public boolean contains(Object key) {
    try {
        return Arrays.binarySearch(array, key, comparator) >= 0;
    } catch (ClassCastException e) {
        return false;
    }
}

假设 source< /code> 集合类型正确，如果我这样做会出现什么问题？

Answer 1

我认为这没有任何问题，因为 Collection.contains 的 Javadoc明确指出抛出 ClassCastException 是可选的。

我看到的唯一问题是，如果某处有错误，不抛出异常将阻止您查明它。

Answer 2

TreeSet 类确实会因 contains() 的参数不兼容而引发 ClassCastException（与使用的 Comparator 不兼容）放）。所以抛出这个异常并没有什么问题。只要确保您记录了这种情况可能会发生即可。

Answer 3

让 CCE 从 contains() 中抛出是完全合法的。然而，许多集合实现捕获并返回 false，我认为这也是完全合法的，事实上这是更用户友好的行为。

在 equals() 中你没有选择；你必须抓住那个CCE。

捕获未经检查的异常总是让人感觉很脏，但有时这是正确的做法。