Java - Итерируемый с собственными hasNext() и next(): как это сделать правильно?

Question

У меня есть класс, который реализует Iterable, но с собственными hasNext() и next() методами, как это:

public class BLMovie implements Iterable<BLMovieFrame> { 

@Override 
public Iterator<BLMovieFrame> iterator() { 
    iteratorVar = 0; 
    return liste.iterator(); 
} 

public boolean hasNext() { 
    if (loop) 
     return true; 
    return iteratorVar < liste.size(); 
} 

public BLMovieFrame next() { 
    iteratorVar = (iteratorVar++)%(liste.size()-1); 
    return liste.get(iteratorVar); 
} 

public void remove() { 
    throw new UnsupportedOperationException(); 
} 

Я довольно уверен, что это не самый лучший способ сделать это,
другие темы о Iterable не похоже, беспокоятся о собственных методах, так кто-то имеет для меня совет?

Answer 1

Ваши методы определены в неправильном классе. Методы hasNext, next и remove должны быть определены в реализации Iterator, а не на Iterable. Если вы разместили аннотацию @Override для каждого метода, компилятор проинформировал вас об этой ошибке.

Как это, вы можете:

• Создать анонимный внутренний класс для итератора, и перенести эти методы в тело
• Создать автономный класс для итератора, и еще раз, шаг эти методы в него

Обратите внимание: если вы сделаете это, вы больше не сможете использовать итератор встроенного списка, но вы будете эффективно определять свои собственные (хотя вы можете его обернуть, конечно).

Answer 2

На самом деле вы не будете обрабатывать свои методы итератора, так как вы фактически возвращаете итератор своего объекта liste.

Если вы хотите реализовать свой собственный итератор, лучший способ - создать анонимный внутренний класс.

@Override 
     public Iterator<BLMovieFrame> iterator() { 
      // TODO Auto-generated method stub  
      return new Iterator <BLMovieFrame>(){ 

       @Override 
       public boolean hasNext() { 
        if (loop) 
         return true; 
        return iteratorVar < liste.size(); 
       } 

       @Override 
       public BLMovieFrame next() { 
         iteratorVar = (iteratorVar++)%(liste.size()-1); 
       return liste.get(iteratorVar); 
       } 
      @Override 
      public void remove() { 
        throw new UnsupportedOperationException(); 
      }    
      };