从lambda表达式引用的局部变量必须是最终的或有效的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 -> l.setText(readln2));行期待静态值.

我每次按下屏幕上的按钮时,如何调整我所拥有的只需创建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

您可以将值复制readln2到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表达式中访问的唯一局部变量是final(声明final,这意味着它们必须被分配一个值一次)或"有效最终"(这基本上意味着你可以使它们成为最终的,而无需对代码进行任何其他更改)

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

Answer 2

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

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

String readln2 = null;

该变量稍后被声明和分配，编译器无法检测它是否被分配一次或多次，因此它不是有效的最终变量。

解决此问题的最简单方法是使用包装对象，在本例中是 StringProperty 而不是 String。该包装器仅分配一次，因此实际上是最终的：

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

我缩短了代码以仅显示相关部分。