我应该使用包含构造函数中的其他逻辑的setter吗？

Question

每当设置一个新列表时,我有一个类需要在列表上重置其内部迭代器:

public class ListElementReceiver implements ElementReceiver {
    private List<Element> elements;
    private Iterator<Element> elementIter;

    public void reset() {
        elementIter = elements.iterator();
    }

    public void setElements(List<Element> elements) {
        this.elements = elements;
        reset();
    }
}

它基本上只是一个列表及其迭代器的包装器,所以我可以将它与我给定的接口一起使用ElementReceiver.我遇到的问题是为这个类构建构造函数.哪两种方法更可取？

// Approach 1: Duplicate logic, independant of Setter
public ListElementReceiver() {
    elements = new List<Element>();
    reset();
}

public ListElementReceiver(List<Element> elements) {
    this.elements = elements;
    reset();
}

//Approach 2: Make dependant on Setter
public ListElementReceiver() {
    setElements(new List<Element>());
}

public ListElementReceiver(List<Element> elements) {
    setElements(elements);
}

Answer 1

从构造函数中,您不应该调用可以被子类覆盖的方法(包括setter).这可能导致子类中的代码在子类完全构造之前执行,这可能导致难以跟踪的错误.

调用private或final方法(如果他们不调用任何可重写的代码),或调用final类中的方法,这是可以的,但这看起来不像你在这里.

通过让一个构造函数调用另一个构造函数,可以避免使用第一种方法重复逻辑:

public ListElementReceiver() {
    this(new ArrayList<>());
}

public ListElementReceiver(List<Element> elements) {
    this.elements = elements;
    elementIter = elements.iterator();
}