JavaFX点击按钮隐藏VBox后Stage自动调整大小及空指针问题排查

首先，咱们来定位你遇到的NullPointerException根源：你的primaryStage变量在控制器实例中是null！

为什么会这样？因为你把Application类同时当作了FXML控制器，而FXMLLoader.load()方法会自动创建一个新的FunctionOwnerFX实例作为控制器——这个实例和start()方法中赋值primaryStage的那个Application实例完全是两个独立的对象。所以当你在splitPaneControl()里调用primaryStage.sizeToScene()时，这个变量根本没被初始化，自然抛出空指针异常。

下面给你几个可行的解决方案：

方案1：通过控件获取Stage（最简洁） #

不需要依赖primaryStage成员变量，直接从按钮控件获取当前窗口的Stage：

@FXML protected void splitPaneControl(){
    isHidden = !isHidden;
    if (isHidden) {
        vbox.getChildren().remove(libraryVbox);
        // 通过按钮获取当前Stage
        Stage stage = (Stage) btn2.getScene().getWindow();
        stage.sizeToScene();
        btn2.setText(SHOW);
    } else {
        vbox.getChildren().add(libraryVbox);
        Stage stage = (Stage) btn2.getScene().getWindow();
        stage.sizeToScene();
        btn2.setText(HIDE);
    }
}

这样你甚至可以删掉类里的primaryStage成员变量，代码更解耦。

方案2：让Application实例作为控制器（避免重复实例） #

如果你坚持要把Application类当作控制器，需要手动创建FXMLLoader并指定控制器为当前实例，同时去掉FXML文件中的fx:controller属性：

@Override
public void start(Stage stage) throws Exception{
    primaryStage = stage;
    // 手动创建FXMLLoader，指定控制器为当前Application实例
    FXMLLoader loader = new FXMLLoader(getClass().getResource("new_function_owner.fxml"));
    loader.setController(this);
    Parent root = loader.load();
    
    Scene scene = new Scene(root);
    stage.setTitle("New Function Owner");
    stage.setScene(scene);
    stage.sizeToScene();
    stage.show();
}

这样primaryStage变量就会被正确赋值到控制器实例中，调用sizeToScene()就不会报错了。

方案3：拆分Application和控制器（最佳实践） #

这是JavaFX开发的标准做法：把Application类和FXML控制器分开，避免职责混乱。比如创建一个独立的FunctionOwnerController类，然后在start方法中获取控制器实例并传递Stage：

// 独立的控制器类
public class FunctionOwnerController {
    public static String HIDE = "Hide Libraries <<<";
    public static String SHOW = "Show Libraries >>>";
    private boolean isHidden = false;
    @FXML private TextField textfield;
    @FXML private Button btn2;
    @FXML private VBox vbox;
    @FXML private VBox libraryVbox;
    private Stage stage;

    // 提供方法设置Stage
    public void setStage(Stage stage) {
        this.stage = stage;
    }

    @FXML protected void splitPaneControl(){
        isHidden = !isHidden;
        if (isHidden) {
            vbox.getChildren().remove(libraryVbox);
            stage.sizeToScene();
            btn2.setText(SHOW);
        } else {
            vbox.getChildren().add(libraryVbox);
            stage.sizeToScene();
            btn2.setText(HIDE);
        }
    }
}

// Application类
public class FunctionOwnerFX extends Application{
    @Override
    public void start(Stage stage) throws Exception{
        FXMLLoader loader = new FXMLLoader(getClass().getResource("new_function_owner.fxml"));
        Parent root = loader.load();
        // 获取控制器实例并传递Stage
        FunctionOwnerController controller = loader.getController();
        controller.setStage(stage);
        
        Scene scene = new Scene(root);
        stage.setTitle("New Function Owner");
        stage.setScene(scene);
        stage.sizeToScene();
        stage.show();
    }

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

这种方式代码结构更清晰，后期维护也更方便。

另外，还有个小优化：你可以用libraryVbox.setVisible(!isHidden)和libraryVbox.setManaged(!isHidden)来替代移除/添加节点的操作，这样节点不会被从布局中移除，只是隐藏且不占布局空间，性能和逻辑上更高效哦~

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