[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: scribus export.py



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


Reply to: