问题描述
先贴代码，然后再说我的疑问：public class Concrete<E> extends HashSet<E> {private static final long serialVersionUID = 8191406567257721779L;private int count = 0;public Concrete() {}public Concrete(int initCapacity, float loadFactor) {super(initCapacity, loadFactor);}@Overridepublic boolean add(E e) {count++;return super.add(e);}@Overridepublic boolean addAll(Collection<? extends E> c) {count += c.size();return super.addAll(c);}public int getCount() {return count;}public static void main(String[] args) {Concrete<String> c = new Concrete();System.out.println(c.getCount());c.addAll(Arrays.asList("123", "456", "44444"));System.out.println(c.getCount());}}为什么我复写了addAll方法之后，然后getCount返回的不是3而是6？？？？我方法都重写了，不明白！！！希望大侠给我给个详细的解释。 问题补充：可能是我没表述明白,我知道HashSet中的addAll是用调用add来实现的，但是我就是搞不明白，即使我这样写：@Override public boolean addAll(Collection<? extends E> c) { count += c.size(); return true;} count的值还是3，为什么我复写了addAll方法，应该是调用我子类的逻辑。跟hashSet中的addAll好像是没什么关系啊。？
解决方案
因为HashSet里面addAll是通过all来实现的。所以当你调用return super.addAll(c); 这句话时，add方法被调用了3次，所以你的count又自加了3次，因此就变成六了。。建议你把count += c.size(); 这句话去掉。
解决方案二：
你是指你原来的代码还是修改后的代码？如果是原来的代码的话，因为你最后return的是super.addAll(c); 是你自己调用了父类HashSet的方法。如果是修改后的代码，没有调用HashSet的方法啊
解决方案三：
@Override public boolean addAll(Collection<? extends E> c) { count += c.size(); return true; } 你改成这样肯定是3啊。。对的啊。。c.size()就是3啊。
解决方案四：
public boolean add(E e) { count++; //打一个断点return super.add(e); } @Override public boolean addAll(Collection<? extends E> c) { count += c.size(); //打一个断点return super.addAll(c); }鼠标右键 debug as -> Java Application 运行一下 自己看看他的运行顺序就知道。
解决方案五：
这种情况下，最好不要用继承，使用组合。让Concrete持有一个HashSet成员。Effective Java 上有，LZ可以好好看看