JavaFX与继承问题求助：MVC架构下事件控制器失效

你的核心问题在于错误地通过继承实现MVC耦合，并且实际运行的是TestGUI实例而非TestController，导致重写的doAction方法从未被调用。以下是具体问题拆解和修复方案：

问题根源 #

1. 你让TestController继承TestGUI，这完全违背了MVC的职责分离原则——Controller不应继承View，两者是关联关系而非继承关系。
2. JavaFX启动时会创建TestGUI的实例（因为main方法在TestGUI中），而非TestController，所以绑定的this::doAction始终指向TestGUI中的空实现，不会触发子类的重写方法。

修复方案 #

1. 重构为标准MVC结构 #

将View（TestGUI）、Controller（TestController）完全解耦，通过依赖注入关联两者：

启动类（单独抽离，负责初始化）

package testNinja;

import javafx.application.Application;
import javafx.stage.Stage;

public class Main extends Application {
    public static void main(String[] args) {
        launch(args);
    }

    @Override
    public void start(Stage stage) {
        // 实例化Controller
        TestController controller = new TestController();
        // 将Controller传入View
        TestGUI gui = new TestGUI(controller);
        gui.init(stage);
    }
}

View层（TestGUI，仅负责界面渲染）

package testNinja;

import javafx.event.ActionEvent;
import javafx.scene.control.Menu;
import javafx.scene.control.MenuBar;
import javafx.scene.control.MenuItem;
import javafx.scene.layout.VBox;
import javafx.stage.Stage;
import javafx.scene.Scene;

public class TestGUI {
    public Stage window;
    public Menu fileMenu;
    public MenuItem fileNewConnection;
    public MenuBar menuBar;
    private TestController controller;

    // 通过构造函数注入Controller
    public TestGUI(TestController controller) {
        this.controller = controller;
    }

    public void init(Stage stage) {
        window = stage;
        fileMenu = new Menu("File");
        fileNewConnection = new MenuItem("New Connection");
        fileMenu.getItems().add(fileNewConnection);
        // 将事件绑定到Controller的处理方法
        fileNewConnection.setOnAction(controller::doAction);
        
        menuBar = new MenuBar();
        menuBar.getMenus().add(fileMenu);

        VBox topPane = new VBox(16, menuBar);
        Scene scene = new Scene(topPane, 500, 300);
        
        stage.setTitle("Test Ninja");
        stage.setScene(scene);
        stage.show();
    }
}

Controller层（TestController，仅负责事件处理）

package testNinja;

import javafx.event.ActionEvent;
import javafx.scene.control.MenuItem;

public class TestController {
    public void doAction(ActionEvent evt) {
        if (evt.getSource() instanceof MenuItem menuItem) {
            System.out.println("You clicked the " + menuItem.getText() + " Menu Item ");
        }
    }
}

2. 简化版（不抽离启动类） #

如果不想单独写启动类，也可以直接在TestGUI的start方法中实例化Controller并绑定事件：
修改TestGUI的start方法：

@Override
public void start(Stage stage) {
    window = stage;
    // 实例化Controller
    TestController controller = new TestController();
    
    fileMenu = new Menu("File");
    fileNewConnection = new MenuItem("New Connection");
    fileMenu.getItems().add(fileNewConnection);
    // 绑定Controller的处理方法
    fileNewConnection.setOnAction(controller::doAction);
    
    menuBar = new MenuBar();
    menuBar.getMenus().add(fileMenu);

    VBox topPane = new VBox(16, menuBar);
    Scene scene = new Scene(topPane, 500, 300);
    
    stage.setTitle("Test Ninja");
    stage.setScene(scene);
    stage.show();
}

同时删除TestController的继承关系，保持独立类即可。

关键改进点 #

• 取消TestController对TestGUI的继承，实现职责分离
• 将事件绑定目标从this::doAction改为controller::doAction，确保触发Controller的处理逻辑
• 遵循MVC设计：View负责界面展示，Controller负责业务/事件处理，两者通过关联而非继承交互

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