I really don't think we should put methods that have side-effects into a short circuit logic. Can we use a traditional if statement here like 
```
if ( docIdSetIterator.docID() == target) {
  return true;
} else {
  return docIdSetIterator.advance(target) == target;
}
```