La variabile utilizzata nell'espressione lambda deve essere finale o effettivamente definitiva

La variabile utilizzata nell'espressione lambda deve essere finale o effettivamente definitiva

Quando provo a usarlo calTzmostra questo errore.

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

Una finalvariabile significa che può essere istanziata una sola volta. in Java non è possibile utilizzare variabili non finali in lambda e in classi interne anonime.

Puoi riformattare il tuo codice con il vecchio ciclo 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;
}

Anche se non ho la sensazione di alcuni pezzi di questo codice:

• chiami a v.getTimeZoneId();senza usare il suo valore di ritorno
• con l'assegnazione calTz = TimeZone.getTimeZone(v.getTimeZoneId().getValue());non modifichi l'originale passato calTze non lo usi in questo metodo
• Ritorni sempre null, perché non imposti voidcome tipo di ritorno?

Spero anche che questi suggerimenti ti aiutino a migliorare.

Sebbene altre risposte dimostrino il requisito, non spiegano perché esiste il requisito.

Il JLS menziona perché in § 15.27.2 :

La restrizione a variabili finali efficaci proibisce l'accesso a variabili locali che cambiano dinamicamente, la cui acquisizione introdurrebbe probabilmente problemi di concorrenza.

Per ridurre il rischio di bug, hanno deciso di garantire che le variabili acquisite non siano mai mutate.

Da una lambda, non puoi ottenere un riferimento a qualcosa che non sia definitivo. È necessario dichiarare un wrapper finale dall'esterno della lamda per contenere la variabile.

Ho aggiunto l'oggetto finale "di riferimento" come questo wrapper.

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 ha un nuovo concetto chiamato variabile "efficacemente finale". Significa che una variabile locale non finale il cui valore non cambia mai dopo l'inizializzazione è chiamata "efficacemente finale".

Questo concetto è stato introdotto perché prima di Java 8 non potevamo usare una variabile locale non finale in una classe anonima . Se vuoi avere accesso a una variabile locale in classe anonima , devi renderla definitiva.

Quando è stata introdotta la lambda, questa restrizione è stata attenuata. Da qui la necessità di rendere definitiva la variabile locale se non viene cambiata una volta inizializzata come lambda in sé non è altro che una classe anonima.

Java 8 ha realizzato il dolore di dichiarare la variabile locale finale ogni volta che uno sviluppatore ha utilizzato lambda, ha introdotto questo concetto e ha reso superfluo rendere definitive le variabili locali. Quindi se vedi che la regola per le classi anonime non è cambiata, è solo che non devi scrivere la finalparola chiave ogni volta che usi lambdas.

Ho trovato una buona spiegazione qui

Nel tuo esempio, puoi sostituire the forEachcon lamdba con un semplice forloop e modificare liberamente qualsiasi variabile. O, probabilmente, refactoring il codice in modo da non dover modificare alcuna variabile. Tuttavia, spiegherò per completezza cosa significa l'errore e come aggirarlo.

Specifiche del linguaggio Java 8, §15.27.2 :

Qualsiasi variabile locale, parametro formale o parametro di eccezione utilizzato ma non dichiarato in un'espressione lambda deve essere dichiarato finale o essere effettivamente finale ( §4.12.4 ), oppure si verifica un errore in fase di compilazione nel tentativo di utilizzo.

Fondamentalmente non è possibile modificare una variabile locale ( calTzin questo caso) dall'interno di una lambda (o una classe locale / anonima). Per ottenere ciò in Java, devi usare un oggetto mutabile e modificarlo (tramite una variabile finale) dalla lambda. Un esempio di un oggetto mutabile qui sarebbe una matrice di un elemento:

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

se non è necessario modificare la variabile di una soluzione generale per questo tipo di problema sarebbe estrarre la parte di codice che usa lambda e usare la parola chiave finale sul parametro-metodo.

Una variabile utilizzata nell'espressione lambda dovrebbe essere un finale o effettivamente finale, ma è possibile assegnare un valore a un array di un elemento finale.

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