On Wed, Dec 30, 2020 at 05:10:21PM +0100, Federico Di Gregorio wrote: > Comunque quel pezzo di codice fa schifo: > > try: > if textfile == '': > raise Exception > exportText(textfile) > except Exception, e: > print e > > Tira una eccezione vuota al solo sopo di stampare... niente. :) > > Molto meglio così: > > if textfile == '': > print("Non hai specificato il nome del file") > else: > exportText(textfile) Non solo. nel branch in try c'è anche tutto `exportTest()` che potrebbe tirare eccezioni da qualche parte, quindi ha comunque senso un try/except. -- regards, Mattia Rizzolo GPG Key: 66AE 2B4A FCCF 3F52 DA18 4D18 4B04 3FCD B944 4540 .''`. More about me: https://mapreri.org : :' : Launchpad user: https://launchpad.net/~mapreri `. `'` Debian QA page: https://qa.debian.org/developer.php?login=mattia `-
Attachment:
signature.asc
Description: PGP signature