在構造函數中泄漏這個

Controller類是一個單例，它似乎是一個特殊情況，允許將this安全地傳遞給Controller。在構造函數中泄漏這個

的Netbeans給

配置這讓我向更好的技術會是什麼暗示

爲controller.addObserver(this);「的構造函數傳遞參數可疑」，雖然我gather it's not a good approach。

package net.bounceme.dur.usenet.swing; 

import java.util.Observable; 
import java.util.Observer; 
import java.util.logging.Logger; 
import javax.mail.Folder; 
import javax.swing.ListModel; 
import net.bounceme.dur.usenet.controller.Controller; 
import net.bounceme.dur.usenet.controller.MessageBean; 
import net.bounceme.dur.usenet.controller.MessagesDefaultListModel; 

public class MessageSelect extends javax.swing.JPanel implements Observer { 

    private static final Logger LOG = Logger.getLogger(MessageSelect.class.getName()); 
    private Controller controller = Controller.getInstance(); 
    private ListModel messages = new MessagesDefaultListModel(); 
    private MessageBean messageBean = new MessageBean(); 

    @SuppressWarnings("unchecked") 
    public MessageSelect() { 
     controller.addObserver(this); 
     initComponents(); 
     messagesJList.setPrototypeCellValue("xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"); 
    }

來源

2012-07-26 Thufir

你傳入this到外部類（Controller）時，對象還沒有被完全構造。 Controller然後可以引用你的對象，而其構造尚未完成。

大多數人通過使用工廠方法來解決此問題，該方法首先創建對象，然後從外部傳遞this。

// private to force clients to use the static factory method 
private MessageSelect() { 
    initComponents(); 
    messagesJList.setPrototypeCellValue("xxx"); 
} 

public static MessageSelect createInstance() { 
    MessageSelect instance = new MessageSelect(); 
    instance.controller.addObserver(instance); 
    return instance; 
}

看看this excellent Brian Goetz article安全物體的建設。

來源

2012-07-26 05:31:08