Boris Borcic wrote: > Slawomir Nowaczyk wrote: >> >> try: >> if int(text) > 0: >> return True >> except ValueError: >> pass >> self.error_message() >> return False >> > > Nicely DRY. To make it even more compact, it may be noticed that the > default return value None is false in a boolean context - so that the > last line is superfluous if the return value is only wanted to test it > in such a context.
In this case the method must return False, because it's a wxPython method that needs a True or False value. If it doesn't, the program will continue even after the error message. What I did to make it compact was have the error_message() method return False, so I can just call return self.error_message() -- http://mail.python.org/mailman/listinfo/python-list