どのコンストラクタの実装が正しいか？

Question

例1

import javax.swing.*; 
import java.awt.*; 

public class Painter extends JPanel{ 

    public Painter(){ 
    buildGUI(); 
    } 

    private void buildGUI(){ 
    JFrame frame = new JFrame(); 
    frame.setLayout(new BorderLayout()); 
    frame.setTitle("Paint drawing demonstration"); 
    new Center_frame(frame); 

    JPanel headerPanel = new JPanel(); 
    headerPanel.add(new JLabel("The drawing panel is below")); 

    Drawing_panel dp = new Drawing_panel(); 

    frame.add(BorderLayout.NORTH,headerPanel); 
    frame.add(BorderLayout.SOUTH,dp); 

    frame.pack(); 
    frame.setVisible(true); 
    } 

    public static void main(String args[]){ 
     new Painter(); 
    } 

}//end class Painter 

例2、私が好むだろうけれども、それらの両方がほぼ均等に "正しい"（ "正しい" :-)の一つの定義に依存している

import javax.swing.*; 
import java.awt.*; 

public class Painter extends JPanel{ 

    public Painter(){ 
     JFrame frame = new JFrame(); 
     frame.setLayout(new BorderLayout()); 
     frame.setTitle("Paint drawing demonstration"); 
     new Center_frame(frame); 

     JPanel headerPanel = new JPanel(); 
     headerPanel.add(new JLabel("The drawing panel is below")); 

     Drawing_panel dp = new Drawing_panel(); 

     frame.add(BorderLayout.NORTH,headerPanel); 
     frame.add(BorderLayout.SOUTH,dp); 

     frame.pack(); 
     frame.setVisible(true); 
    } 

public static void main(String args[]){ 
    //create a Painter object 
    new Painter(); 
} 

}//end class Painter 

Answer 1

後者の方が簡単です。

buildGUIを呼び出す他のコンストラクタ/メソッド（重複を排除する）がある場合、または本体を複数の別個の小さなメソッドに分かりやすい名前で分けることができる場合は、コンストラクタ全体を抽出するだけでコードがきれいになりますコードをより読みやすくする）。

Answer 2

機能的には同じです。それはスタイルの問題です。

一般に、どちらも正しいとは思わないでしょう。コンストラクタ内のロジックは最小限に抑えられ、エラーが発生する可能性は最小限に抑えられます。 buildGUIの呼び出しをinitメソッドに移動し、オブジェクトをインスタンス化した後にMainとは別に呼び出します。