Come fare riferimento all'oggetto corrente in un iteratore

Question

Sto tentando di implementare un metodo di ricerca in un TreeSet. Usando un iteratore con un conditional mi piacerebbe essere in grado di correre attraverso il set e stampare l'oggetto che corrisponde alla condizione. Tuttavia, il modo in cui lo sto facendo al momento sta stampando l'oggetto successivo anziché la corrente. Questo è quello che ho finora:

public void getDetails() { 
     Iterator<Person> it = this.getPersonSet().iterator(); 
     System.out.println("Enter First Name"); 
     String first = in.next().toLowerCase(); 
     System.out.println("Enter Second Name"); 
     String last = in.next().toLowerCase(); 

     while (it.hasNext()) { 
      if (it.next().getLast().toLowerCase().equals(last)) { 
       Person p = it.next(); 
       System.out.println(p); 
      } 
     } 

    } 

Qualsiasi aiuto sarebbe grande

Answer 1

Questo è ciò che si vorrebbe fare:

while (it.hasNext()) { 
      Person p = it.next(); 
      if (p.getLast().toLowerCase().equals(last)) { 
       System.out.println(p); 
      } 
     } 

Answer 2

Tenere il riferimento dell'oggetto in un separato var:

Person current = it.next(); 
current.methodOne(); 
current.methodTwo(); 

Quando hai finito con il valore corrente, riassociare il prossimo

... 
// done? 
current = it.next(); 

In un ciclo assomiglia:

while(it.hasNext()) { 
    Person current = it.next(); 
    current.doA(); 
    current.doB(); 
    current.doC(); 
} 

Answer 3

Come posso fare riferimento all'oggetto corrente in un iteratore

Per la cronaca, l'API Iterator non consente di per fare questo. Non c'è idea di un oggetto "attuale". Il metodo Iterator.next() ti dà l'oggetto successivo ... e si sposta.

(I ListIterator.previous() e ListIterator.next() metodi sono analoghi. Si noti che nel caso ListIterator, comportamento metodo è documentato in termini di un cursore che indica una posizione prima/tra/dopo elementi nella sequenza essendo iterato.)

La soluzione è assegnare il risultato di chiamare it.next() a una variabile temporanea, come descritto dalla risposta accettata.

---

Non so per certo perché i progettisti non hanno incluso la nozione di un oggetto "corrente" in API, ma mi viene in mente un paio di motivi:

• E ' renderebbe un tipico oggetto iteratore più grande; vale a dire un campo in più per contenere l'oggetto corrente.
• Significherebbe più 1 metodo per implementare una classe Iterator.
• La nozione di oggetto corrente non si adatta bene al modello "cursore" documentato nell'interfaccia ListIterator ... e implicito nell'attuale versione Iterator.
• C'è un problema minore di Iterator "aggrappato" all'oggetto corrente, impedendo quindi che venga eseguito da GC.
• La maggior parte dei casi di utilizzo di iteratore non richiede un oggetto corrente.
• Ci sono altri modi per affrontare questo.

Suona come una buona chiamata ...

Answer 4

Se è necessaria un'implementazione esistente, è possibile utilizzare quelli da Google Guava o Apache Commons Collections.
Le altre risposte sono più facili per il tuo semplice problema, ma se hai bisogno di passare l'iteratore e di tenere traccia dell'ultimo elemento restituito da next(), ciò sarebbe di aiuto.

Ecco un esempio utilizzando Guava con il codice del PO (assumging Person anzi ha un metodo String toLowerCase()):

import com.google.common.collect.PeekingIterator; 
import static com.google.common.collect.Iterators.peekingIterator; 

public void getDetails() { 
    PeekingIterator<Person> it = peekingIterator(this.getPersonSet().iterator()); 
    System.out.println("Enter First Name"); 
    String first = in.next().toLowerCase(); 
    System.out.println("Enter Second Name"); 
    String last = in.next().toLowerCase(); 

    while (it.hasNext()) { 
     // note the usage of peek() instead of next() 
     if (it.peek().getLast().toLowerCase().equals(last)) { 
      Person p = it.next(); 
      System.out.println(p); 
     } 
    } 

}