Течь этот конструктор

Question

Можно создать дубликат:
“this” reference escaping during construction?

Я хотел бы знать, является ли утечка этого в вопросе конструктора применяется на этом примере, так как я считаю, что конструктор должен вызывать setJMenuBar() для завершения его строительства, здесь применяется класс:

public class StaffManagerMainWindow extends JFrame implements ActionListener { 

    public StaffManagerMainWindow(String title, Image icon) throws HeadlessException { 
     ... 
     setJMenuBar(); 
    } 

    private void setJMenuBar() { 
     ... 
     exitItem.addActionListener(this); 
     ... 
    } 

    @Override 
    public void actionPerformed(ActionEvent e) { 
     throw new UnsupportedOperationException("Not supported yet."); 
    } 
} 

Answer 1

I t hink есть утечка в конструкторе. Вы передаете «это» перед полной инициализацией объекта.

Answer 2

Утечка this в конструкторе может быть проблемой, если ваша программа многопоточная. Вы даете это exitItem, который может вызывать actionPerformed() из другого потока до завершения вашего конструктора. Это не в порядке. Это может сильно испортить вашу программу, особенно если вы добавите подклассы к изображению.

Создать метод инициализации() и фабричный метод вместо:

public class StaffManager { 
    public static StaffManager create() { 
     StaffManager staffManager = new StaffManager(); 
     staffManager.init(); 
     return staffManager; 
    } 

    private StaffManager() { 
    } 

    private void init() { 
     // Add listeners here. 
    } 
} 

Edit: Поскольку объекты распашных всегда создается на событие диспетчеризации нити, утечка this не будет иметь никакого эффекта в этом случае. Тем не менее я предпочитаю не делать этого.