การแก้ไขตัวแปรภายในforEachทำให้เกิดข้อผิดพลาดในการคอมไพล์:

ปกติ

    int ordinal = 0;
    for (Example s : list) {
        s.setOrdinal(ordinal);
        ordinal++;
    }

ด้วยแลมด้า

    int ordinal = 0;
    list.forEach(s -> {
        s.setOrdinal(ordinal);
        ordinal++;
    });

มีความคิดอย่างไรในการแก้ไขปัญหานี้

ใช้กระดาษห่อหุ้ม

ห่อแบบไหนก็ได้ดี

ด้วยJava 8+ให้ใช้AtomicInteger:

AtomicInteger ordinal = new AtomicInteger(0);
list.forEach(s -> {
  s.setOrdinal(ordinal.getAndIncrement());
});

... หรืออาร์เรย์:

int[] ordinal = { 0 };
list.forEach(s -> {
  s.setOrdinal(ordinal[0]++);
});

ด้วยJava 10+ :

var wrapper = new Object(){ int ordinal = 0; };
list.forEach(s -> {
  s.setOrdinal(wrapper.ordinal++);
});

หมายเหตุ: โปรดระวังให้มากหากคุณใช้สตรีมคู่ขนาน คุณอาจไม่ได้ผลลัพธ์ที่คาดหวัง วิธีแก้ปัญหาอื่น ๆ เช่น Stuart's อาจปรับให้เข้ากับกรณีเหล่านั้นได้มากขึ้น

สำหรับประเภทอื่นที่ไม่ใช่ int

ของหลักสูตรนี้ยังคงถูกต้องประเภทอื่น ๆ intกว่า คุณจะต้องเปลี่ยนประเภทการห่อเป็นAtomicReferenceหรืออาร์เรย์ของประเภทนั้นเท่านั้น ตัวอย่างเช่นหากคุณใช้ a Stringให้ทำดังต่อไปนี้:

AtomicReference<String> value = new AtomicReference<>();
list.forEach(s -> {
  value.set("blah");
});

ใช้อาร์เรย์:

String[] value = { null };
list.forEach(s-> {
  value[0] = "blah";
});

หรือด้วย Java 10+:

var wrapper = new Object(){ String value; }
list.forEach(s->{
  wrapper.value = "blah";
});

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

จากประสบการณ์ของฉันสูงกว่า 80% ของเวลาที่มีคำถามเกี่ยวกับวิธีการกลายพันธุ์ที่จับได้จากภายในแลมด้ามีวิธีที่ดีกว่าในการดำเนินการ โดยปกติจะเกี่ยวข้องกับการลดลง แต่ในกรณีนี้เทคนิคการเรียกใช้สตรีมผ่านดัชนีรายการใช้ได้ดี:

IntStream.range(0, list.size())
         .forEach(i -> list.get(i).setOrdinal(i));

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

list.forEach(new Consumer<Example>() {
    int ordinal = 0;
    public void accept(Example s) {
        s.setOrdinal(ordinal);
        ordinal++;
    }
});

เนื่องจากตัวแปรที่ใช้จากภายนอก lamda จะต้องเป็นขั้นสุดท้าย (โดยปริยาย) คุณจึงต้องใช้บางสิ่งเช่นAtomicIntegerหรือเขียนโครงสร้างข้อมูลของคุณเอง

ดู https://docs.oracle.com/javase/tutorial/java/javaOO/lambdaexpressions.html#accessing-local-variables

หากคุณใช้ Java 10 คุณสามารถใช้varเพื่อ:

var ordinal = new Object() { int value; };
list.forEach(s -> {
    s.setOrdinal(ordinal.value);
    ordinal.value++;
});

อีกทางเลือกหนึ่งสำหรับAtomicInteger(หรือวัตถุอื่น ๆ ที่สามารถเก็บค่าได้) คือการใช้อาร์เรย์:

final int ordinal[] = new int[] { 0 };
list.forEach ( s -> s.setOrdinal ( ordinal[ 0 ]++ ) );

แต่ดูคำตอบของ Stuart : อาจมีวิธีที่ดีกว่าในการจัดการกับกรณีของคุณ

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

final int[] ordinal = new int[1];
list.forEach(s -> {
    s.setOrdinal(ordinal[0]);
    ordinal[0]++;
});

อาจจะไม่สวยหรูที่สุดหรือถูกต้องที่สุด แต่ก็ทำได้

คุณสามารถสรุปได้เพื่อแก้ไขปัญหาคอมไพเลอร์ แต่โปรดจำไว้ว่าผลข้างเคียงใน lambdas ไม่ได้รับการสนับสนุน

เพื่ออ้างถึงjavadoc

โดยทั่วไปผลข้างเคียงในพารามิเตอร์พฤติกรรมต่อการดำเนินการสตรีมเป็นสิ่งที่ไม่พึงปรารถนาเนื่องจากมักนำไปสู่การละเมิดข้อกำหนดการไร้สัญชาติโดยไม่เจตนาการดำเนินการสตรีมจำนวนเล็กน้อยเช่น forEach () และ peek () สามารถทำงานผ่านด้านข้างเท่านั้น - ผลกระทบ; ควรใช้ด้วยความระมัดระวัง

ฉันมีปัญหาที่แตกต่างกันเล็กน้อย แทนที่จะเพิ่มตัวแปรโลคัลใน forEach ฉันต้องกำหนดอ็อบเจกต์ให้กับตัวแปรโลคัล

ฉันแก้ไขสิ่งนี้โดยกำหนดคลาสโดเมนภายในส่วนตัวที่รวมทั้งรายการที่ฉันต้องการวนซ้ำ (countryList) และผลลัพธ์ที่ฉันหวังว่าจะได้รับจากรายการนั้น (foundCountry) จากนั้นใช้ Java 8 "forEach" ฉันวนซ้ำในฟิลด์รายการและเมื่อพบวัตถุที่ฉันต้องการฉันกำหนดอ็อบเจ็กต์นั้นให้กับฟิลด์เอาต์พุต ดังนั้นจึงกำหนดค่าให้กับฟิลด์ของตัวแปรโลคัลโดยไม่เปลี่ยนตัวแปรโลคัลเอง ฉันเชื่อว่าเนื่องจากไม่มีการเปลี่ยนแปลงตัวแปรภายในคอมไพเลอร์จึงไม่บ่น จากนั้นฉันสามารถใช้ค่าที่ฉันจับได้ในฟิลด์เอาต์พุตนอกรายการ

วัตถุโดเมน:

public class Country {

    private int id;
    private String countryName;

    public Country(int id, String countryName){
        this.id = id;
        this.countryName = countryName;
    }

    public int getId() {
        return id;
    }

    public void setId(int id) {
        this.id = id;
    }

    public String getCountryName() {
        return countryName;
    }

    public void setCountryName(String countryName) {
        this.countryName = countryName;
    }
}

วัตถุ Wrapper:

private class CountryFound{
    private final List<Country> countryList;
    private Country foundCountry;
    public CountryFound(List<Country> countryList, Country foundCountry){
        this.countryList = countryList;
        this.foundCountry = foundCountry;
    }
    public List<Country> getCountryList() {
        return countryList;
    }
    public void setCountryList(List<Country> countryList) {
        this.countryList = countryList;
    }
    public Country getFoundCountry() {
        return foundCountry;
    }
    public void setFoundCountry(Country foundCountry) {
        this.foundCountry = foundCountry;
    }
}

ดำเนินการซ้ำ:

int id = 5;
CountryFound countryFound = new CountryFound(countryList, null);
countryFound.getCountryList().forEach(c -> {
    if(c.getId() == id){
        countryFound.setFoundCountry(c);
    }
});
System.out.println("Country found: " + countryFound.getFoundCountry().getCountryName());

คุณสามารถลบเมธอดคลาส wrapper "setCountryList ()" และทำให้ฟิลด์ "countryList" เป็นขั้นสุดท้าย แต่ฉันไม่ได้รับข้อผิดพลาดในการคอมไพล์โดยทิ้งรายละเอียดเหล่านี้ไว้ตามที่เป็นอยู่

หากต้องการวิธีแก้ปัญหาทั่วไปคุณสามารถเขียนคลาส Wrapper ทั่วไป:

public static class Wrapper<T> {
    public T obj;
    public Wrapper(T obj) { this.obj = obj; }
}
...
Wrapper<Integer> w = new Wrapper<>(0);
this.forEach(s -> {
    s.setOrdinal(w.obj);
    w.obj++;
});

(นี่คือตัวแปรของการแก้ปัญหาที่ Almir Campos กำหนด)

ในกรณีเฉพาะนี่ไม่ใช่วิธีแก้ปัญหาที่ดี แต่Integerแย่กว่าintสำหรับจุดประสงค์ของคุณอย่างไรก็ตามฉันคิดว่าวิธีนี้เป็นวิธีที่กว้างกว่า

ได้คุณสามารถแก้ไขตัวแปรภายในจาก lambdas ภายใน (ตามวิธีที่แสดงโดยคำตอบอื่น ๆ ) แต่คุณไม่ควรทำเช่นนั้น Lambdas เป็นรูปแบบการเขียนโปรแกรมที่ใช้งานได้จริงและหมายความว่า: ไม่มีผลข้างเคียง สิ่งที่คุณต้องการทำถือเป็นรูปแบบที่ไม่ดี นอกจากนี้ยังเป็นอันตรายในกรณีของสายน้ำคู่ขนาน

คุณควรหาวิธีแก้ปัญหาโดยไม่มีผลข้างเคียงหรือใช้แบบเดิมสำหรับการวนซ้ำ