从lambda表达式引用的局部变量必须是final或有效final

Question

我有一个JavaFX 8程序（用于JavaFXPorts交叉平台），几乎可以做我想做的事情，但又短了一步。该程序读取文本文件，对行进行计数以建立随机范围，从该范围中选取一个随机数，然后读取该行以进行显示。

The error is: local variables referenced from a lambda expression must be final or effectively final
        button.setOnAction(e -> l.setText(readln2));

我对java有点陌生，但似乎我是否使用Lambda在Label l中显示下一个随机行，我的button.setOnAction（e-

有什么想法，我可以调整我必须简单地使var readln2显示的下一个值，每次我按下屏幕上的按钮？

提前感谢，这是我的代码:

String readln2 = null;
in = new BufferedReader(new FileReader("/temp/mantra.txt"));
long linecnt = in.lines().count();
int linenum = rand1.nextInt((int) (linecnt - Low)) + Low;
try {
    //open a bufferedReader to file 
    in = new BufferedReader(new FileReader("/temp/mantra.txt"));

    while (linenum > 0) {
        //read the next line until the specific line is found
        readln2 = in.readLine();
        linenum--;
    }

    in.close();
} catch (IOException e) {
    System.out.println("There was a problem:" + e);
}

Button button = new Button("Click the Button");
button.setOnAction(e -> l.setText(readln2));
//  error: local variables referenced from a lambda expression must be final or effectively final




Answer 1

我经常以这种方式将外部对象传递到接口实现中：1.创建一些对象持有者，2.用一些所需的状态设置此对象持有者，3.更改对象持有者中的内部变量，4.获取这些变量并使用它们。

这是Vaadin的一个例子：

Object holder : 
    public class ObjectHolder<T> {
    private T obj;
    public ObjectHolder(T obj) {
        this.obj = obj;
    }
    public T get() {
        return obj;
    }
    public void set(T obj) {
        this.obj = obj;
    }
}

我想传递外部定义的按钮标题，如下所示：

String[] bCaption = new String[]{"Start", "Stop", "Restart", "Status"};
String[] commOpt = bCaption;

接下来，我有一个for循环，希望动态创建按钮，并传递如下值：

for (Integer i = 0; i < bCaption.length; i++) {
    ObjectHolder<Integer> indeks = new ObjectHolder<>(i);
    b[i] = new Button(bCaption[i], 
        (Button.ClickEvent e) -> {
            remoteCommand.execute(
                cred, 
                adresaServera, 
                 comm + " " + commOpt[indeks.get()].toLowerCase()
             );
         }
        );

        b[i].setWidth(70, Unit.PIXELS);
        commandHL.addComponent(b[i]);
        commandHL.setComponentAlignment(b[i], Alignment.MIDDLE_CENTER);
  }

希望这有帮助。。

Answer 2

遇到的错误意味着在lambda表达式体中访问的每个变量必须是final或实际上是final。有关差异，请参见此处的答案：最终和有效最终之间的差异

您的代码中的问题是以下变量

String readln2 = null;

变量稍后会被声明和分配，编译器无法检测它是被分配一次还是多次，因此它实际上不是最终的。

解决这个问题最简单的方法是使用一个包装对象，在这种情况下是StringProperty而不是String。这个包装器只被赋值一次，因此实际上是最终的:

StringProperty readln2 = new SimpleStringProperty();
readln2.set(in.readLine());
button.setOnAction(e -> l.setText(readln2.get()));

我缩短了代码，只显示了相关的部分..

Answer 3

匿名用户

您可以将< code>readln2的值复制到< code>final变量中:

    final String labelText = readln2 ;
    Button button = new Button("Click the Button");
    button.setOnAction(e -> l.setText(labelText));

如果要每次都抓取新的随机行，则可以缓存感兴趣的行，然后在事件处理程序中选择一个随机行：

Button button = new Button("Click the button");
Label l = new Label();
try {
    List<String> lines = Files.lines(Paths.get("/temp/mantra.txt"))
        .skip(low)
        .limit(high - low)
        .collect(Collectors.toList());
    Random rng = new Random();
    button.setOnAction(evt -> l.setText(lines.get(rng.nextInt(lines.size()))));
} catch (IOException exc) {
    exc.printStackTrace();
}
// ...

或者，您可以直接在事件处理程序中重新读取该文件。第一种技术（快得多），但可能会消耗大量内存;第二个不会在内存中存储任何文件内容，但每次按下按钮时都会读取一个文件，这可能会导致UI无响应。

您得到的错误基本上告诉您错误所在：您可以从lambda表达式内部访问的唯一局部变量是＜code＞final＜/code＞（声明为＜code＞final＜/code＞，这意味着它们必须被赋值一次）或“有效final”（这基本上意味着您可以使它们成为final，而无需对代码进行任何其他更改）。

您的代码无法编译，因为< code>readln2被多次赋值(在循环内)，所以它不能被声明为< code>final。因此你不能在lambda表达式中访问它。在上面的代码中，lambda中访问的唯一变量是< code>l 、< code>lines和< code>rng，它们都是“有效的最终变量”,因为它们只被赋值一次。(您可以将它们声明为final，代码仍然可以编译。)