The same naming blablaDo: is heavily used in Bloc and therefore in Brick. I personally like blablaDo: more, because it hides check logic from user and doesn't force to think about if-elses. It's all about "if" word. You know, when reader sees "if" a complexity of an app instantly rises (in the head).

But let's discuss :) I agree that it can be associated with iteration and is not the best idea...

On Wed, Sep 2, 2015 at 10:56 AM, Peter Uhn��k <i.uhnak@gmail.com> wrote:

Hi,

in Spec Adapter there's method

widgetDo: aBlock
^ self widget ifNotNil: aBlock

my question however is:
does the naming make sense?

#*Do: is generally associated with iterating over collections. Widget is always a single entity.
On the other hand we have #ifNil: #ifTrue: #ifFalse:

There is also #ifNotNilDo:, however that is superseded by #ifNotNil:

So from this perspective, wouldn't it make more sense to have this?
ifWidget: aBlock
�� ^ self widget ifNotNil: aBlock

What do you think?

Thanks,
Peter