Java Swing战舰游戏：JButton点击事件传递矩阵坐标报错

问题根源 #

你遇到的编译报错核心来自Java语法限制：匿名内部类访问方法局部变量时，变量必须声明为final或为有效final（即初始化后从未被重新赋值）。你双层循环里的i和j每次迭代都会更新值，不属于有效final变量，所以无法直接在ActionListener的匿名内部类中引用。

可行实现方案 #

以下三种方案都可以实现点击按钮时获取对应棋盘坐标，按改造成本从低到高排序：

方案1：循环内声明final临时变量（改造成本最低） #

在每次循环迭代时，用final修饰的临时变量存储当前轮次的i、j值，这类临时变量在单次迭代里值固定，符合有效final要求，可以直接被内部类访问。
修改后的按钮绑定代码如下：

for (int i = 0; i < 10; i++) {
     for (int j = 0; j < 10; j++) {
          // 声明final临时变量存储当前坐标
          final int currentI = i;
          final int currentJ = j;

          tabuleiro1[i][j] = new JButton();
          tabuleiro1[i][j].addActionListener(new ActionListener() {
                @Override
                public void actionPerformed(ActionEvent e) {
                     // 传入固定的临时坐标值
                     selected(((JButton)e.getSource()), currentI, currentJ);
                }
           });

          tabuleiro2[i][j] = new JButton();
          // 玩家2的按钮同理绑定，注意区分对应玩家实例，不要混用player1的逻辑
          final int p2I = i;
          final int p2J = j;
          tabuleiro2[i][j].addActionListener(new ActionListener() {
                @Override
                public void actionPerformed(ActionEvent e) {
                     selectedForPlayer2(((JButton)e.getSource()), p2I, p2J);
                }
           });
     }
}

方案2：给JButton组件绑定自定义坐标属性（鲁棒性最高） #

自定义一个继承JButton的棋盘按钮类，自带行、列坐标属性，点击时直接从触发事件的按钮对象上拿坐标，完全不需要依赖外部变量，后续就算调整组件布局顺序也不会出现坐标错位问题。
实现示例：

// 自定义棋盘按钮类
class BoardButton extends JButton {
    private final int row;
    private final int col;

    public BoardButton(int row, int col) {
        this.row = row;
        this.col = col;
    }

    public int getRow() {
        return row;
    }

    public int getCol() {
        return col;
    }
}

// 初始化按钮时直接使用自定义类
for (int i = 0; i < 10; i++) {
     for (int j = 0; j < 10; j++) {
          tabuleiro1[i][j] = new BoardButton(i, j);
          tabuleiro1[i][j].addActionListener(new ActionListener() {
                @Override
                public void actionPerformed(ActionEvent e) {
                     BoardButton clickedBtn = (BoardButton) e.getSource();
                     // 直接从按钮对象取坐标，不需要外部传参
                     selected(clickedBtn, clickedBtn.getRow(), clickedBtn.getCol());
                }
           });
          // tabuleiro2的初始化逻辑完全一致，绑定对应玩家的处理逻辑即可
     }
}

方案3：点击时遍历按钮数组反查坐标（不推荐） #

点击事件触发后拿到源按钮，遍历整个tabuleiro1/tabuleiro2二维数组，对比找到当前按钮对应的i、j下标。这个方法不需要改初始化逻辑，但每次点击都要遍历最多100个组件，性能差，后续如果数组和组件对应关系调整很容易出bug，仅作为补充方案提及。

配套逻辑优化 #

你现有的selected方法可以补充棋盘状态同步逻辑，点击后同步把矩阵里对应位置的值改成-1（已射击状态），避免重复点击触发无效逻辑：

public void selected(JButton B, int i, int j){
    int[][] boardPos = player.getBoard().getPositions();
    if(boardPos[i][j] == -1){
        System.out.println("Already shoted");
        return;
    }
    if(boardPos[i][j] == 0){
        B.setBackground(Color.black);
    } else if(boardPos[i][j] == 1){
        B.setBackground(Color.red);
    }
    // 标记当前位置为已射击状态
    boardPos[i][j] = -1;
}

内容的提问来源于stack exchange，提问作者Mathguy on CS