JavaFX程序中Button对象为空引发NullPointerException的原因排查

嘿，我一眼就揪出问题啦！你遇到的空指针异常完全是因为成员变量和局部变量重名导致的，咱们一步步来解决：

问题根源 #

你在类顶部声明了成员变量：

private Button blurButton, rotateButton, scaleButton;

但在start()方法里，你又重新创建了同名的局部变量：

Button rotateButton = new Button();
Button blurButton = new Button();
Button scaleButton = new Button();

这就导致类的成员变量根本没被初始化，一直是null。当handle()方法调用rotateButton.isPressed()时，它访问的是未赋值的成员变量，自然就抛出NullPointerException了。

另外还有个小问题：ActionEvent是在按钮释放时触发的，此时调用isPressed()会返回false，所以这个判断本来就不会生效，应该用e.getSource()来判断点击的按钮。

修正后的代码 #

只需要修改start()方法里的变量声明，去掉前面的Button，直接给成员变量赋值，同时调整handle()里的判断逻辑：

package com.example.effectslikerotation;
import javafx.application.Application;
import javafx.event.ActionEvent;
import javafx.event.EventHandler;
import javafx.geometry.Pos;
import javafx.scene.Scene;
import javafx.scene.control.Button;
import javafx.scene.control.Label;
import javafx.scene.layout.FlowPane;
import javafx.scene.transform.Rotate;
import javafx.stage.Stage;
import java.io.IOException;

public class HelloApplication extends Application implements EventHandler<ActionEvent> {
    private Button blurButton, rotateButton, scaleButton;
    private Rotate rotate;
    private double angle;
    Object buttonClicked;
    Object buttonText;

    @Override
    public void start(Stage stage) throws IOException {
        // 去掉前面的Button，直接给成员变量赋值
        rotateButton = new Button();
        blurButton = new Button();
        scaleButton = new Button();
        angle = 0.0;
        rotateButton.setOnAction(this);
        blurButton.setOnAction(this);
        scaleButton.setOnAction(this);
        rotateButton.setText("Rotate");
        blurButton.setText("Blur");
        scaleButton.setText("Scale");
        Label reflect = new Label("I'm a reflection of you.");
        FlowPane FProot = new FlowPane();
        Scene scene = new Scene(FProot, 600, 500);
        FProot.setAlignment(Pos.CENTER);
        FProot.getChildren().addAll(rotateButton, blurButton, scaleButton, reflect);
        stage.setTitle("Hello!");
        stage.setScene(scene);
        stage.show();
        //rotateButton.getTransforms().add(rotate);
    }

    public static void main(String[] args) {
        launch();
    }

    @Override
    public void handle(ActionEvent e) {
        Object buttonClicked = e.getSource();
        // 改用equals判断点击的按钮，和另外两个保持一致
        if (buttonClicked.equals(rotateButton)) {
            System.out.println("Rotate button clicked");
        }
        if (buttonClicked.equals(blurButton)) {
            System.out.println("Blur button clicked");
        }
        if (buttonClicked.equals(scaleButton)) {
            System.out.println("Scale button clicked");
        }
    }
}

关键要点总结 #

• 不要让局部变量覆盖成员变量：当你需要给类的成员变量赋值时，不要重新声明变量类型，直接使用成员变量名赋值。
• 正确判断点击的按钮：在ActionEvent的处理方法中，通过e.getSource()获取触发事件的组件，再用equals()判断是哪个按钮，而不是依赖isPressed()（因为事件触发时按钮已经不再处于按下状态）。

内容的提问来源于stack exchange，提问作者foffor