ArrayList yineleyicisini kullanırken ArrayIndexOutOfBoundsException

Şu anda, şuna benzeyen bir kod parçası içeren bir programım var:

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

ArrayList'te yinelemeye kadar bunu doğru yapıyor muyum?

Aldığım hata:

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)

Kodun geri kalanını gösterirdim, ancak oldukça kapsamlı ve yinelemeyi doğru yapmazsam, tek olasılığın ArrayListdoğru şekilde başlatmadığım olduğunu varsayabilirim .

Dizi Listesinde yinelemeye kadar bunu doğru yapıyor muyum?

Hayır: iteratorher yinelemede iki kez arayarak , her zaman yeni yineleyiciler alıyorsunuz.

Bu döngüyü yazmanın en kolay yolu, her bir yapı için kullanmaktır :

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

Gelince

java.lang.ArrayIndexOutOfBoundsException: -1

-1Bir diziden eleman numarasını almaya çalıştınız . Sayma sıfırdan başlar.

Kabul edilen cevabın genellikle en iyi çözüm olduğunu ve kesinlikle kullanımı daha kolay olduğunu kabul etsem de, hiç kimsenin yineleyicinin doğru kullanımını göstermediğini fark ettim. İşte hızlı bir örnek:

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

veya

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

Ancak dikkatli olun ArrayList boş değerler tutabilir . Öyleyse karşılaştırma olmalı

value.equals(arrayList.get(i))

değerin boş olmadığından eminseniz veya verilen elemanın boş olup olmadığını kontrol etmelisiniz.

Ayrıca şu şekilde kullanabilirsiniz:

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

Nesneyi yayınlamak ve kullanmak iyi bir uygulamadır. Örneğin, 'diziListesi' 'Nesne1' nesnelerinin bir listesini içeriyorsa. Ardından kodu şu şekilde yeniden yazabiliriz:

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

Bir dizi için yaptığınız gibi bir for döngüsü de yapabilirsiniz, ancak dizi [i] yerine list.get (i) kullanırsınız.

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

Larsmans cevabının yanı sıra (gerçekten doğru olan), get () yöntemine yapılan bir çağrıdaki istisna, yani gönderdiğiniz kod hataya neden olan kod değildir.

ArrayListBu bağlantıyı takip ederek yinelemenin etkili yolu . Bu tür, yineleme sırasında döngü performansını artıracaktır

int size = list.size();

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

Yineleyici kullanarak yineleme, hata güvenli değildir, örneğin yineleyici oluşturulduktan sonra koleksiyona öğe eklerseniz, eşzamanlı değişiklik istisnası oluşturur. Ayrıca iş parçacığı için güvenli değildir, dışarıdan iş parçacığını güvenli hale getirmelisiniz.

Bu nedenle, for döngüsünün her yapısı için kullanmak daha iyidir. En azından güvenli.