incubator-chemistry-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Klevenz, Stephan" <>
Subject RE: Iterable
Date Wed, 05 May 2010 12:47:26 GMT
Ok, this is done as David has suggested.

The implementation requires some little clean up but that is just a minor task and should
not have further effect to API or functionality. 


-----Original Message-----
From: Florian Müller [] 
Sent: Mittwoch, 5. Mai 2010 12:34
Subject: RE: Iterable


I agree with David, keep getPageNumItems().


-----Original Message-----
From: David Caruana [] 
Sent: Mittwoch, 5. Mai 2010 11:37
Subject: Re: Iterable

On 5 May 2010, at 09:59, Klevenz, Stephan wrote:

> Hi,
> I did fix this issue and decided to re-name the Iterable to ItemIterable. I'm not sure
if we have a consensus on the naming, but the naming can change again easily.
> Furthermore I would like to support Florents idea to eliminate the Iterator interface
and would propose the following:

If you mean collapse Iterator and Iterable then +1.

> ItemIteraror to be eliminated
>        long getPosition() -> eliminate method, only required for implementation


>        long getPageNumItems() -> eliminate -> no use case

The use case is to determine how many items are in the collection backed by the iterator without
iterating through them all first. For example, to display the number of items (or range of
items) in the current page of query results. I don't know of any other way.

>        ling getTotalNumItems() -> move to ItemIterable


> Regards,
> Stephan

View raw message