Việc sửa đổi một biến cục bộ trong gây forEachra lỗi biên dịch:

Bình thường

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

Với Lambda

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

Bất kỳ ý tưởng làm thế nào để giải quyết điều này?

Sử dụng một trình bao bọc

Bất kỳ loại giấy gói nào cũng tốt.

Với Java 8+ , hãy sử dụng AtomicInteger:

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

... hoặc một mảng:

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

Với Java 10+ :

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

Lưu ý: hãy rất cẩn thận nếu bạn sử dụng dòng song song. Bạn có thể không đạt được kết quả như mong đợi. Các giải pháp khác như Stuart's có thể phù hợp hơn cho những trường hợp đó.

Đối với các loại khác int

Tất nhiên, điều này vẫn có giá trị đối với các loại khác int. Bạn chỉ cần thay đổi kiểu gói thành một AtomicReferencehoặc một mảng của kiểu đó. Ví dụ: nếu bạn sử dụng a String, chỉ cần làm như sau:

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

Sử dụng một mảng:

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

Hoặc với Java 10+:

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

Điều này khá gần với một vấn đề XY . Đó là, câu hỏi được đặt ra về cơ bản là làm thế nào để biến đổi một biến cục bộ đã được bắt từ lambda. Nhưng nhiệm vụ thực tế trước mắt là làm thế nào để đánh số các phần tử của một danh sách.

Theo kinh nghiệm của tôi, có đến 80% thời gian có câu hỏi làm thế nào để thay đổi một cục bộ đã bị bắt từ bên trong lambda, có một cách tốt hơn để tiếp tục. Thông thường điều này liên quan đến việc giảm bớt, nhưng trong trường hợp này, kỹ thuật chạy luồng qua các chỉ mục danh sách áp dụng tốt:

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

Nếu bạn chỉ cần truyền giá trị từ bên ngoài vào lambda và không lấy nó ra ngoài, bạn có thể thực hiện điều đó với một lớp ẩn danh thông thường thay vì lambda:

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

Vì các biến được sử dụng từ bên ngoài lamda phải là (ngầm định) cuối cùng, bạn phải sử dụng một cái gì đó như AtomicIntegerhoặc viết cấu trúc dữ liệu của riêng bạn.

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

Nếu bạn đang sử dụng Java 10, bạn có thể sử dụng var:

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

Một thay thế cho AtomicInteger(hoặc bất kỳ đối tượng nào khác có thể lưu trữ một giá trị) là sử dụng một mảng:

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

Nhưng hãy xem câu trả lời của Stuart : có thể có cách tốt hơn để giải quyết trường hợp của bạn.

Tôi biết đó là một câu hỏi cũ, nhưng nếu bạn cảm thấy thoải mái với cách giải quyết khác, xem xét thực tế là biến bên ngoài phải là cuối cùng, bạn có thể chỉ cần thực hiện điều này:

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

Có thể không phải là trang nhã nhất, hoặc thậm chí là chính xác nhất, nhưng nó sẽ làm được.

Bạn có thể kết thúc nó để giải quyết trình biên dịch nhưng hãy nhớ rằng không khuyến khích các tác dụng phụ trong lambdas.

Để trích dẫn javadoc

Nói chung, không khuyến khích các tác dụng phụ trong các tham số hành vi đối với hoạt động luồng vì chúng thường có thể dẫn đến việc vô tình vi phạm yêu cầu không trạng thái Một số lượng nhỏ các hoạt động luồng, chẳng hạn như forEach () và peek (), chỉ có thể hoạt động thông qua bên -Các hiệu ứng; những thứ này nên được sử dụng cẩn thận

Tôi đã có một vấn đề hơi khác. Thay vì tăng một biến cục bộ trong forEach, tôi cần gán một đối tượng cho biến cục bộ.

Tôi đã giải quyết vấn đề này bằng cách xác định một lớp miền bên trong riêng tư bao bọc cả danh sách tôi muốn lặp lại (countryList) và kết quả mà tôi hy vọng nhận được từ danh sách đó (foundCountry). Sau đó, sử dụng Java 8 "forEach", tôi lặp qua trường danh sách và khi đối tượng tôi muốn được tìm thấy, tôi gán đối tượng đó cho trường đầu ra. Vì vậy, điều này chỉ định một giá trị cho một trường của biến cục bộ, không thay đổi chính biến cục bộ. Tôi tin rằng vì bản thân biến cục bộ không bị thay đổi, trình biên dịch không phàn nàn. Sau đó, tôi có thể sử dụng giá trị mà tôi đã nắm bắt trong trường đầu ra, bên ngoài danh sách.

Đối tượng miền:

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

Đối tượng gói:

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

Lặp lại hoạt động:

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());

Bạn có thể xóa phương thức lớp wrapper "setCountryList ()" và đặt trường "countryList" là cuối cùng, nhưng tôi không gặp lỗi biên dịch khi để nguyên các chi tiết này.

Để có một giải pháp chung hơn, bạn có thể viết một lớp Wrapper chung:

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

(đây là một biến thể của giải pháp do Almir Campos đưa ra).

Trong trường hợp cụ thể, đây không phải là một giải pháp tốt, cũng như Integertệ hơn intcho mục đích của bạn, dù sao thì giải pháp này tôi nghĩ là tổng quát hơn.

Có, bạn có thể sửa đổi các biến cục bộ từ bên trong lambdas (theo cách được hiển thị bởi các câu trả lời khác), nhưng bạn không nên làm điều đó. Lambdas dành cho phong cách lập trình chức năng và điều này có nghĩa là: Không có tác dụng phụ. Những gì bạn muốn làm được coi là phong cách xấu. Nó cũng nguy hiểm trong trường hợp các dòng song song.

Bạn nên tìm một giải pháp không có tác dụng phụ hoặc sử dụng vòng lặp for truyền thống.