ArrayIndexOutOfBoundsException quando si utilizza l'iteratore di ArrayList

In questo momento, ho un programma contenente un pezzo di codice che assomiglia a questo:

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

Lo sto facendo bene, per quanto riguarda l'iterazione di ArrayList?

L'errore che ricevo è:

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)

Vorrei mostrare il resto del codice, ma è piuttosto ampio e se non sto eseguendo correttamente l'iterazione, presumo che l'unica possibilità sia che non sto inizializzando ArrayListcorrettamente.

Lo sto facendo bene, per quanto riguarda l'iterazione nell'Arraylist?

No: chiamando iteratordue volte in ogni iterazione, ottieni sempre nuovi iteratori.

Il modo più semplice per scrivere questo ciclo è usare il costrutto for-each :

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

Quanto a

java.lang.ArrayIndexOutOfBoundsException: -1

Hai appena provato a ottenere il numero di elemento -1da un array. Il conteggio inizia da zero.

Sebbene sia d'accordo sul fatto che la risposta accettata sia di solito la soluzione migliore e decisamente più facile da usare, ho notato che nessuno ha mostrato l'uso corretto dell'iteratore. Quindi ecco un rapido esempio:

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

o

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

Ma attenzione, ArrayList può contenere valori nulli . Quindi il confronto dovrebbe essere

value.equals(arrayList.get(i))

quando sei sicuro che il valore non è nullo o dovresti controllare se l'elemento dato è nullo.

Puoi anche usare in questo modo:

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

È una buona pratica lanciare e utilizzare l'oggetto. Ad esempio, se "arrayList" contiene un elenco di oggetti "Object1". Quindi, possiamo riscrivere il codice come:

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

Potresti anche fare un ciclo for come faresti per un array, ma invece di array [i] dovresti usare list.get (i)

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

A parte la risposta di larsmans (che è effettivamente corretta), l'eccezione in una chiamata a un metodo get (), quindi il codice che hai postato non è quello che causa l'errore.

Modo efficiente per iterare il tuo ArrayListseguito da questo link . Questo tipo migliorerà le prestazioni del loop durante l'iterazione

int size = list.size();

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

l'iterazione utilizzando l'iteratore non è sicura, ad esempio se aggiungi un elemento alla raccolta dopo la creazione dell'iteratore, verrà generata un'eccezione di modifica simultanea. Inoltre non è thread-safe, devi renderlo thread-safe esternamente.

Quindi è meglio usare la struttura for-each del ciclo for. È almeno a prova di errore.