ArrayIndexOutOfBoundsException przy użyciu ArrayList jest iterator

Question

Teraz mam program zawierający fragment kodu, który wygląda tak:

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

robię to prawo, o ile iteracja ArrayList idzie?

Błąd otrzymuję to:

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) 

chciałbym pokazać resztę kodu, ale jest to dość obszerna, a jeśli nie mam właściwie robić iteracji, chciałbym założyć, jedyną możliwością jest to, że Nie zainicjalizuję poprawnie ArrayList.

Answer 1

Czy robię to dobrze, o ile chodzi o powtarzanie przez tablicę?

Nie: dzwoniąc pod numer iterator dwa razy w każdej iteracji, cały czas otrzymujesz nowe iteratory.

Najłatwiej napisać pętlę używa for-each konstrukt:

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

jak dla

java.lang.ArrayIndexOutOfBoundsException: -1

Po prostu starał się elementu numer -1 z tablicy . Liczenie rozpoczyna się od zera.

Answer 2

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

lub

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

Ale bądź ostrożny ArrayList może posiadać wartości null. Tak więc porównanie powinno być

value.equals(arrayList.get(i)) 

gdy jesteś pewien, że wartość nie jest zerowa lub powinieneś sprawdzić, czy dany element ma wartość null.

Answer 3

Oprócz odpowiedzi larsmans (kto jest rzeczywiście poprawny), wyjątek w wywołaniu metody get(), więc kod, który wysłałeś, nie jest tym, który powoduje błąd.

Answer 4

Można również zrobić dla pętli jak dla tablicy, ale zamiast array [i] należałoby użyć list.get (i)

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

Answer 5

Chociaż zgadzam się, że przyjęte rozwiązanie jest zazwyczaj najlepsze rozwiązanie i zdecydowanie łatwiejsze w obsłudze, zauważyłem, że nikt nie wyświetla poprawnego użycia iteratora. Więc tutaj jest krótki przykład:

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

Answer 6

Można również użyć tak:

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

Jest to dobra praktyka, aby rzucić i korzystania z obiektu. Na przykład, jeśli 'arrayList' zawiera listę obiektów 'Object1'.Wtedy możemy ponownie napisać kod jako:

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

Answer 7

skutecznym sposobem iteracyjne swój ArrayList następnie ten link. Ten typ będzie poprawić wydajność podczas iteracji pętli

int size = list.size(); 

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

Answer 8

iteracji za pomocą iteratora nie jest fail-safe na przykład jeśli dodać element do kolekcji po utworzeniu iteratora to będzie rzucać concurrentmodificaionexception. Ponadto nie jest bezpieczny dla wątków, musisz nosić go bezpiecznie na zewnątrz.

Lepiej więc użyć dla każdej struktury pętli for. Jest przynajmniej bezpieczny w razie awarii.