This iterator handles some common edge conditions very poorly. It should be possible to create null iterators, or iterators that iterate over lists of length one. Moreover, the decision to make the iterator non-reusable, and segreate the reusability functionality to a subclass is silly, given that this functionality is achieved simply by setting an internal cursor variable equal to 0. It is hard to see what benefit one derives from not having this feature in Array::Iterator in the first place.
4 out of 4 found this review helpful. Was this review helpful to you? Yes No