ArrayIndexOutOfBoundsException ao usar o iterador de ArrayList

No momento, tenho um programa que contém um trecho de código semelhante a este:

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

Estou fazendo isso certo, no que diz respeito à iteração por meio da ArrayList?

O erro que estou recebendo é:

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)

Eu mostraria o resto do código, mas é bastante extenso e, se não estou fazendo a iteração corretamente, presumo que a única possibilidade é não estar inicializando ArrayListcorretamente.

Estou fazendo isso certo, no que diz respeito à iteração pelo Arraylist?

Não: chamando iteratorduas vezes em cada iteração, você obtém novos iteradores o tempo todo.

A maneira mais fácil de escrever esse loop é usando a construção for-each :

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

Quanto a

java.lang.ArrayIndexOutOfBoundsException: -1

Você acabou de tentar obter o número -1do elemento de uma matriz. A contagem começa em zero.

Embora eu concorde que a resposta aceita geralmente é a melhor solução e definitivamente mais fácil de usar, percebi que ninguém exibiu o uso adequado do iterador. Então, aqui está um exemplo rápido:

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

ou

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

Mas tenha cuidado, ArrayList pode conter valores nulos . Portanto, a comparação deve ser

value.equals(arrayList.get(i))

quando tiver certeza de que o valor não é nulo ou você deve verificar se determinado elemento é nulo.

Você também pode usar desta forma:

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

É uma boa prática lançar e usar o objeto. Por exemplo, se 'arrayList' contém uma lista de objetos 'Object1'. Então, podemos reescrever o código como:

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

Você também pode fazer um loop for como faria para um array, mas em vez de array [i], você usaria list.get (i)

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

Além da resposta de larsmans (que de fato está correta), a exceção em uma chamada para um método get (), então o código que você postou não é o que está causando o erro.

Maneira eficiente de iterar seu ArrayListseguido por este link . Este tipo irá melhorar o desempenho do loop durante a iteração

int size = list.size();

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

iterar usando o iterador não é seguro contra falhas, por exemplo, se você adicionar um elemento à coleção após a criação do iterador, ele lançará concurrentmodificaionexception. Além disso, não é thread-safe, você deve torná-lo thread-safe externamente.

Portanto, é melhor usar for-each estrutura de loop for. É pelo menos à prova de falhas.