Effective Java（Third Edition） item52：慎用重载

冷不防 2022-03-20 10:16 129阅读 0赞

public class SetList {
        public static void main(String[] args) {
            Set<Integer> set = new TreeSet<>();
            List<Integer> list = new ArrayList<>();
    
            for (int i = -3; i < 3; i++) {
                set.add(i);
                list.add(i);
            }
            for (int i = 0; i < 3; i++) {
                set.remove(i);
                list.remove(i);
            }
            System.out.println(set + " " + list);
        }
    }

你预期的结果是不是  \[-3, -2, -1\] \[-3, -2, -1\]  ，但结果却是 \[-3, -2, -1\] \[-2, 0, 2\]

道理很简单，set调用的remove方法是

boolean remove(Object o);

而list调用的remove方法为

E remove(int index);

List类中还有一个重载的方法

boolean remove(Object o);

也就是说，调用add的时候是一个对象，而调用remove的时候变为了另一个对象（一个新的对象），所以list的remove操作是这样的：原数组为 \[-3, -2, -1, 0, 1, 2\]，第一次调用remove(0)后变为\[-2, -1, 0, 1, 2\]，第二次会在一个新对象上操作，也就是在\[-2, -1, 0, 1, 2\]上操作remove(1)，结果为\[-2, 0, 1, 2\]，最后一次调用remove(2)同样会在一个新对象(\[-2, 0, 1, 2\])上操作，最后结果为\[-2, 0, 2\]，因为int与Integer调用的是不同的重载的remove方法，导致结果与我们的预期不一致，所以重载要慎用。

tips：上面的代码要想正常运行也很简单，把第二个for循环中的int改为Integer，或者把

list.remove(i);

改为

list.remove((Integer) i);

进而调用正确的重载remove方法，这样，运行结果就是我们预期的那样了。