按下按钮时访问变量

Question

我正在使用javafx编写一个小型棋盘游戏，我已经为棋盘上的每个“单元格”创建了一个网格。

但是，在按下单元格按钮时，我在更改单元格颜色时遇到了一些问题。

下面是我得到的一个错误:“Error:(44，43) java:从内部类引用的局部变量必须是final或有效的final”

但是，我不能将变量设为final——因为变量是for循环的一部分，如下所示:

for (int columns = 0; columns < 8; columns++) {
        for (int rows = 0; rows < 8; rows++) {
            positions[columns][rows] = new Button();

            positions[columns][rows].setText("X:" + columns + " Y:" + rows); // current location text (temp)

            positions[columns][rows].setMinHeight(cellDimensions);
            positions[columns][rows].setMaxHeight(cellDimensions);
            positions[columns][rows].setMinWidth(cellDimensions);
            positions[columns][rows].setMaxWidth(cellDimensions);

            String zoneColour = getColourBoard(columns, rows);
            positions[columns][rows].setStyle("-fx-background-color: " + zoneColour + "; ");

            positions[columns][rows].setOnAction(new EventHandler<ActionEvent>() {
                @Override
                public void handle(ActionEvent event) {
                            positions[columns][rows].setStyle("-fx-background-color: #FFA500; ");
                        }
                    });

            grid.add(positions[columns][rows], columns, rows);

        }
    }

这里真正的问题是，我需要向作为按钮数组一部分的按钮添加一个事件处理程序，但是似乎不能访问所述按钮数组的计数器变量。

Answer 1

如果问题是关于句柄方法中的位置[列][行]引用，您可以尝试用event.getSource（）替换它，将其转换为正确的类型，以删除对其中位置的引用。

Answer 2

最简单的方法是在每次迭代时立即将循环计数器设置为最终变量。

for (int columns = 0; columns < 8; columns++) {
    final int columnNum = columns;
    for (int rows = 0; rows < 8; rows++) {
        final int rowNum = rows;
        // now use columnNum and rowNum
    }
}

Answer 3

您之所以会遇到这个问题，是因为列和行在整个循环中都会发生变化。您可以创建这些的最终实例，但创建一个＜code＞最终按钮＜/code＞以在侦听器中引用更有意义。

for (int columns = 0; columns < 8; columns++) {
    for (int rows = 0; rows < 8; rows++) {
        positions[columns][rows] = new Button();
        positions[columns][rows].setText("X:" + columns + " Y:" + rows); // current location text (temp)
        positions[columns][rows].setMinHeight(cellDimensions);
        positions[columns][rows].setMaxHeight(cellDimensions);
        positions[columns][rows].setMinWidth(cellDimensions);
        positions[columns][rows].setMaxWidth(cellDimensions);

        String zoneColour = getColourBoard(columns, rows);
        positions[columns][rows].setStyle("-fx-background-color: " + zoneColour + "; ");

        final Button b = positions[columns][rows];
        // final int c = columns, r = rows;
        positions[columns][rows].setOnAction(new EventHandler<ActionEvent>() {
            public void handle(ActionEvent event) {
                // Could use c and r above if needed.
                // positions[c][r]
                b.setStyle("-fx-background-color: #FFA500; ");
            }
        });
        grid.add(positions[columns][rows], columns, rows);
    }
}