通过ArrayList迭代的无限循环Java

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

迭代器类：

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()); 
    } 
    }

来源

2014-02-21 Jameswr.11

'if（found = true）'< - 这里。应该是'=='。或者只是“如果（找到）”。 – fge

为什么要使用迭代器？您可以更简单地使用for循环的用户吗？ – TheBrenny

啊，把它改为只要（发现）有帮助，但现在我得到了没有这样的元素例外，我推它作为默认情况。想法？ –

下一个（）方法需要增加CURRENTINDEX当前项是否匹配。正如所写的，只要你达到真正的比赛，你就会得到一个无限循环。

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

来源

2014-02-21 06:07:51

我现在得到了没有这样的元素例外，我把它作为默认情况。 –

现在，只要当前元素不匹配，并且您调用next（），就会得到NoSuchElementException。 –

我想我的下一个任务是弄清楚循环结构的位置 –

通过ArrayList迭代的无限循环Java

回答

相关问题