June 18, 2013
12:53 p.m.
It feels to me as if this code wants to be on the temp. So perhaps,
| temp | temp := <a long expression>. temp <aComplexTest> ifTrue: [ temp <something> ] ifFalse: [ temp <something else> ].
can become <a long expression> doSomething and on Temp: doSomething self <aComplexTest> ifTrue: [ self <something> ] ifFalse: [ self <something else> ] It often works out nicer if you move the code around closer to where it belongs. Make <a long expression> a method somewhere and it may end up being simpler. my2c