Memodifikasi variabel lokal forEachmemberikan kesalahan kompilasi:

Normal

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

Dengan Lambda

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

Ada ide bagaimana mengatasi ini?

Gunakan pembungkus

Semua jenis pembungkus itu bagus.

Dengan Java 8+ , gunakan salah satu AtomicInteger:

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

... atau array:

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

Dengan Java 10+ :

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

Catatan: berhati-hatilah jika Anda menggunakan aliran paralel. Anda mungkin tidak mendapatkan hasil yang diharapkan. Solusi lain seperti Stuart mungkin lebih disesuaikan untuk kasus tersebut.

Untuk tipe selain int

Tentu saja, ini masih berlaku untuk tipe selain int. Anda hanya perlu mengubah jenis pembungkusan AtomicReferencemenjadi larik atau jenis itu. Misalnya, jika Anda menggunakan a String, cukup lakukan hal berikut:

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

Gunakan array:

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

Atau dengan Java 10+:

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

Ini cukup dekat dengan masalah XY . Artinya, pertanyaan yang diajukan pada dasarnya adalah bagaimana memutasi variabel lokal yang ditangkap dari lambda. Tetapi tugas sebenarnya yang ada adalah bagaimana memberi nomor pada elemen daftar.

Dalam pengalaman saya, lebih dari 80% dari waktu ada pertanyaan tentang bagaimana mengubah lokal yang ditangkap dari dalam lambda, ada cara yang lebih baik untuk melanjutkan. Biasanya ini melibatkan pengurangan, tetapi dalam kasus ini teknik menjalankan aliran melalui indeks daftar berlaku dengan baik:

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

Jika Anda hanya perlu meneruskan nilai dari luar ke lambda, dan tidak mengeluarkannya, Anda dapat melakukannya dengan kelas anonim biasa alih-alih lambda:

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

Karena variabel yang digunakan dari luar lamda harus (secara implisit) final, Anda harus menggunakan sesuatu seperti AtomicIntegeratau menulis struktur data Anda sendiri.

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

Jika Anda menggunakan Java 10, Anda dapat menggunakannya varuntuk itu:

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

Alternatif untuk AtomicInteger(atau objek lain yang dapat menyimpan nilai) adalah dengan menggunakan array:

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

Tapi lihat jawaban Stuart : mungkin ada cara yang lebih baik untuk menangani kasus Anda.

Saya tahu itu pertanyaan lama, tetapi jika Anda merasa nyaman dengan solusinya, mengingat fakta bahwa variabel eksternal harus final, Anda cukup melakukan ini:

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

Mungkin bukan yang paling elegan, atau bahkan yang paling benar, tetapi itu akan berhasil.

Anda dapat membungkusnya untuk mengatasi kompiler tetapi harap diingat bahwa efek samping di lambda tidak disarankan.

Mengutip javadoc

Efek samping dalam parameter perilaku untuk operasi streaming, secara umum, tidak disarankan, karena sering kali dapat menyebabkan pelanggaran tanpa disadari terhadap persyaratan keadaan tanpa kewarganegaraan Sejumlah kecil operasi aliran, seperti forEach () dan peek (), dapat beroperasi hanya melalui sisi -efek; ini harus digunakan dengan hati-hati

Saya punya masalah yang sedikit berbeda. Alih-alih menambahkan variabel lokal di forEach, saya perlu menetapkan objek ke variabel lokal.

Saya menyelesaikan ini dengan mendefinisikan kelas domain dalam pribadi yang membungkus kedua daftar yang ingin saya ulangi (countryList) dan keluaran yang saya harapkan dari daftar itu (foundCountry). Kemudian menggunakan Java 8 "forEach", saya mengulangi bidang daftar, dan ketika objek yang saya inginkan ditemukan, saya menetapkan objek itu ke bidang keluaran. Jadi ini memberikan nilai ke bidang variabel lokal, tidak mengubah variabel lokal itu sendiri. Saya percaya bahwa karena variabel lokal itu sendiri tidak berubah, kompilator tidak mengeluh. Saya kemudian dapat menggunakan nilai yang saya tangkap di bidang keluaran, di luar daftar.

Objek Domain:

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

Objek pembungkus:

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

Operasi berulang:

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

Anda dapat menghapus metode kelas pembungkus "setCountryList ()" dan membuat kolom "countryList" final, tetapi saya tidak mendapatkan kesalahan kompilasi yang membiarkan detail ini sebagaimana adanya.

Untuk mendapatkan solusi yang lebih umum, Anda dapat menulis kelas Wrapper generik:

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

(ini adalah varian dari solusi yang diberikan oleh Almir Campos).

Dalam kasus khusus ini bukan solusi yang baik, karena Integerlebih buruk daripada inttujuan Anda, bagaimanapun solusi ini lebih umum menurut saya.

Ya, Anda dapat memodifikasi variabel lokal dari dalam lambda (dengan cara yang ditunjukkan oleh jawaban lain), tetapi Anda tidak boleh melakukannya. Lambda telah digunakan untuk gaya pemrograman fungsional dan ini berarti: Tidak ada efek samping. Apa yang ingin Anda lakukan dianggap gaya yang buruk. Ini juga berbahaya jika terjadi aliran paralel.

Anda harus menemukan solusi tanpa efek samping atau menggunakan for loop tradisional.