ตัวแปรที่ใช้ในแลมบ์ดานิพจน์ควรเป็นขั้นสุดท้ายหรือมีประสิทธิผลสุดท้าย

ตัวแปรที่ใช้ในแลมบ์ดานิพจน์ควรเป็นขั้นสุดท้ายหรือมีประสิทธิผลสุดท้าย

เมื่อฉันพยายามใช้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หมายถึงตัวแปรที่จะสามารถ instantiated เพียงครั้งเดียว ใน Java คุณไม่สามารถใช้ตัวแปรที่ไม่ใช่ขั้นสุดท้ายในแลมบ์ดาและในคลาสภายในที่ไม่ระบุชื่อได้

คุณสามารถ refactor โค้ดของคุณด้วยลูปสำหรับแต่ละอันเก่า:

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 :

ข้อ จำกัด ของตัวแปรสุดท้ายที่มีประสิทธิภาพห้ามการเข้าถึงตัวแปรโลคัลที่เปลี่ยนแปลงแบบไดนามิกซึ่งการดักจับอาจทำให้เกิดปัญหาการทำงานพร้อมกัน

เพื่อลดความเสี่ยงของข้อบกพร่องพวกเขาตัดสินใจที่จะตรวจสอบให้แน่ใจว่าตัวแปรที่จับได้จะไม่มีการกลายพันธุ์

จากแลมบ์ดาคุณไม่สามารถอ้างอิงถึงสิ่งที่ยังไม่สิ้นสุดได้ คุณต้องประกาศ wrapper สุดท้ายจากนอก 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();
}   

Java 8มีแนวคิดใหม่ที่เรียกว่าตัวแปร“ Effectively final” หมายความว่าตัวแปรโลคัลที่ไม่ใช่ขั้นสุดท้ายซึ่งค่าไม่เคยเปลี่ยนแปลงหลังจากการเริ่มต้นเรียกว่า“ ผลสุดท้ายอย่างมีประสิทธิภาพ”

แนวคิดนี้ถูกนำมาใช้เพราะก่อนที่จะJava 8เราไม่สามารถใช้ตัวแปรท้องถิ่นไม่ใช่ครั้งสุดท้ายในระดับที่ไม่ระบุชื่อ หากคุณต้องการเข้าถึงตัวแปรโลคัลในคลาสที่ไม่ระบุชื่อคุณต้องทำให้มันเป็นขั้นสุดท้าย

เมื่อมีการนำแลมด้ามาใช้ข้อ จำกัด นี้ก็คลายลง ดังนั้นความจำเป็นในการทำให้ตัวแปรท้องถิ่นเป็นขั้นสุดท้ายหากไม่มีการเปลี่ยนแปลงเมื่อเริ่มต้นเป็นแลมบ์ดาในตัวเองก็ไม่มีอะไรนอกจากคลาสที่ไม่ระบุชื่อ

Java 8ตระหนักถึงความเจ็บปวดจากการประกาศตัวแปรภายในทุกครั้งที่นักพัฒนาใช้แลมด้านำแนวคิดนี้มาใช้และทำให้ไม่จำเป็นต้องทำให้ตัวแปรโลคัลเป็นขั้นสุดท้าย ดังนั้นหากคุณเห็นว่ากฎสำหรับคลาสที่ไม่ระบุตัวตนไม่มีการเปลี่ยนแปลงคุณไม่จำเป็นต้องเขียนfinalคีย์เวิร์ดทุกครั้งเมื่อใช้ lambdas

ฉันพบคำอธิบายที่ดีที่นี่

ในตัวอย่างของคุณคุณสามารถแทนที่forEachด้วย lamdba ด้วยforลูปธรรมดาและแก้ไขตัวแปรใด ๆ ได้อย่างอิสระ หรืออาจจะ refactor โค้ดของคุณเพื่อที่คุณจะได้ไม่ต้องแก้ไขตัวแปรใด ๆ อย่างไรก็ตามฉันจะอธิบายเพื่อความครบถ้วนว่าข้อผิดพลาดหมายถึงอะไรและจะแก้ไขอย่างไร

ข้อกำหนดภาษา 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;
}