[Pharo-dev] Missing collectAsSet: in Pharo 3.0

Igor Stasenko siguctua at gmail.com
Tue Dec 10 09:08:44 EST 2013


On 10 December 2013 14:47, Nicolas Cellier <
nicolas.cellier.aka.nice at gmail.com> wrote:

> A fact is that there are way too many methods in (Sequenceable)Collection
> already, with duplicated implementations/features.
> So if collect:as: does the job why the hell add yet another selector?
>
> +1


>
> 2013/12/10 Roberto Minelli <roberto.minelli at usi.ch>
>
>> IMHO having two common usages of #flatCollect:as: and #collect:as: is
>> good. Then, do whatever you want. In my image I have #collectAsSet:.
>>
>> On Dec 10, 2013, at 2:40 PM, Esteban Lorenzano <estebanlm at gmail.com>
>> wrote:
>>
>> > IMO, if there is an error, then is the presence of #flatCollectAsSet:,
>> not the absence of  the other(s).
>> >
>> > Esteban
>> >
>> >
>> > On Tue, Dec 10, 2013 at 2:38 PM, Nicolas Cellier <
>> nicolas.cellier.aka.nice at gmail.com> wrote:
>> > And then rejectAsSet: selectAsHeap: etc...
>> > A big -1
>> >
>> >
>> > 2013/12/10 Yuriy Tymchuk <yuriy.tymchuk at me.com>
>> >
>> > On 10 Dec 2013, at 14:26, Roberto Minelli <roberto.minelli at usi.ch>
>> wrote:
>> >
>> > > No. Since #flatCollectAsSet: is in the image, #collectAsSet: should
>> be in the image as well.
>> > >
>> > > And then, since we want clean code #collectAsSet: is cleaner than
>> #collect:as:.
>> >
>> > Yes, but this also means that you need to implement:
>> >
>> > collectAsArray:
>> > collectAsOrderedCollection:
>> > collectAsBag:
>> > …
>> >
>> > :)
>> >
>> > >
>> > > On Dec 10, 2013, at 2:23 PM, Esteban Lorenzano <estebanlm at gmail.com>
>> wrote:
>> > >
>> > >> I don't think it has to be in the image,
>> > >>
>> > >> After all, you can just replace your calls for #collect:as:
>> > >>
>> > >> Esteban
>> > >>
>> > >>
>> > >>
>> > >>
>> > >> On Tue, Dec 10, 2013 at 2:21 PM, Roberto Minelli <
>> roberto.minelli at usi.ch> wrote:
>> > >> Hi,
>> > >>
>> > >> In Pharo 3 you guys integrated Doru’s CollectionsExtension package,
>> which is great.
>> > >>
>> > >> You forgot, however, to integrate the utility method #collectAsSet:
>> I was using in several places in my code.
>> > >>
>> > >> It should be simply
>> > >> ^ self collect: aBlock as: Set
>> > >>
>> > >> Could someone please put this in the standard image please? Thanks.
>> > >>
>> > >> Cheers,
>> > >> R
>> > >>
>> > >
>> > >
>> >
>> >
>> >
>> >
>>
>>
>>
>


-- 
Best regards,
Igor Stasenko.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.pharo.org/pipermail/pharo-dev_lists.pharo.org/attachments/20131210/ad289014/attachment-0002.html>


More information about the Pharo-dev mailing list