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

时间:2014-12-21 18:24:33

标签: java lambda

我有一个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

3 个答案:

答案 0 :(得分:15)

您只需将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,这意味着必须为它们分配一个值一次)或&#34;有效决赛&#34; (这基本上意味着你可以让它们最终而不需要对代码进行任何其他更改)。

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

答案 1 :(得分:3)

您遇到的错误意味着您在lambda表达式主体内访问的每个变量都必须是最终的或有效的最终变量。有关差异,请在此处查看此答案:Difference between final and effectively final

代码中的问题是以下变量

String readln2 = null;

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

解决此问题的最简单方法是使用包装器对象,在本例中为StringProperty而不是String。这个包装器只被分配一次,因此实际上是最终的:

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

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

答案 2 :(得分: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);
  }

希望这会有所帮助..