ArrayListの反復子を使用すると、ArrayIndexOutOfBoundsException

現在、次のようなコードを含むプログラムがあります。

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

ArrayListを反復処理する限り、私はそれを正しく行っていますか？

私が得ているエラーは：

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)

残りのコードを表示しますが、それはかなり広範であり、反復を正しく実行していない場合、ArrayList適切に初期化していないことが唯一の可能性であると想定します。

Arraylistを反復処理する限り、私はそれを正しく行っていますか？

いいえ：iterator各反復で2回呼び出すことにより、常に新しいイテレーターを取得します。

このループを記述する最も簡単な方法は、for-each構文を使用することです。

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

はどうかと言うと

java.lang.ArrayIndexOutOfBoundsException: -1

-1配列から要素番号を取得しようとしました。カウントはゼロから始まります。

受け入れられた回答が通常最良の解決策であり、間違いなく使いやすいことに同意しますが、イテレータの適切な使用法を誰も表示していないことに気付きました。だからここに簡単な例があります：

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

または

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

ただし、ArrayListはnull値を保持できることに注意してください。したがって、比較は

value.equals(arrayList.get(i))

値がnullでないことが確実な場合、または指定された要素がnullかどうかを確認する必要がある場合。

次のように使用することもできます：

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

オブジェクトをキャストして使用することをお勧めします。たとえば、 'arrayList'に 'Object1'オブジェクトのリストが含まれている場合。次に、コードを次のように書き換えます。

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

配列の場合と同様にforループを実行することもできますが、array [i]の代わりにlist.get（i）を使用します

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

ラースマンの回答（確かに正しい）とは別に、get（）メソッドの呼び出しの例外。そのため、投稿したコードはエラーの原因ではありません。

ArrayListこのリンクをたどってあなたをフォローする効率的な方法。このタイプは、反復中のループのパフォーマンスを向上させます

int size = list.size();

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

イテレータを使用した反復はフェイルセーフではありません。たとえば、イテレータの作成後に要素をコレクションに追加すると、concurrentmodificaionexceptionがスローされます。また、スレッドセーフではありません。外部でスレッドセーフにする必要があります。

したがって、forループのfor-each構造を使用することをお勧めします。少なくともフェイルセーフです。