Question

我正在尝试创建自己的迭代器，它遍历由MenuItems组成的Menu对象的ArrayList。每个menuItem都有4个值。我试图遍历arrayList并只返回具有类别值MainDish的值。我一直在无限循环。它必须在我的迭代器的next（）方法中实现迭代器接口，但我不能在我的生活中找到错误的位置。它必须是我增加currentIndex的位置，但无法弄明白。任何和所有的帮助表示赞赏。

迭代器类：

package menu;
import java.util.ArrayList;
import java.util.Iterator;
import java.util.NoSuchElementException;


public class ItemIterator implements Iterator<MenuItem> {
private ArrayList<MenuItem> menuList;
private int currentIndex;
private String type;

public ItemIterator(ArrayList<MenuItem> menuList, String type) {
    this.menuList = menuList;
    this.type = type;
    }

@Override
public boolean hasNext() {
    return !(menuList.size() == currentIndex);
}

@Override
public MenuItem next() {

boolean found = false;

    if(hasNext() && !found)
        if(menuList.get(currentIndex).getCategory().equals(type))   
            found = true;
        else
            currentIndex++;         


    if(found = true)
        return menuList.get(currentIndex);
    else
        throw new NoSuchElementException();
    }   


@Override
public void remove() {
    // TODO Auto-generated method stub

}

 }

这是我的主要内容：

     public static void main(String[] args) {


    MenuItem item1 = new MenuItem("burger", mainDish, false, 10);
    MenuItem item2 = new MenuItem("sandwhich", appetizer, true, 5);

    Menu newMenu = new Menu();

    newMenu.add(item1);
    newMenu.add(item2);




     Iterator<MenuItem> itr = newMenu.getMenuIterator(); 
     System.out.println("ALL MENU ITEMS"); 


     while (itr.hasNext()) 
     { 
     System.out.println(itr.next()); 
     } 

    itr = newMenu.getItemIterator(mainDish); 
     System.out.println("ALL MAIN DISH ITEMS"); 

     while (itr.hasNext()) 
     { 
     System.out.println(itr.next()); 
     } 
  }

Answer 1

next（）方法需要增加currentIndex，无论当前项是否匹配。如上所述，一旦达到实际匹配，就会得到无限循环。

顺便说一下，如上所述，没有理由不使用常规迭代器并检查结果。如果你真正想要的是一个向前跳到下一个匹配的迭代器，你需要在next（）方法中添加一个增量循环。

无限循环迭代ArrayList Java

1 个答案: