Problema de iterador da classe interna java
-
18-09-2019 - |
Pergunta
Estou tendo dificuldade em usar um interior Iterator
.
private List<List<? extends HasWord>> sentences = new ArrayList<List<? extends HasWord>>();
private Iterator<String> wordIterator = new Words();
private class Words implements Iterator<String> {
int currSentence = 0;
int currWord = 0;
@Override
public boolean hasNext() {
return currSentence != sentences.size() - 1 && currWord != sentences.get(currSentence).size() - 1;
}
@Override
public String next() {
String nextWord = sentences.get(currSentence).get(currWord).word();
currSentence++;
currWord++;
return nextWord;
}
@Override
public void remove() {
throw new UnsupportedOperationException();
}
}
Então, eu tento itera sobre isso:
for (String s : wordIterator) { //Error: Can only iterate over an array or an instance of java.lang.Iterable
words.add(s);
Mas não funciona. (Veja o erro do compilador comentado na linha problemática). O que eu estou fazendo errado aqui?
Em uma nota de engenharia, o DO é o caminho certo para resolver meu problema? Eu tenho um monte de loops deste formulário:
for (List<? extends HasWord> sent : sentences) {
for (HasWord token : sent) {
//do stuff
}
}
Então eu decidi um Iterator
seria mais limpo. Isso é um exagero, ou há outra maneira de fazer isso?
Solução
Não há nada fundamentalmente errado em ter dois aninhados for
Loops para fazer isso, mas acho que isso seria mais limpo:
public class Words implements Iterator<String> {
private final Iterator<HasWord> sentences;
private Iterator<String> currentSentence;
public boolean hasNext() {
return currentSentence.hasNext() || sentences.hasNext();
}
public String next() {
if (currentSentence.hasNext()) {
return currentSentence.next();
}
currentSentence = sentences.next();
return next(); // will return the first word of the next sentence
}
//remove() omitted for brevity
}
Devolver uma nova instância desta classe toda vez que você precisar de um iterador por várias frases e inicialize o sentences
campo usando sentences.iterator();
(Editado depois de ler sua pergunta com mais cuidado)
Outras dicas
private class Words implements Iterator<String>, Iterable<String> {
...
public Iterator<String> iterator() {
return this;
}
...
}