私のコンストラクタから追加のロジックを含むセッターを使うべきでしょうか？

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

コンストラクターから、サブクラスによってオーバーライドできるメソッド（セッターを含む）を呼び出さないでください。そのため、サブクラスが完全に構築される前にサブクラス内のコードが実行され、追跡が困難なバグが発生する可能性があります。

それは（彼らは、順番に、任意のoverrideableコードを呼び出しません提供）privateまたはfinalメソッドを呼び出すために、またはfinalクラスのメソッドを呼び出すには大丈夫だが、それはあなたがここに持っているもののようには見えません。

あなたが1つのコンストラクタは、別のを呼び出すことによって、あなたの最初のアプローチとの重複ロジックを避けることができます。

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

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