Найкращий спосіб обробити list.index (можливо, не існує) в python?

У мене є код, який виглядає приблизно так:

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

Ок, це спрощено, але ви розумієте. Зараз thingнасправді у списку може не бути, і в цьому випадку я хочу передати -1 як thing_index. В інших мовах це очікування index()повернення, якщо ви не зможете знайти елемент. Фактично це кидає a ValueError.

Я міг би це зробити:

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

Але це відчувається брудно, плюс я не знаю, чи ValueErrorможна було б підняти з якоїсь іншої причини. Я придумав таке рішення на основі функцій генератора, але це здається трохи складним:

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

Чи є більш чистий спосіб досягти того самого? Припустимо, що список не відсортований.

Немає нічого "брудного" у використанні випробування, крім пункту. Це пітонічний шлях. ValueErrorбуде піднято .indexлише методом, тому що це єдиний код, який у вас є!

Щоб відповісти на коментар:
У Python простіше попросити пробачення, ніж отримати філософію дозволу , добре встановлена, і жодна index проблема не призведе до подібних помилок. Не те, що я можу придумати будь-яке.

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

Один рядок. Простий. Немає винятків.

dictТипу має getфункцію , де , якщо ключ не існує в словнику, другий аргумент getцього значення , яке він повинен повернутися. Так само є setdefault, який повертає значення у випадку, dictякщо ключ існує, інакше він встановлює значення відповідно до параметру за замовчуванням, а потім повертає ваш параметр за замовчуванням.

Ви можете розширити listтип, щоб мати getindexdefaultметод.

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

Які потім можна використовувати:

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

У вашому коді немає нічого поганого ValueError. Ось ще одна однолінійка, якщо ви хочете уникати винятків:

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

Це питання є філософією мови. Наприклад, у Java завжди існувала традиція, що винятки повинні використовуватися лише у "виняткових обставинах", тобто коли трапляються помилки, а не для контролю потоку . Спочатку це було з міркувань продуктивності, оскільки винятки Java були повільними, але тепер це стало загальноприйнятим стилем.

На відміну від цього, Python завжди використовував винятки, щоб вказати на нормальний потік програми, як, наприклад, підвищення a, ValueErrorяк ми обговорюємо тут. У цьому немає нічого "брудного" в стилі Python, і є багато інших, звідки це взялося. Ще більш поширеним прикладом є StopIterationвиняток, який піднімається методом ітератора next()для сигналізації про відсутність подальших значень.

Якщо ви робите це часто, то краще прибрати його в помічну функцію:

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

Що з цим 😃:

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 

Як що до цього:

otherfunction(thing_collection, thing)

Замість того, щоб виставити щось настільки залежне від впровадження, як індекс списку у функціональному інтерфейсі, передайте колекцію та річ та дозвольте іншим функціям вирішувати питання "тесту на членство". Якщо інша функція записується як колекційно-агностична, вона, ймовірно, починається з:

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

який буде працювати, якщо thing_collection - це список, кортеж, набір чи діктант.

Це, можливо, зрозуміліше, ніж:

if thing_index != MAGIC_VALUE_INDICATING_NOT_A_MEMBER:

який код у вас вже є в іншій функції.

Як щодо цього:

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

У мене така ж проблема з методом ".index ()" у списках. Я не маю жодних проблем з тим, що він кидає виняток, але я категорично не згоден з тим, що це не описовий ValueError. Я можу зрозуміти, чи це був би IndexError, хоча

Я бачу, чому повернення "-1" теж буде проблемою, оскільки це дійсний індекс у Python. Але реально, я ніколи не очікую, що метод ".index ()" поверне від'ємне число.

Тут йде один вкладиш (добре, це досить довгий рядок ...), проходить через список точно один раз і повертає "Ні", якщо елемент не знайдено. Було б тривіально переписати це, щоб повернути -1, якщо ви цього хочете.

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

Як використовувати:

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

Я не знаю, чому ви повинні вважати, що це брудно ... через виняток? якщо ви хочете oneliner, ось це:

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

але я б радив не використовувати його; Я думаю, що рішення Росса Роджерса - найкраще, використовуйте об'єкт, щоб інкапсулювати свою поведінку, не намагайтеся підштовхувати мову до її меж ціною читабельності.

Я б запропонував:

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