Vaut-il mieux "essayer" quelque chose et attraper l'exception ou tester d'abord si c'est possible d'éviter une exception ?

Question

Dois-je tester if quelque chose est valable ou juste try pour le faire et attraper l'exception ?

• Existe-t-il une documentation solide indiquant qu'une voie est préférable ?
• Est-ce qu'un moyen est plus pythonique ?

Par exemple, devrais-je :

if len(my_list) >= 4:
    x = my_list[3]
else:
    x = 'NO_ABC'

Ou :

try:
    x = my_list[3]
except IndexError:
    x = 'NO_ABC'

---

Quelques réflexions...
PEP 20 dit :

Les erreurs ne doivent jamais passer en silence.
Sauf si elle est explicitement réduite au silence.

L'utilisation d'un try au lieu d'un if être interprété comme une erreur passant silencieusement ? Et si c'est le cas, est-ce que vous la réduisez explicitement au silence en l'utilisant de cette façon, ce qui la rend OK ?

---

Je suis no se référant à des situations où vous ne pouvez faire les choses que d'une seule façon ; par exemple :

try:
    import foo
except ImportError:
    import baz

Answer 1

Vous devriez préférer try/except sur if/else si cela entraîne

• des accélérations (par exemple en évitant des recherches supplémentaires)
• un code plus propre (moins de lignes/plus facile à lire)

Souvent, ils vont de pair.

---

accélère

Dans le cas où l'on essaie de trouver un élément dans une longue liste par :

try:
    x = my_list[index]
except IndexError:
    x = 'NO_ABC'

l'essai, sauf est la meilleure option lorsque le index se trouve probablement dans la liste et l'IndexError n'est généralement pas soulevé. De cette façon, vous évitez la nécessité d'une recherche supplémentaire par if index < len(my_list) .

Python encourage l'utilisation des exceptions, que vous manipulez est une phrase de Plongez dans Python . Votre exemple ne se contente pas de gérer l'exception (de manière élégante), au lieu de la laisser se produire. passer en silence En outre, l'exception ne se produit que dans le fichier exceptionnel cas d'index non trouvé (d'où le mot exception !).

---

code plus propre

La documentation officielle de Python mentionne EAFP : Il est plus facile de demander le pardon que la permission. y Rob Knight note que détecter les erreurs plutôt que de les éviter peut donner lieu à un code plus propre et plus facile à lire. Son exemple le dit ainsi :

Pire (LBYL "regarde avant de sauter") :

#check whether int conversion will raise an error
if not isinstance(s, str) or not s.isdigit():
    return None
elif len(s) > 10:    #too many digits for int conversion
    return None
else:
    return int(s)

Meilleur (EAFP : Il est plus facile de demander le pardon que la permission) :

try:
    return int(s)
except (TypeError, ValueError, OverflowError): #int conversion failed
    return None

Answer 2

Dans ce cas particulier, vous devriez utiliser quelque chose d'entièrement différent :

x = myDict.get("ABC", "NO_ABC")

En général, cependant : si vous vous attendez à ce que le test échoue fréquemment, utilisez if . Si le test est coûteux par rapport au simple fait d'essayer l'opération et d'attraper l'exception en cas d'échec, utilisez l'option try . Si aucune de ces conditions ne s'applique, choisissez ce qui se lit le mieux.

Answer 3

S'il est trivial de vérifier si quelque chose va échouer avant de le faire, vous devriez probablement le favoriser. Après tout, la construction d'exceptions (y compris leurs traces associées) prend du temps.

Les exceptions doivent être utilisées pour :

1. des choses qui sont inattendues, ou...
2. les choses pour lesquelles vous devez sauter plus d'un niveau de logique (par exemple, lorsqu'une break ne vous mène pas assez loin), ou...
3. les choses pour lesquelles vous ne savez pas exactement ce qui va gérer l'exception à l'avance, ou...
4. les choses pour lesquelles la vérification préalable de l'échec est coûteuse (par rapport à la simple tentative de l'opération)

---

Notez que souvent, la vraie réponse est "ni l'un ni l'autre" - par exemple, dans votre premier exemple, ce que vous avez réellement devrait est simplement d'utiliser .get() pour fournir une valeur par défaut :

x = myDict.get('ABC', 'NO_ABC')

Answer 4

Comme le mentionnent les autres messages, cela dépend de la situation. Il y a quelques dangers à utiliser try/except au lieu de vérifier la validité de vos données à l'avance, surtout lorsque vous l'utilisez sur des projets plus importants.

• Le code dans le bloc d'essai peut avoir la possibilité de faire toutes sortes de dégâts avant que l'exception ne soit détectée - si vous vérifiez de manière proactive à l'avance avec une instruction if, vous pouvez éviter cela.
• Si le code appelé dans votre bloc d'essai lève un type d'exception commun, comme TypeError ou ValueError, il se peut que vous n'attrapiez pas réellement l'exception que vous vous attendiez à attraper - il se peut que ce soit autre chose qui lève la même classe d'exception avant ou après avoir atteint la ligne où votre exception peut être levée.

par exemple, supposez que vous ayez :

try:
    x = my_list[index_list[3]]
except IndexError:
    x = 'NO_ABC'

L'IndexError ne dit pas si elle s'est produite en essayant d'obtenir un élément de index_list ou de my_list.

Answer 5

L'utilisation d'un try au lieu d'un if doit-elle être interprétée comme une erreur passant silencieusement ? Et si c'est le cas, est-ce que vous la réduisez explicitement au silence en l'utilisant de cette façon, ce qui la rend acceptable ?

Utilisation de try c'est reconnaître qu'une erreur peut passer, ce qui est le contraire de la faire passer silencieusement. Utilisation de except fait qu'il ne passe pas du tout.

Utilisation de try: except: est préférable dans les cas où if: else: La logique est plus compliquée. Le simple est mieux que le complexe ; le complexe est mieux que le compliqué ; et il est plus facile de demander le pardon que la permission.

L'avertissement "les erreurs ne devraient jamais passer silencieusement" concerne le cas où le code pourrait lever une exception que vous connaissez, et où votre conception admet la possibilité, mais vous n'avez pas conçu un moyen de traiter l'exception. Silencier explicitement une erreur, à mon avis, serait de faire quelque chose comme pass dans un except ce qui ne devrait être fait que si l'on comprend que "ne rien faire" est vraiment le traitement correct de l'erreur dans cette situation particulière. (C'est l'une des rares fois où j'ai l'impression qu'un commentaire dans un code bien écrit est probablement vraiment nécessaire).

Cependant, dans votre exemple particulier, aucun des deux n'est approprié :

x = myDict.get('ABC', 'NO_ABC')

La raison pour laquelle tout le monde le souligne - même si vous reconnaissez votre désir de comprendre en général, et votre incapacité à trouver un meilleur exemple - est que des pas de côté équivalents existent en fait dans un grand nombre de cas, et que les rechercher est la première étape de la résolution du problème.