JavaFX FXML程序按键切换图片时出现NullPointerException求助

核心问题 #

你手动创建的HelloController实例，和FXMLLoader加载FXML时自动创建的控制器实例不是同一个对象：

• 你在start方法里new HelloController()得到的实例，并没有和FXML文件绑定，它里面的Image1等ImageView成员变量从未被初始化（始终为null）。
• 真正和FXML绑定、能获取到ImageView实例的是FXMLLoader内部创建的控制器对象，但你并没有给这个实例绑定按键监听。

另外，你定义的currentID、selected、box如果是方法内的局部变量，每次调用createPage都会重置为初始值，导致状态无法保留，这也是currentID始终为1的原因。

具体修复步骤 #

1. 修正控制器成员变量定义 #

把currentID、selected、box改成控制器类的成员变量，同时确保FXML中的ImageView通过@FXML正确注入：

public class HelloController {
    // 改为类成员变量，保留状态
    private int currentID = 1;
    private int selected = 1;
    private int box = currentID % 3;

    // 与FXML中fx:id同名的ImageView变量，必须加@FXML注解
    @FXML
    private ImageView Image1;
    @FXML
    private ImageView Image2;
    @FXML
    private ImageView Image3;

    // FXML加载完成后自动调用的初始化方法
    @FXML
    public void initialize() throws URISyntaxException {
        createPage();
    }

    public void createPage() throws URISyntaxException {
        do {
            for (ImageInfo image : Database.ImgList){
                if(image.getId() == currentID){
                    changeImage(image.getName(), box);
                }
            }
            currentID++;
            box = currentID % 3;
        } while (box != 1);
    }

    // 示例changeImage实现（根据box赋值对应ImageView）
    private void changeImage(String imageName, int box) throws URISyntaxException {
        Image image = new Image(getClass().getResource(imageName).toURI().toString());
        switch(box) {
            case 1:
                Image1.setImage(image);
                break;
            case 2:
                Image2.setImage(image);
                break;
            case 0: // currentID为3的倍数时，取余结果为0
                Image3.setImage(image);
                break;
        }
    }

    public void moveCharacter(KeyEvent event) {
        switch (event.getCode()){
            case D, RIGHT:
                try {
                    moveForward();
                } catch (URISyntaxException e) {
                    throw new RuntimeException(e);
                }
                break;
            // 其他按键逻辑...
        }
    }

    private void moveForward() throws URISyntaxException {
        if (selected % 3 == 0){
            createPage();
            selected = 1; // 重置计数，避免重复触发
        } else {
            selected++;
        }
    }
}

2. 修正start方法的控制器获取逻辑 #

不要手动new控制器，而是从FXMLLoader获取它创建的绑定实例：

public void start(Stage stage) throws IOException {
    Database.loadImages();
    FXMLLoader fxmlLoader = new FXMLLoader(HelloApplication.class.getResource("hello-view.fxml"));
    Scene scene = new Scene(fxmlLoader.load(), 676, 503);
    
    // 获取FXMLLoader绑定的真实控制器实例
    HelloController controller = fxmlLoader.getController();
    scene.setOnKeyPressed(controller::moveCharacter);
    
    stage.setTitle("For You");
    stage.setScene(scene);
    stage.show();
}

额外注意事项 #

• 确保FXML文件中ImageView的fx:id和控制器中@FXML注解的变量名完全一致（大小写敏感）。
• initialize方法必须加@FXML注解，这样FXMLLoader加载时会自动调用它初始化图片。

内容的提问来源于stack exchange，提问作者Zhe Hin Yap