Вращающийся список влево и вправо

Question

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

public class Test { 

    private char heading = 'N'; 
    List<Character> cardinal = Arrays.asList('N', 'E', 'S', 'W'); 
    ListIterator<Character> iterator = cardinal.listIterator(); 

    public static void main(String[] args) { 

     Test test = new Test(); 
     System.out.println("heading = " + test.heading); 

     test.rotateLeft(); System.out.println("Rotating left..."); 
     System.out.println("heading = " + test.heading); 
     test.rotateLeft(); System.out.println("Rotating left..."); 
     System.out.println("heading = " + test.heading); 
    } 

    public void rotateRight() { 
     this.heading = 
      (iterator.hasNext() ? iterator.next() : cardinal.get(0)); 
    } 

    public void rotateLeft() { 
     this.heading = 
       (iterator.hasPrevious() ? iterator.previous() : cardinal.get(3)); 
    } 

} 

Answer 1

Проблема в том, что, когда вы достигаете конца списка, вы не сбрасываете итератор. С этого момента iterator.hasNext() всегда будет возвращать false. Легко исправить:

public void rotateRight() { 
    if (!iterator.hasNext()) 
     iterator = cardinal.listIterator(); 
    this.heading = iterator.next(); 
} 

public void rotateLeft() { 
    if (!iterator.hasPrevious()) 
     iterator = cardinal.listIterator(cardinal.size()); 
    this.heading = iterator.previous(); 
} 

Если ваш список final, становится еще проще:

int index = 0; 

public void rotateRight() { 
    index = (index + 1) % cardinal.size(); 
    this.heading = cardinal.get(index); 
} 

public void rotateLeft() { 
    index = (index - 1 + cardinal.size()) % cardinal.size(); 
    this.heading = cardinal.get(index); 
}