¿La mejor manera de manejar list.index (podría no existir) en Python?

Tengo un código que se parece a esto:

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

ok, eso está simplificado, pero entiendes la idea. Es thingposible que ahora no esté en la lista, en cuyo caso quiero pasar -1 como thing_index. En otros idiomas, esto es lo que esperaría index()devolver si no pudiera encontrar el elemento. De hecho arroja un ValueError.

Yo podría hacer esto:

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

Pero esto se siente sucio, además de que no sé si se ValueErrorpodría plantear por alguna otra razón. Se me ocurrió la siguiente solución basada en funciones de generador, pero parece un poco compleja:

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

¿Existe una forma más limpia de lograr lo mismo? Supongamos que la lista no está ordenada.

No hay nada "sucio" en el uso de la cláusula try-except. Esta es la forma pitónica. ValueErrorsolo se generará con el .indexmétodo, ¡porque es el único código que tiene allí!

Para responder al comentario:
en Python, es más fácil pedir perdón que obtener permiso, la filosofía está bien establecida, y no index generará este tipo de error para otros problemas. No es que pueda pensar en ninguno.

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

Una línea. Sencillo. Sin excepciones.

El dicttipo tiene una getfunción , donde si la clave no existe en el diccionario, el segundo argumento getes el valor que debe devolver. De manera similar, hay setdefault, que devuelve el valor en el dictsi la clave existe; de ​​lo contrario, establece el valor de acuerdo con su parámetro predeterminado y luego devuelve su parámetro predeterminado.

Puede extender el listtipo para tener un getindexdefaultmétodo.

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

Que luego podría usarse como:

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

No hay nada de malo en el código que usa ValueError. Aquí hay otro resumen si desea evitar excepciones:

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

Este problema es de filosofía del lenguaje. En Java, por ejemplo, siempre ha existido la tradición de que las excepciones sólo deberían utilizarse en "circunstancias excepcionales", es decir, cuando se han producido errores, en lugar de para el control de flujo . Al principio, esto fue por razones de rendimiento, ya que las excepciones de Java eran lentas, pero ahora este se ha convertido en el estilo aceptado.

Por el contrario, Python siempre ha usado excepciones para indicar el flujo normal del programa, como generar un ValueErrorcomo estamos discutiendo aquí. No hay nada "sucio" en esto en estilo Python y hay muchos más de donde vino eso. Un ejemplo aún más común es la StopIterationexcepción que se genera mediante el next()método de un iterador para indicar que no hay más valores.

Si lo hace con frecuencia, es mejor que lo elimine en una función de ayuda:

def index_of(val, in_list):
    try:
        return in_list.index(val)
    except ValueError:
        return -1 

¿Qué pasa con esto?:

li = [1,2,3,4,5] # create list 

li = dict(zip(li,range(len(li)))) # convert List To Dict 
print( li ) # {1: 0, 2: 1, 3: 2, 4:3 , 5: 4}
li.get(20) # None 
li.get(1)  # 0 

¿Qué pasa con esto?

otherfunction(thing_collection, thing)

En lugar de exponer algo tan dependiente de la implementación como un índice de lista en una interfaz de función, pase la colección y la cosa y deje que otra función se ocupe de los problemas de "prueba de pertenencia". Si se escribe otra función para que sea independiente del tipo de colección, probablemente comenzaría con:

if thing in thing_collection:
    ... proceed with operation on thing

que funcionará si thing_collection es una lista, tupla, conjunto o dict.

Esto posiblemente sea más claro que:

if thing_index != MAGIC_VALUE_INDICATING_NOT_A_MEMBER:

que es el código que ya tiene en otra función.

¿Qué tal esto?

temp_inx = (L + [x]).index(x) 
inx = temp_inx if temp_inx < len(L) else -1

Tengo el mismo problema con el método ".index ()" en las listas. No tengo ningún problema con el hecho de que arroja una excepción, pero estoy totalmente en desacuerdo con el hecho de que es un ValueError no descriptivo. Sin embargo, podría entender si hubiera sido un IndexError.

Puedo ver por qué devolver "-1" también sería un problema porque es un índice válido en Python. Pero de manera realista, nunca espero que un método ".index ()" devuelva un número negativo.

Aquí va una línea (vale, es una línea bastante larga ...), recorre la lista exactamente una vez y devuelve "Ninguno" si no se encuentra el elemento. Sería trivial reescribirlo para devolver -1, si así lo desea.

indexOf = lambda list, thing: \
            reduce(lambda acc, (idx, elem): \
                   idx if (acc is None) and elem == thing else acc, list, None)

Cómo utilizar:

>>> indexOf([1,2,3], 4)
>>>
>>> indexOf([1,2,3], 1)
0
>>>

No sé por qué debería pensar que está sucio ... ¿por la excepción? si quieres un delineador, aquí está:

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

pero desaconsejaría su uso; Creo que la solución de Ross Rogers es la mejor, use un objeto para encapsular su comportamiento deseado, no intente llevar el lenguaje al límite a costa de la legibilidad.

Sugeriría:

if thing in thing_list:
  list_index = -1
else:
  list_index = thing_list.index(thing)