Метод hashSet addAll изменяет внутреннее поле класса

Question

У меня есть класс, который расширяет параметризованную версию HashSet. Этот класс имеет внутреннее поле currentSize, которое отслеживает, сколько элементов было добавлено до сих пор. Класс child переопределяет базовый класс 'add и addAll, поэтому соответственно увеличивается currentSize. Моя проблема заключается в том, что внутри addAll размер набора добавляется два раза к счетчику currentSize, не один раз. Ниже мой класс Код:

public class InheritHashSet extends HashSet<Integer> { 
    private int currentSize = 0; 

    @Override 
    public boolean add(Integer e) { 
     currentSize++; 
     super.add(e); 
    } 

    @Override 
    public boolean addAll(Collection<? extends Integer> e) { 
     currentSize += e.size(); 
     super.addAll(e); 
    } 

    public int getCurrentSize() { 
     return currentSize; 
    } 
} 

Метод add, кажется, работает нормально, так как после добавления трех элементов, currentSize является 3, но следующий будет добавить 6 к currentSize вместо 3:

InheritHashSet ihs = new InheritHashSet(); 
Collection<Integer> c = new LinkedList<Integer>(); 
c.add(new Integer(0)); 
c.add(new Integer(1)); 
c.add(new Integer(2)); 
ihs.addAll(c); 
System.out.println(ihs.getCurrentSize()); // prints 6 instead of 3 

Похоже, что вызов super.addAll(e) также изменяет currentSize (это, похоже, не имеет никакого смысла).

Answer 1

Метод addAll()InheritHashSet класс первый комплект currentSize = 3, с currentSize += e.size(); инструкция. Затем super.addAll(e); звонки add(e) способ три раза. Для динамического связывания public boolean add(Integer e) класса InheritHashSet вызывается первым, а затем currentSize затем увеличивается до 6.

по этой причине вы должны реализовать addAll метод таким образом:

@Override 
public boolean addAll(Collection<? extends Integer> e) { 
    return super.addAll(e); 
} 

Answer 2

Практически не рекомендуется распространять существующий класс коллекции, особенно когда вы пытаетесь обеспечить соблюдение нового контракта.

Я бы порекомендовал вам перепроектировать ваш класс, чтобы иметь поле HashSet, а не расширять HashSet.

Прочитано Effective Java by Joshua Bloch Пункт 16: Преимущество композиции над наследованием.