JavaFX中如何为数据库动态生成的MenuItem绑定点击事件

你之前尝试在menu.getItems().add(new MenuItem(xxx))这类链式调用后面直接追加事件绑定方法，本质是搞错了方法的返回值类型：

• JavaFX菜单的getItems()返回的是ObservableList<MenuItem>，它的add()方法返回值是boolean类型（表示添加是否成功），并不是你创建的MenuItem对象
• 在一个boolean类型的返回值后面调用setOnAction()/addEventHandler()这类属于MenuItem的方法，自然会触发类型不匹配的编译错误，Eclipse因为识别不出你链式调用的类型错配，才会乱推荐换成addAll/其他方法。

针对动态生成的菜单项，有两种零报错的写法可以选，对初学者最友好的是第一种分步写法：

写法1：分步创建，逻辑最清晰 #

完全绕开链式调用的类型坑，在循环里按顺序做三件事：

1. 单独创建MenuItem实例
2. 给实例绑定点击事件
3. 把绑好事件的实例加到子菜单的列表里
   示例代码：

// 获取数据库返回的配对名称列表
ArrayList<String> pairNameList = DatabaseMethods.getPairNames();
for (String pairName : pairNameList) {
    // 1. 单独初始化菜单项
    MenuItem pairMenuItem = new MenuItem(pairName);
    // 2. 给菜单项绑定点击事件
    pairMenuItem.setOnAction(event -> {
        // 这里写点击后要执行的业务逻辑，比如加载对应配对的统计数据
        loadPairStatsView(pairName);
    });
    // 3. 把菜单项加到Choose pair子菜单下
    choosePairSubMenu.getItems().add(pairMenuItem);
}

小提示：循环里用增强for的pairName变量天然满足JavaFX lambda要求的“有效final”规则，不会触发编译报错，不需要额外做特殊处理。

写法2：流式紧凑写法 #

如果想把代码写得更简洁，可以用Stream流的方式组织逻辑，避免链式调用的类型错误：

DatabaseMethods.getPairNames().stream()
    // 把每个配对名称转成绑好事件的MenuItem
    .map(pairName -> {
        MenuItem item = new MenuItem(pairName);
        item.setOnAction(e -> loadPairStatsView(pairName));
        return item;
    })
    // 把转好的MenuItem逐个加到子菜单列表里
    .forEach(choosePairSubMenu.getItems()::add);

• 不要把事件绑定的逻辑直接塞在add()方法的参数括号里做链式拼接，本质上add()只接收MenuItem类型的参数，不接收链式调用后的返回值
• 如果循环里的变量要在lambda事件里使用，不要在循环过程中对这个变量重新赋值，否则会触发“变量未满足有效final要求”的编译错误
• Eclipse弹出的替换addAll的提示不用理会，这是工具识别不到类型错配的误提示，按分步逻辑写完就会自动消失

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