Змінна, що використовується в лямбда-експресії, повинна бути остаточною або фактично остаточною

Змінна, що використовується в лямбда-експресії, повинна бути остаточною або фактично остаточною

Коли я намагаюся його використовувати 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 ви не можете використовувати не остаточні змінні в лямбда, а також в анонімних внутрішніх класах.

Ви можете перефактурувати свій код зі старим циклом для кожного:

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;
}

Навіть якщо я не розумію деяких фрагментів цього коду:

• ви викликаєте a, v.getTimeZoneId();не використовуючи його зворотного значення
• з призначенням calTz = TimeZone.getTimeZone(v.getTimeZoneId().getValue());ви не змінюєте початково переданий calTzі не використовуєте його в цьому методі
• Ви завжди повертаєтесь null, чому б вам не встановити voidяк тип повернення?

Сподіваюся, що ці поради допоможуть вам покращитись.

Хоча інші відповіді підтверджують вимогу, вони не пояснюють, чому ця вимога існує.

JLS зазначає, чому в § 15.27.2 :

Обмеження ефективних кінцевих змінних забороняє доступ до динамічно мінливих локальних змінних, захоплення яких, ймовірно, спричинить проблеми одночасності.

Щоб знизити ризик помилок, вони вирішили гарантувати, що захоплені змінні ніколи не змінюються.

Від лямбда ви не можете отримати посилання на те, що не є остаточним. Вам потрібно оголосити остаточну обгортку поза ламдою, щоб зберегти свою змінну.

Я додав остаточний 'посилальний' об'єкт як цю обгортку.

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();
}   

У Java 8 є нова концепція, що називається "Ефективно остаточна" змінна. Це означає, що нефінальна локальна змінна, значення якої ніколи не змінюється після ініціалізації, називається "Ефективно кінцевою".

Ця концепція була введена тому, що до Java 8 ми не могли використовувати не остаточну локальну змінну в анонімному класі . Якщо ви хочете отримати доступ до локальної змінної в анонімному класі , ви повинні зробити її остаточною.

Коли ввели лямбда, це обмеження було полегшено. Отже, необхідність зробити локальну змінну остаточною, якщо вона не змінюється, коли вона ініціалізується як лямбда, сама по собі є не що інше, як анонімний клас.

Java 8 усвідомила, що оголошує локальну змінну остаточним щоразу, коли розробник використовував лямбда, вводив цю концепцію та робив непотрібним робити локальні змінні остаточними. Отже, якщо ви бачите, що правило для анонімних класів не змінилося, просто не потрібно писати finalключове слово щоразу, коли використовуєте лямбда.

Я знайшов гарне пояснення тут

У вашому прикладі ви можете замінити forEachlamdba простим forциклом і вільно змінювати будь-яку змінну. Або, можливо, перефактуруйте свій код, щоб не потрібно змінювати жодні змінні. Однак я поясню для повноти, що означає помилка та як її вирішити.

Специфікація мови Java 8, § 15.27.2 :

Будь-який локальний змінний, формальний параметр або параметр винятку, який використовується, але не оголошений у лямбда-виразі, повинен або бути оголошений остаточним, або бути фактично остаточним ( §4.12.4 ), або помилка часу компіляції виникає при спробі використання.

В основному ви не можете змінювати локальну змінну ( calTzв даному випадку) зсередини лямбда (або локальний / анонімний клас). Для досягнення цього в Java вам потрібно використовувати об'єкт, що змінюється, і змінити його (через остаточну змінну) з лямбда. Одним із прикладів об'єкта, що змінюється, може бути масив одного елемента:

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];
}

якщо не потрібно змінювати змінну, то загальним способом вирішення цього виду проблеми було б вилучення частини коду, яка використовує лямбда і використання остаточного ключового слова методу-параметра.

Змінна, що використовується в лямбда-виразі, повинна бути остаточним або фактично остаточним, але ви можете призначити значення остаточному масиву одного елемента.

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;
}