Meilleur moyen de gérer list.index(might-not-exist) en python ?

Question

J'ai un code qui ressemble à quelque chose comme ceci :

thing_index = thing_list.index(thing)
otherfunction(thing_list, thing_index)

Ok, c'est simplifié mais vous voyez l'idée. Maintenant thing pourrait ne pas être dans la liste, dans ce cas je veux passer -1 comme thing_index . Dans d'autres langues, c'est ce à quoi on s'attendrait index() à retourner s'il ne trouvait pas l'élément. En fait, il lance un ValueError .

Je pourrais le faire :

try:
    thing_index = thing_list.index(thing)
except ValueError:
    thing_index = -1
otherfunction(thing_list, thing_index)

Mais ça semble sale, en plus je ne sais pas si ValueError pourrait être soulevée pour une autre raison. J'ai trouvé la solution suivante, basée sur des fonctions de générateur, mais elle semble un peu complexe :

thing_index = ( [(i for i in xrange(len(thing_list)) if thing_list[i]==thing)] or [-1] )[0]

Existe-t-il un moyen plus propre d'obtenir la même chose ? Supposons que la liste ne soit pas triée.

Answer 1

thing_index = thing_list.index(elem) if elem in thing_list else -1

Une ligne. Simple. Pas d'exceptions.

Answer 2

Il n'y a rien de "sale" à utiliser la clause try-except. C'est la méthode pythonique. ValueError sera soulevée par le .index uniquement, car c'est le seul code que vous avez là !

Pour répondre au commentaire :
En Python, il est plus facile de demander le pardon que d'obtenir la permission la philosophie est bien établie, et pas de index ne soulèvera pas ce type d'erreur pour d'autres problèmes. Je ne pense pas qu'il y en ait.

Answer 3

El dict type a un get función où, si la clé n'existe pas dans le dictionnaire, le deuxième argument de la commande get est la valeur qu'il doit retourner. De même, il existe setdefault qui renvoie la valeur dans le champ dict si la clé existe, sinon elle définit la valeur en fonction de votre paramètre par défaut et renvoie ensuite votre paramètre par défaut.

Vous pourriez étendre le list pour avoir un getindexdefault méthode.

class SuperDuperList(list):
    def getindexdefault(self, elem, default):
        try:
            thing_index = self.index(elem)
            return thing_index
        except ValueError:
            return default

Qui pourrait alors être utilisé comme :

mylist = SuperDuperList([0,1,2])
index = mylist.getindexdefault( 'asdf', -1 )

Answer 4

Il n'y a rien de mal dans votre code qui utilise ValueError . Voici encore une autre phrase, si vous voulez éviter les exceptions :

thing_index = next((i for i, x in enumerate(thing_list) if x == thing), -1)

Answer 5

Cette question relève de la philosophie du langage. En Java, par exemple, il y a toujours eu une tradition selon laquelle les exceptions ne devraient être utilisées que dans des "circonstances exceptionnelles", c'est-à-dire lorsque des erreurs se sont produites, plutôt que pour contrôle du débit . Au début, c'était pour des raisons de performance, car les exceptions Java étaient lentes, mais aujourd'hui, c'est devenu le style accepté.

En revanche, Python a toujours utilisé des exceptions pour indiquer le déroulement normal du programme, comme la levée d'un message de type ValueError comme nous en discutons ici. Il n'y a rien de "sale" à ce sujet dans le style Python et il y en a beaucoup d'autres là d'où il vient. Un exemple encore plus courant est StopIteration exception qui est soulevée par la méthode de l'itérateur. next() pour signaler qu'il n'y a pas d'autres valeurs.