#withOccurrencesDo: matches better the rest of the pharo system (#withIndexDo:, etc) :)

On Aug 12, 2013, at 9:16 AM, St�phane Ducasse <stephane.ducasse@inria.fr> wrote:

On Aug 12, 2013, at 1:49 AM, Gabriel Cotelli <g.cotelli@gmail.com> wrote:

Why not #doWithOccurrences: ??

ok it was a proposal for a new selector :)

Eg:

bag doWithOccurrences: [:element :count | "do something"]

I dont'like the keysAndValuesDo: idea, is just an implementation detail.

On Sun, Aug 11, 2013 at 4:52 PM, St�phane Ducasse <stephane.ducasse@inria.fr> wrote:

Hi guys

I was surprised to see the API of bag for iteration.

I propose to add

keysAndValuesDo: aBlock
        "Iterate over the receiver and apply a two argument block on the element and its occurrences."

        contents associationsDo: [:assoc | aBlock value: assoc key value: assoc value ]

and

elementsAndOccurrencesDo: aTwoArgBlock
        "Iterate over the receiver and apply a two argument block on the element and its occurrences."

        contents associationsDo: [:assoc | aTwoArgBlock value: assoc key value: assoc value ]

Once we agree I will add a bug entry.

Stef