Quel style de retour est "meilleur" pour une méthode qui peut renvoyer None?

Question

J'ai une méthode qui retournera un objet ou None si la recherche échoue. Quel style de ce qui suit est le meilleur?

def get_foo(needle): 
    haystack = object_dict() 
    if needle not in haystack: return None 
    return haystack[needle] 

ou,

def get_foo(needle): 
    haystack = object_dict() 
    try: 
     return haystack[needle] 
    except KeyError: 
     # Needle not found 
     return None 

Je suis indécis quant à ce qui est plus plus souhaitable moi-même. Un autre choix serait return haystack[needle] if needle in haystack else None, mais je ne suis pas sûr que ce soit mieux.

Answer 1

Pour cet exemple, il ressemble à la méthode dictget est la plus concise:

def get_foo(needle): 
    haystack = object_dict() 
    return haystack.get(needle) 

En général, en Python, les gens ont tendance à préférer essayer/sauf que de vérifier quelque chose d'abord - voir l'entrée EAFP dans le glossary. Notez que de nombreuses fonctions "test for membership" utilisent des exceptions dans les coulisses.

Je ne vais pas commencer une flamewar sur les mérites relatifs des multiples déclarations ou les alternatives :)

Answer 2

si les deux font exactement la même chose, choisir celui que vous aimez le plus et c'est plus facile à lire - j'aime la deuxième, mais les deux sont bons.

(peut-être un si cela est plus rapide que l'autre - donc si preformance est importante realy, tester et choisir le plus véloce)

Answer 3

Dans toutes les langues que j'ai utilisé la première version est perferable. Les exceptions sont essentiellement des goto déguisés avec plusieurs des mêmes problèmes, donc je ne les utilise pas si je peux l'éviter.

Il existe également un coût de performance possible. Je ne connais pas Python, mais dans beaucoup d'autres langues, il y a un coût important pour créer l'exception que vous allez jeter sur la ligne suivante.