Dois-je toujours spécifier un type d'exception dans les instructions «except»?

Lorsque vous utilisez PyCharm IDE, l'utilisation de except:sans type d'exception déclenche un rappel de l'EDI indiquant que cette clause d'exception est Too broad.

Dois-je ignorer ce conseil? Ou est-ce pythonique de toujours spécifier le type d'exception?

Il est presque toujours préférable de spécifier un type d'exception explicite. Si vous utilisez une except:clause nue , vous pourriez finir par attraper des exceptions autres que celles que vous prévoyez d'attraper - cela peut masquer des bogues ou rendre plus difficile le débogage des programmes lorsqu'ils ne font pas ce que vous attendez.

Par exemple, si vous insérez une ligne dans une base de données, vous souhaiterez peut-être intercepter une exception indiquant que la ligne existe déjà, afin de pouvoir effectuer une mise à jour.

try:
    insert(connection, data)
except:
    update(connection, data)

Si vous spécifiez un bare except:, vous attraperez également une erreur de socket indiquant que le serveur de base de données est tombé. Il est préférable de n'attraper que les exceptions que vous savez gérer - il est souvent préférable que le programme échoue au point de l'exception que de continuer mais se comporte de manière étrange et inattendue.

Un cas où vous voudrez peut-être utiliser un bare except:est au niveau supérieur d'un programme dont vous devez toujours être en cours d'exécution, comme un serveur réseau. Mais alors, vous devez faire très attention à enregistrer les exceptions, sinon il sera impossible de déterminer ce qui ne va pas. Fondamentalement, il ne devrait y avoir au plus qu'un seul endroit dans un programme qui fait cela.

Un corollaire à tout cela est que votre code ne devrait jamais faire raise Exception('some message')car il oblige le code client à utiliser except:(ou except Exception:qui est presque aussi mauvais). Vous devez définir une exception spécifique au problème que vous souhaitez signaler (peut-être héritant d'une sous-classe d'exceptions intégrée comme ValueErrorou TypeError). Ou vous devez lever une exception intégrée spécifique. Cela permet aux utilisateurs de votre code d'être prudents en n'attrapant que les exceptions qu'ils souhaitent gérer.

Vous ne devez pas ignorer les conseils que vous donne l'interprète.

À partir du guide de style PEP-8 pour Python:

Lorsque vous interceptez des exceptions, mentionnez des exceptions spécifiques autant que possible au lieu d'utiliser une clause except: nue.

Par exemple, utilisez:

 try:
     import platform_specific_module 
 except ImportError:
     platform_specific_module = None 

Une clause except: nue interceptera les exceptions SystemExit et KeyboardInterrupt, rendant plus difficile l'interruption d'un programme avec Control-C, et peut masquer d'autres problèmes. Si vous voulez intercepter toutes les exceptions qui signalent des erreurs de programme, utilisez except Exception: (bare except équivaut à except BaseException :).

Une bonne règle d'or est de limiter l'utilisation de clauses d'exception nues à deux cas:

Si le gestionnaire d'exceptions imprimera ou enregistrera le traçage; au moins, l'utilisateur sera conscient qu'une erreur s'est produite. Si le code doit faire un travail de nettoyage, mais laisse ensuite l'exception se propager vers le haut avec augmentation. essayez ... enfin peut être une meilleure façon de gérer cette affaire.

Pas spécifique à Python ceci.

Le but des exceptions est de traiter le problème aussi près que possible de son origine.

Vous gardez donc le code qui pourrait dans des circonstances exceptionnelles déclencher le problème et la résolution "à côté" l'un de l'autre.

Le fait est que vous ne pouvez pas connaître toutes les exceptions qui pourraient être levées par un morceau de code. Tout ce que vous pouvez savoir, c'est que s'il s'agit d'une exception, par exemple un fichier non trouvé, vous pouvez la piéger et inviter l'utilisateur à en obtenir une qui exécute ou annuler la fonction.

Si vous mettez try catch round that, quel que soit le problème qu'il y avait dans votre routine de fichiers (lecture seule, autorisations, UAC, pas vraiment un pdf, etc.), tout le monde tombera dans votre fichier non trouvé catch, et votre utilisateur crie "mais il est là, ce code est de la merde"

Maintenant, il y a quelques situations où vous pourriez tout attraper, mais elles devraient être choisies consciemment.

Ils sont catch, annulez une action locale (comme créer ou verrouiller une ressource, (ouvrir un fichier sur le disque pour écrire par exemple), puis vous relancez l'exception, à traiter à un niveau supérieur)

L'autre vous, c'est que vous ne vous souciez pas de la raison pour laquelle cela a mal tourné. L'impression par exemple. Vous pourriez avoir un hic, pour dire qu'il y a un problème avec votre imprimante, veuillez le régler et ne pas tuer l'application à cause de cela. Si votre code exécutait une série de tâches séparées en utilisant une sorte de calendrier, vous ne voudriez pas que le tout meure, car l'une des tâches a échoué.

Remarque Si vous faites ce qui précède, je ne peux pas recommander une sorte de journalisation des exceptions, par exemple essayez la fin du journal de capture, assez fortement.

Vous attraperez également par exemple Control-C avec cela, alors ne le faites pas à moins que vous ne le "jetiez" à nouveau. Cependant, dans ce cas, vous devriez plutôt utiliser "enfin".

Toujours préciser le type d'exception, il existe de nombreux types que vous ne voulez pas attraper, comme SyntaxError, KeyboardInterrupt, MemoryErroretc.

Voici les endroits où j'utilise sauf sans type

1. prototypage rapide et sale

C'est la principale utilisation dans mon code pour les exceptions non vérifiées

2. fonction main () de niveau supérieur, où je consigne chaque exception non interceptée

J'ajoute toujours ceci, pour que le code de production ne déborde pas de stacktraces

3. entre les couches d'application

J'ai deux façons de le faire:

• Première façon de le faire: lorsqu'une couche de niveau supérieur appelle une fonction de niveau inférieur, elle encapsule les appels dans des excepts typés pour gérer les exceptions de niveau inférieur "supérieures". Mais j'ajoute une instruction except générique, pour détecter les exceptions de niveau inférieur non gérées dans les fonctions de niveau inférieur.

Je préfère cela, je trouve plus facile de détecter quelles exceptions auraient dû être capturées correctement: je "vois" mieux le problème lorsqu'une exception de niveau inférieur est enregistrée par un niveau supérieur

• Deuxième façon de le faire: chaque fonction de niveau supérieur des couches de niveau inférieur ont leur code enveloppé dans un générique, sauf qu'il capture toutes les exceptions non gérées sur cette couche spécifique.

Certains collègues préfèrent cette méthode, car elle conserve les exceptions de niveau inférieur dans les fonctions de niveau inférieur, auxquelles elles «appartiennent».

Essaye ça:

try:
    #code
except ValueError:
    pass

J'ai obtenu la réponse de ce lien, si quelqu'un d'autre rencontre ce problème Vérifiez-le