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

在 addPropertyChangeListener 上遇到 NullPointerException 的唯一方法是 PropertyChangeSupport 为 null。但显然在正常创建的对象中它不应该为空，但我猜这是你的问题，你的对象没有正常创建。 由于您的对象实现了 Serialized 接口，我猜测您的问题是由于您对这种类型的对象进行反序列化，但没有为反序列化对象创建瞬态 PropertyChangeSupport 对象。由于它是暂时的，因此反序列化时默认情况下不会创建它。如果是这样，您必须更改序列化在对象中的读取方式。您将需要执行

Custom Serialization

，特别是必须重写 readObject(...) 方法。另请查看

Effective Java 关于序列化的章节

。

---

编辑

顺便说一句，您的 PropertyChangeSupport 对象应该是 SwingPropertyChangeSupport 对象，因为您的是 Swing GUI

---

编辑2

但由于您的类扩展了 JPanel，所以这一切都毫无意义，因为 JPanel 已经具有
内在的 PropertyChangeSupport 以及 addPropertyChangeListener 和 removePropertyChangeListener 方法，因此只需使用 JPanel 的内在支持即可。

---

编辑3

哦，是的，一只小鸟通知我你的 paintComponent(...)
方法重写有缺陷，因为你几乎应该总是在重写方法中调用 super 的方法，以便可以绘制背景并完成内务处理。即

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

请注意，这不会帮助您解决 NPE，但有助于解决其他错误。

Answer 2