order classVar in Stepper is suspect

Hi,
there is a class variable named order in Stepper.

First, it is well behaved to capitalize class var names

Second, I think there is a confusion between class variable and class side instanceVariable.
A class var is shared between all subclasses, and it's clearly not what we want here: each subclass should have its own order.

Third, finally, we don't need no class instance variable at all right now, because every subclass redefine order (the message).
So I suggest removing order (the class var) and implement order ^self subclassResponsibility in Stepper.

What do you think?

On Wednesday, April 30, 2014 1:01:38 AM UTC+2, Nicolas Cellier wrote:

Hi,
there is a class variable named order in Stepper.

First, it is well behaved to capitalize class var names

Second, I think there is a confusion between class variable and class side instanceVariable.
A class var is shared between all subclasses, and it's clearly not what we want here: each subclass should have its own order.

Third, finally, we don't need no class instance variable at all right now, because every subclass redefine order (the message).
So I suggest removing order (the class var) and implement order ^self subclassResponsibility in Stepper.

What do you think?