addPropertyChangeListener上的NullPointerException

Question

我创建了一个简单的LED，可以接收来自许多数字组件（如开关/门）的输入。问题是，当尝试实现PropertyChangeListener接口时，我得到一个NullPointerException。使用下面的代码，如果我只是将它添加到JFrame表单并尝试运行它我得到此异常。我已经实现了与门/开关相同的LED，但由于某些原因，我的代码会产生错误。任何帮助表示赞赏。

package Digital;

import java.awt.Image;
import java.beans.PropertyChangeEvent;
import java.beans.PropertyChangeListener;
import java.beans.PropertyChangeSupport;
import java.io.Serializable;


public class LED extends javax.swing.JPanel implements PropertyChangeListener {

    private Image led_on;
    private Image led_off;
    private Image image;
    private Terminal input;
    private transient PropertyChangeSupport propertyChangeSupport = new PropertyChangeSupport(this);

    public LED() {
        java.net.URL url_on = getClass().getResource("images/LED_on.gif");
        led_on = new javax.swing.ImageIcon(url_on).getImage();
        this.setSize(led_on.getWidth(null), led_on.getHeight(null));
        java.net.URL url_off = getClass().getResource("images/LED_off.gif");
        led_off = new javax.swing.ImageIcon(url_off).getImage();
        this.setSize(led_off.getWidth(null), led_off.getHeight(null));
        this.image = led_off;    
    }

    @Override
    public void paintComponent(java.awt.Graphics g) {
        g.drawImage(image, 0, 0, null);
    }


    public static final String PROP_INPUT = "input";

    public Terminal getInput() {
        return input;
    }

    public void setInput(Terminal input) {
        if (input != null) {
            input.addPropertyChangeListener(this);
            this.addPropertyChangeListener(this);
        }

        Terminal oldInput = this.input;
        this.input = input;
        propertyChangeSupport.firePropertyChange(PROP_INPUT, oldInput, input);
    }



    public void addPropertyChangeListener(PropertyChangeListener listener) {
        propertyChangeSupport.addPropertyChangeListener(listener);
    }


    public void removePropertyChangeListener(PropertyChangeListener listener) {
        propertyChangeSupport.removePropertyChangeListener(listener);
    }



    public boolean Recalculate(Terminal input) {
        if (input!=null) {
            return input.getValue();
        } else {
            return false;
        }
    }


    public void ledChange(boolean ledValue) {
        if (ledValue) {
            image = led_on;
        } else {
            image = led_off;
        }
        repaint();
    }


    public void propertyChange(PropertyChangeEvent pce) {
        boolean terminalValue = Recalculate(input);
        ledChange(terminalValue);
    }


}

Answer 1

如果PropertyChangeSupport为null，则在addPropertyChangeListener上体验NullPointerException的唯一方法是。但是在通常创建的对象中显然不应该为null，但我猜这是你的问题，你的对象没有正常创建。

由于您的对象实现了Serializable接口，我猜测您的问题是由于您反序列化此类型的对象而不是为反序列化对象创建瞬态PropertyChangeSupport对象。由于它是瞬态的，因此在反序列化时不会默认创建它。如果是这样，您必须更改序列化在对象中的读取方式。您需要执行Custom Serialization，特别是必须覆盖readObject(...)方法。另请查看Effective Java chapter on Serialization。

---

修改
另外，您的PropertyChangeSupport对象应该是SwingPropertyChangeSupport对象，因为您的对象是Swing GUI

---

编辑2
但是由于你的类扩展了JPanel，所以这一切都没有用，因为JPanel 已经使用addPropertyChangeListener和removePropertyChangeListener方法完成了内在的PropertyChangeSupport，所以只需使用JPanel的内在支持。

---

编辑3
哦，是的，有点小鸟通知我你的paintComponent(...)方法覆盖是不足的，因为你几乎总是在覆盖方法中调用super的方法，以便可以绘制背景并完成内务处理。即，

@Override
public void paintComponent(java.awt.Graphics g) {
    super.paintComponent(g); // **** add this
    g.drawImage(image, 0, 0, null);
}

请注意，这对您的NPE没有帮助，但有助于解决其他错误。