1

该类Controller是一个单例，这似乎是一种允许安全传递this到Controller.

Netbeans 给

配置“在构造函数中传递可疑参数”提示

这让controller.addObserver(this);我问更好的技术是什么，尽管我认为这不是一个好方法。

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");
    }
4

2 回答 2

7

当对象尚未完全构造时，您将传递this给外部类 ( )。然后可以在其构造尚未完成时引用您的对象。ControllerController

大多数人通过使用工厂方法来解决这个问题，该方法首先创建对象，然后从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;
}

看看这篇关于安全对象构造的优秀 Brian Goetz 文章。

于 2012-07-26T05:31:08.060 回答
1

在构造函数中使用它作为参数可能很危险，因为对象没有完全初始化

来自http://wiki.netbeans.org/Java_Hints

我想重点是，超类可能会尝试访问尚未初始化的类的一部分（或者您稍后在自己的构建过程中进行更改）

于 2012-07-26T05:33:10.500 回答