lambda表达式中使用的变量应该是final或有效final

当我尝试使用calTz时，它显示了这个错误。

private TimeZone extractCalendarTimeZoneComponent(Calendar cal, TimeZone calTz) {
    try {
        cal.getComponents().getComponents("VTIMEZONE").forEach(component -> {
            VTimeZone v = (VTimeZone) component;
            v.getTimeZoneId();
            if (calTz == null) {
                calTz = TimeZone.getTimeZone(v.getTimeZoneId().getValue());
            }
        });
    } catch (Exception e) {
        log.warn("Unable to determine ical timezone", e);
    }
    return null;
}

当前回答

final变量意味着它只能被实例化一次。在Java中，你不能在lambda和匿名内部类中重新分配非最终局部变量。

你可以用旧的for-each循环重构你的代码:

private TimeZone extractCalendarTimeZoneComponent(Calendar cal,TimeZone calTz) {
    try {
        for(Component component : cal.getComponents().getComponents("VTIMEZONE")) {
        VTimeZone v = (VTimeZone) component;
           v.getTimeZoneId();
           if(calTz==null) {
               calTz = TimeZone.getTimeZone(v.getTimeZoneId().getValue());
           }
        }
    } catch (Exception e) {
        log.warn("Unable to determine ical timezone", e);
    }
    return null;
}

即使我不明白这段代码的某些部分:

你调用v.getTimeZoneId();不使用它的返回值赋值calTz = TimeZone.getTimeZone(v.getTimeZoneId().getValue());你不会修改最初传递的calTz，也不会在这个方法中使用它你总是返回null，为什么不把void设为返回类型呢?

希望这些建议也能帮助你提高。

2016-01-18 23:43:49

其他回答

lambda表达式中使用的变量应该是final或有效的final，但可以将值赋给最终的单元素数组。

private TimeZone extractCalendarTimeZoneComponent(Calendar cal, TimeZone calTz) {
    try {
        TimeZone calTzLocal[] = new TimeZone[1];
        calTzLocal[0] = calTz;
        cal.getComponents().get("VTIMEZONE").forEach(component -> {
            TimeZone v = component;
            v.getTimeZoneId();
            if (calTzLocal[0] == null) {
                calTzLocal[0] = TimeZone.getTimeZone(v.getTimeZoneId().getValue());
            }
        });
    } catch (Exception e) {
        log.warn("Unable to determine ical timezone", e);
    }
    return null;
}

2020-04-08 14:41:22

从lambda中，你不能得到指向非final对象的引用。您需要从lamda外部声明一个最终包装器来保存您的变量。

我已经添加了最终的“引用”对象作为这个包装器。

private TimeZone extractCalendarTimeZoneComponent(Calendar cal,TimeZone calTz) {
    final AtomicReference<TimeZone> reference = new AtomicReference<>();

    try {
       cal.getComponents().getComponents("VTIMEZONE").forEach(component->{
        VTimeZone v = (VTimeZone) component;
           v.getTimeZoneId();
           if(reference.get()==null) {
               reference.set(TimeZone.getTimeZone(v.getTimeZoneId().getValue()));
           }
           });
    } catch (Exception e) {
        //log.warn("Unable to determine ical timezone", e);
    }
    return reference.get();
}

2016-01-18 23:41:11

如果没有必要修改变量，那么对于这类问题，一般的解决方法就是修改变量提取使用lambda和在method-parameter上使用final关键字的部分代码。

2017-12-14 20:32:57

在您的示例中，您可以使用简单的for循环将forEach替换为lamdba，并自由修改任何变量。或者重构代码，这样就不需要修改任何变量。但是，为了完整起见，我将解释这个错误的含义以及如何解决它。

Java 8语言规范，§15.27.2:

在lambda表达式中使用但未声明的任何局部变量、形式形参或异常形参必须声明为final或有效的final(§4.12.4)，否则尝试使用时将发生编译时错误。

基本上，您不能在lambda(或本地/匿名类)内修改局部变量(在本例中为calTz)。要在Java中实现这一点，必须使用可变对象并从lambda中修改它(通过final变量)。可变对象的一个例子是一个元素数组:

private TimeZone extractCalendarTimeZoneComponent(Calendar cal, TimeZone calTz) {
    TimeZone[] result = { null };
    try {
        cal.getComponents().getComponents("VTIMEZONE").forEach(component -> {
            ...
            result[0] = ...;
            ...
        }
    } catch (Exception e) {
        log.warn("Unable to determine ical timezone", e);
    }
    return result[0];
}

2016-01-18 23:41:54