1

我有一个包含元素 1 到 10 的列表。我尝试从中删除素数 2、3、5、7，然后使用迭代器打印列表的其余部分。但是这段代码抛出了 NoSuchElementException。这是我的代码：

public static void editerate2(Collection<Integer> list3)
{
    Iterator<Integer> it=list3.iterator();
    while(it.hasNext())
    {
        if(it.next()==2 || it.next()==3 || it.next() ==5 || it.next()==7 ) 
        {
            it.remove();
        }
    }
    System.out.println("List 3:");
    System.out.println("After removing prime numbers  : " + list3);
}

这样做的正确方法是什么？还有使用“|”有什么区别 和“||” ？？？

4

2 回答 2

6

每次调用it.next()迭代器时都会前进到下一个元素。

我假设这不是你想做的。

你应该这样做：

Iterator<Integer> it = list.iterator();

while (it.hasNext()) {
    Integer thisInt = it.next();
    if (thisInt == 2 || thisInt == 3 || thisInt == 5 || thisInt == 7) {
       it.remove();
    }
}

之间的区别 | 和||：

如果您使用||并且第一部分为真，则不会评估第二部分。

如果您使用|这两个部分，将始终进行评估。

这对于这样的情况很方便：

if (person == null || person.getName() == null) {
    // do something
}

|如果您使用并且 person 为 null ，上述代码段将抛出 NullPointerException 。

那是因为它将评估条件的两个部分，而后半部分将取消引用空对象。

于 2012-12-12T02:30:44.687 回答
1

您希望避免多次调用您的迭代器，因为这会将其推进到下一个元素。

您可以做的是保留每次迭代获得的值，然后进行比较。

while(it.hasNext()) {
    Integer next = it.next();
    if(next == 2 || next == 3 || next == 5 || next == 7 ) {
        it.remove();
    }
}
于 2012-12-12T02:33:53.940 回答