Quelle implémentation du constructeur est correcte?

Question

Exemple 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 

Example2

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

Les deux sont presque aussi "correct" (selon la définition d'une de de "correct" :-), bien que je préférerais la celui-ci comme il est plus simple.

Extraire tout le corps de votre constructeur ne ferait que rendre votre propre code si vous aviez d'autres constructeurs/méthodes appelant buildGUI (donc double emploi en éliminant), ou si vous pouvez diviser le corps en plusieurs petites méthodes distinctes avec des noms explicites (rendant ainsi le code plus lisible).

Answer 2

Ils sont fonctionnellement identiques. C'est une question de style.

En général, je dirais que ni l'un ni l'autre n'est correct. La logique dans le constructeur doit être minimale et avoir le moins de chance de générer une erreur. Je déplace l'appel buildGUI dans une méthode init, et l'appelle séparément de Main après avoir instancié l'objet.