关于继承HashSet的疑问

先贴代码,然后再说我的疑问:
public class Concrete extends HashSet {

private static final long serialVersionUID = 8191406567257721779L;

private int count = 0;

public Concrete() {

}

public Concrete(int initCapacity, float loadFactor) {

    super(initCapacity, loadFactor);

}

@Override
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);
}

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是通过all来实现的。
所以当你调用return super.addAll(c); 这句话时,add方法被调用了3次,
所以你的count又自加了3次,因此就变成六了。。建议你把count += c.size(); 这句话去掉。

这种情况下,最好不要用继承,使用组合。让Concrete持有一个HashSet成员。
Effective Java 上有,LZ可以好好看看

[code="java"]
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); 
}

[/code]

鼠标右键 debug as -> Java Application 运行一下 自己看看他的运行顺序就知道。

@Override
public boolean addAll(Collection<? extends E> c) {
count += c.size();
return true;
}

你改成这样肯定是3啊。。对的啊。。c.size()就是3啊。

你是指你原来的代码还是修改后的代码?
如果是原来的代码的话,因为你最后return的是super.addAll(c); 是你自己调用了父类HashSet的方法。
如果是修改后的代码,没有调用HashSet的方法啊