Zmienna używana w wyrażeniu lambda powinna być ostateczna lub faktycznie ostateczna

Zmienna używana w wyrażeniu lambda powinna być ostateczna lub faktycznie ostateczna

Kiedy próbuję użyć calTz, pokazuje ten błąd.

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

A finalzmienne oznacza, że może być instancja tylko jeden raz. w Javie nie można używać zmiennych innych niż końcowe w lambdzie ani w anonimowych klasach wewnętrznych.

Możesz refaktoryzować swój kod za pomocą starej pętli 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;
}

Nawet jeśli nie rozumiem niektórych fragmentów tego kodu:

• wywołujesz a v.getTimeZoneId();bez używania wartości zwracanej
• z przypisaniem calTz = TimeZone.getTimeZone(v.getTimeZoneId().getValue());nie modyfikujesz pierwotnie przekazanego calTzi nie używasz go w tej metodzie
• Zawsze wracasz null, dlaczego nie ustawisz voidjako typu powrotu?

Mam nadzieję, że te wskazówki pomogą Ci poprawić.

Chociaż inne odpowiedzi potwierdzają ten wymóg, nie wyjaśniają, dlaczego to wymaganie istnieje.

JLS wspomina, dlaczego w §15.27.2 :

Ograniczenie do efektywnych zmiennych końcowych uniemożliwia dostęp do dynamicznie zmieniających się zmiennych lokalnych, których wychwycenie prawdopodobnie spowodowałoby problemy z współbieżnością.

Aby zmniejszyć ryzyko błędów, postanowili zapewnić, że przechwycone zmienne nigdy nie będą mutowane.

Z lambdy nie można uzyskać odniesienia do niczego, co nie jest ostateczne. Musisz zadeklarować końcowe opakowanie spoza lamdy, aby przechowywać zmienną.

Dodałem ostatni obiekt „referencyjny” jako opakowanie.

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 ma nową koncepcję zwaną zmienną „Efektywnie ostateczna”. Oznacza to, że nieostateczna zmienna lokalna, której wartość nigdy się nie zmienia po inicjalizacji, nazywana jest „Efektywnie końcową”.

Ta koncepcja została wprowadzona, ponieważ przed wersją Java 8 nie mogliśmy używać nieostatecznej zmiennej lokalnej w klasie anonimowej . Jeśli chcesz mieć dostęp do zmiennej lokalnej w klasie anonimowej , musisz to zrobić.

Wraz z wprowadzeniem lambdy to ograniczenie zostało złagodzone. Stąd potrzeba, aby zmienna lokalna była ostateczna, jeśli nie zostanie zmieniona po zainicjowaniu, ponieważ lambda sama w sobie jest niczym innym jak klasą anonimową.

Java 8 zdała sobie sprawę z bólu deklarowania zmiennej lokalnej jako końcowej za każdym razem, gdy programista użył lambdy, wprowadziła tę koncepcję i sprawiła, że ​​nie było potrzeby, aby zmienne lokalne były ostateczne. Więc jeśli widzisz, że reguła dla klas anonimowych się nie zmieniła, to po prostu nie musisz wpisywać finalsłowa kluczowego za każdym razem, gdy używasz lambd.

Znalazłem dobre wyjaśnienie tutaj

W twoim przykładzie możesz zastąpić forEachlamdba prostą forpętlą i dowolnie modyfikować dowolną zmienną. Lub prawdopodobnie zrefaktoryzuj swój kod, aby nie trzeba było modyfikować żadnych zmiennych. Jednak wyjaśnię dla kompletności, co oznacza błąd i jak go obejść.

Specyfikacja języka Java 8, §15.27.2 :

Każda zmienna lokalna, parametr formalny lub parametr wyjątku używany, ale nie zadeklarowany w wyrażeniu lambda, musi być zadeklarowany jako ostateczny lub faktycznie ostateczny ( §4.12.4 ), albo w przypadku próby użycia wystąpi błąd w czasie kompilacji.

Zasadniczo nie można modyfikować zmiennej lokalnej ( calTzw tym przypadku) z poziomu lambda (lub klasy lokalnej / anonimowej). Aby to osiągnąć w Javie, musisz użyć mutowalnego obiektu i zmodyfikować go (poprzez końcową zmienną) z lambda. Przykładem modyfikowalnego obiektu byłaby tutaj tablica jednego elementu:

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

jeśli modyfikacja zmiennej nie jest konieczna, ogólnym obejściem tego rodzaju problemu byłoby wyodrębnienie części kodu, która używa lambda i użycie słowa kluczowego final w metodzie-parametrze.

Zmienna używana w wyrażeniu lambda powinna być ostateczną lub faktycznie ostateczną, ale można przypisać wartość do końcowej tablicy jednoelementowej.

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