[Pharo-dev] Missing collectAsSet: in Pharo 3.0

Nicolas Cellier nicolas.cellier.aka.nice at gmail.com
Tue Dec 10 08:47:45 EST 2013


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?


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
> > >>
> > >
> > >
> >
> >
> >
> >
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.pharo.org/pipermail/pharo-dev_lists.pharo.org/attachments/20131210/b26552de/attachment-0002.html>


More information about the Pharo-dev mailing list