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... Cheers, Alex On Wed, Sep 2, 2015 at 10:56 AM, Peter Uhnák <[email protected]> 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 >
