nacho wrote:
Camille,

I now understood what you've said. It makes sense and I've changed it that
way. The only thing I'm not happy with is that implementing:

QuadraticEquation>>calculateRoots
        ^ self calculateRootsIfNone: [ self error: 'No real solution for the
equation' ]

Makes the debugger appear and I would rather let the user know that there
are no real solutions. Bringing up the debugger can make one think that the
problem is in the code.

What do you think of:

QuadraticEquation>>calculateRoots
        ^ self calculateRootsIfNone: [ ^ 'No real solution for the equation' ]
Thanks!
nacho





-----
Nacho
Smalltalker apprentice.
Buenos Aires, Argentina.
--
View this message in context: 
http://forum.world.st/Question-on-style-tp4752165p4752388.html
Sent from the Pharo Smalltalk Users mailing list archive at Nabble.com.


How about... self inform: 'no real solutions'
cheers -ben

Reply via email to