ラムダ式で使用される変数はfinalまたは事実上finalである必要があります

ラムダ式で使用される変数は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では、ラムダや匿名の内部クラスで非final変数を使用することはできません。

古い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戻り値の型として設定しないのですか？

また、これらのヒントが改善に役立つことを願っています。

他の回答は要件を証明しますが、要件が存在する理由を説明していません。

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には、「実質的に最終的な」変数と呼ばれる新しい概念があります。これは、初期化後に値が変化しないfinal以外のローカル変数を「Effectively Final」と呼ぶことを意味します。

この概念が導入されたのは、Java 8より前のバージョンでは、匿名クラスでfinal以外のローカル変数を使用できなかったためです。匿名クラスのローカル変数にアクセスしたい場合は、それをfinalにする必要があります。

ラムダが導入されたとき、この制限は緩和されました。したがって、ラムダ自体は匿名クラスにすぎないため、初期化された後で変更されない場合はローカル変数をfinalにする必要があります。

Java 8は、開発者がラムダを使用するたびにローカル変数をfinalと宣言する手間を実感し、この概念を導入して、ローカル変数をfinalにする必要をなくしました。したがって、匿名クラスのルールが変更されていないfinal場合は、ラムダを使用するときに毎回キーワードを記述する必要がないだけです。

ここで良い説明を見つけました

あなたの例ではforEach、lambbaを単純なforループに置き換えて、変数を自由に変更できます。または、おそらく、変数を変更する必要がないようにコードをリファクタリングします。ただし、完全を期すために、エラーの意味と回避策を説明します。

Java 8言語仕様、§15.27.2：

使用されているがラムダ式で宣言されていないローカル変数、仮パラメーター、または例外パラメーターは、finalで宣言するか、事実上finalである必要があります（§4.12.4）。そうしないと、使用しようとするとコンパイル時エラーが発生します。

基本的にcalTz、ラムダ（またはローカル/匿名クラス）内からローカル変数（この場合）を変更することはできません。これをJavaで実現するには、可変オブジェクトを使用して、ラムダから（最終変数を介して）変更する必要があります。ここでの可変オブジェクトの1つの例は、1つの要素の配列です。

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

この種の問題に対する一般的な回避策よりも変数を変更する必要がない場合は、 ラムダを使用し、メソッドパラメーターでfinalキーワードを使用するコードの一部を抽出することになります。

ラムダ式で使用される変数は、最終または実質的に最終でなければなりませんが、最後の1要素配列に値を割り当てることができます。

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