ArrayIndexOutOfBoundsException saat menggunakan iterator ArrayList

Saat ini, saya memiliki program yang berisi potongan kode yang terlihat seperti ini:

while (arrayList.iterator().hasNext()) {
     //value is equal to a String value
     if( arrayList.iterator().next().equals(value)) {
          // do something 
     }
}

Apakah saya melakukan itu dengan benar, sejauh iterasi melalui ArrayList berjalan?

Kesalahan yang saya dapatkan adalah:

java.lang.ArrayIndexOutOfBoundsException: -1
    at java.util.ArrayList.get(Unknown Source)
    at main1.endElement(main1.java:244)
    at com.sun.org.apache.xerces.internal.parsers.AbstractSAXParser.endElement(Unknown Source)
    at com.sun.org.apache.xerces.internal.impl.XMLDocumentFragmentScannerImpl.scanEndElement(Unknown Source)
    at com.sun.org.apache.xerces.internal.impl.XMLDocumentFragmentScannerImpl$FragmentContentDriver.next(Unknown Source)
    at com.sun.org.apache.xerces.internal.impl.XMLDocumentScannerImpl.next(Unknown Source)
    at com.sun.org.apache.xerces.internal.impl.XMLDocumentFragmentScannerImpl.scanDocument(Unknown Source)
    at com.sun.org.apache.xerces.internal.parsers.XML11Configuration.parse(Unknown Source)
    at com.sun.org.apache.xerces.internal.parsers.XML11Configuration.parse(Unknown Source)
    at com.sun.org.apache.xerces.internal.parsers.XMLParser.parse(Unknown Source)
    at com.sun.org.apache.xerces.internal.parsers.AbstractSAXParser.parse(Unknown Source)
    at com.sun.org.apache.xerces.internal.jaxp.SAXParserImpl$JAXPSAXParser.parse(Unknown Source)
    at javax.xml.parsers.SAXParser.parse(Unknown Source)
    at javax.xml.parsers.SAXParser.parse(Unknown Source)
    at main1.traverse(main1.java:73)
    at main1.traverse(main1.java:102)
    at main1.traverse(main1.java:102)
    at main1.main(main1.java:404)

Saya akan menunjukkan sisa kode, tetapi itu cukup ekstensif, dan jika saya tidak melakukan iterasi dengan benar, saya akan berasumsi satu-satunya kemungkinan adalah bahwa saya tidak menginisialisasi dengan ArrayListbenar.

Apakah saya melakukan itu dengan benar, sejauh iterasi melalui Arraylist berjalan?

Tidak: dengan memanggil iteratordua kali di setiap iterasi, Anda mendapatkan iterator baru setiap saat.

Cara termudah untuk menulis perulangan ini menggunakan konstruksi for-each :

for (String s : arrayList)
    if (s.equals(value))
        // ...

Adapun

java.lang.ArrayIndexOutOfBoundsException: -1

Anda baru saja mencoba mendapatkan nomor elemen -1dari sebuah array. Penghitungan dimulai dari nol.

Meskipun saya setuju bahwa jawaban yang diterima biasanya merupakan solusi terbaik dan pasti lebih mudah digunakan, saya perhatikan tidak ada yang menunjukkan penggunaan iterator yang tepat. Jadi, inilah contoh singkatnya:

Iterator<Object> it = arrayList.iterator();
while(it.hasNext())
{
    Object obj = it.next();
    //Do something with obj
}

List<String> arrayList = new ArrayList<String>();
for (String s : arrayList) {
    if(s.equals(value)){
        //do something
    }
}

atau

for (int i = 0; i < arrayList.size(); i++) {
    if(arrayList.get(i).equals(value)){
        //do something
    }
}

Tapi hati-hati ArrayList bisa menampung nilai null . Jadi perbandingan seharusnya

value.equals(arrayList.get(i))

ketika Anda yakin bahwa nilainya bukan null atau Anda harus memeriksa apakah elemen yang diberikan adalah null.

Anda juga bisa menggunakan seperti ini:

for(Iterator iterator = arrayList.iterator(); iterator.hasNext();) {
x = iterator.next();
//do some stuff
}

Ini praktik yang baik untuk melemparkan dan menggunakan objek. Misalnya, jika 'arrayList' berisi daftar objek 'Object1'. Kemudian, kita dapat menulis ulang kode tersebut sebagai:

for(Iterator iterator = arrayList.iterator(); iterator.hasNext();) {
x = (Object1) iterator.next();
//do some stuff
}

Anda juga bisa melakukan perulangan for seperti yang Anda lakukan untuk larik tetapi alih-alih larik [i], Anda akan menggunakan list.get (i)

for (int i = 0; i < list.size(); i++) {
    System.out.println(list.get(i));
}

Terlepas dari jawaban larsman (yang memang benar), pengecualian dalam panggilan ke metode get (), jadi kode yang Anda posting bukanlah yang menyebabkan kesalahan.

Cara yang efisien untuk mengulang Anda ArrayListdiikuti oleh tautan ini . Jenis ini akan meningkatkan kinerja perulangan selama iterasi

int size = list.size();

for(int j = 0; j < size; j++) {
    System.out.println(list.get(i));
}

iterasi menggunakan iterator tidak aman untuk kegagalan misalnya jika Anda menambahkan elemen ke koleksi setelah pembuatan iterator maka itu akan memunculkan concurrentmodificaionexception. Juga tidak aman untuk benang, Anda harus membuatnya aman untuk benang secara eksternal.

Jadi lebih baik menggunakan untuk-setiap struktur for loop. Ini setidaknya aman dari kegagalan.