Cara terbaik untuk menangani list.index (mungkin-tidak-ada) dengan python?

Saya memiliki kode yang terlihat seperti ini:

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

ok jadi itu disederhanakan tetapi Anda mendapatkan idenya. Sekarang thingmungkin tidak benar-benar ada dalam daftar, dalam hal ini saya ingin meneruskan -1 sebagai thing_index. Dalam bahasa lain, inilah yang Anda harapkan index()untuk dikembalikan jika tidak dapat menemukan elemennya. Sebenarnya itu melempar ValueError.

Saya bisa melakukan ini:

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

Tapi ini terasa kotor, ditambah lagi saya tidak tahu apakah ValueErrorbisa dibesarkan karena alasan lain. Saya datang dengan solusi berikut berdasarkan fungsi generator, tetapi tampaknya sedikit rumit:

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

Adakah cara yang lebih bersih untuk mencapai hal yang sama? Mari kita asumsikan bahwa daftar tersebut tidak diurutkan.

Tidak ada yang "kotor" tentang penggunaan klausa coba-kecuali. Ini adalah cara pythonic. ValueErrorakan dimunculkan oleh .indexmetode saja, karena itu satu-satunya kode yang Anda miliki di sana!

Untuk menjawab komentar:
Dengan Python, filosofi lebih mudah untuk meminta maaf daripada mendapatkan izin sudah mapan, dan tidak tidak index akan meningkatkan jenis kesalahan ini untuk masalah lainnya. Bukannya aku bisa memikirkan apapun.

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

Satu baris. Sederhana. Tidak ada pengecualian.

The dictjenis memiliki getfungsi , di mana jika kunci tidak ada dalam kamus, argumen 2 untuk getadalah nilai yang harus kembali. Demikian pula ada setdefault, yang mengembalikan nilai dalam dictjika kunci ada, jika tidak, menetapkan nilai sesuai dengan parameter default Anda dan kemudian mengembalikan parameter default Anda.

Anda bisa memperluas listtipe untuk memiliki getindexdefaultmetode.

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

Yang kemudian bisa digunakan seperti:

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

Tidak ada yang salah dengan kode yang Anda gunakan ValueError. Berikut satu baris lain jika Anda ingin menghindari pengecualian:

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

Masalah ini adalah salah satu filosofi bahasa. Di Jawa misalnya, selalu ada tradisi bahwa pengecualian seharusnya hanya digunakan dalam "keadaan luar biasa" yaitu ketika kesalahan terjadi, bukan untuk kontrol aliran. . Pada awalnya ini untuk alasan kinerja karena pengecualian Java lambat tetapi sekarang ini telah menjadi gaya yang diterima.

Sebaliknya Python selalu menggunakan pengecualian untuk menunjukkan aliran program normal, seperti menaikkan a ValueErrorseperti yang kita bahas di sini. Tidak ada yang "kotor" tentang ini dalam gaya Python dan masih banyak lagi dari mana asalnya. Contoh yang lebih umum adalah StopIterationpengecualian yang dimunculkan oleh metode iterator untuk next()memberi sinyal bahwa tidak ada nilai lebih lanjut.

Jika Anda sering melakukan ini, maka lebih baik untuk membakarnya di dalam fungsi pembantu:

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

Bagaimana dengan ini 😃:

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 

Bagaimana dengan ini:

otherfunction(thing_collection, thing)

Daripada mengekspos sesuatu yang sangat tergantung pada implementasi seperti indeks daftar di antarmuka fungsi, teruskan koleksi dan hal tersebut dan biarkan fungsi lain menangani masalah "uji keanggotaan". Jika fungsi lain ditulis menjadi collection-type-agnostic, maka mungkin akan dimulai dengan:

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

yang akan berfungsi jika thing_collection adalah list, tuple, set, atau dict.

Ini mungkin lebih jelas dari:

if thing_index != MAGIC_VALUE_INDICATING_NOT_A_MEMBER:

yang merupakan kode yang sudah Anda miliki di fungsi lain.

Bagaimana dengan seperti ini:

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

Saya memiliki masalah yang sama dengan metode ".index ()" di daftar. Saya tidak memiliki masalah dengan fakta bahwa ini melontarkan pengecualian tetapi saya sangat tidak setuju dengan fakta bahwa ini adalah ValueError yang tidak deskriptif. Aku bisa mengerti jika itu adalah IndexError.

Saya dapat melihat mengapa mengembalikan "-1" akan menjadi masalah juga karena ini adalah indeks yang valid dalam Python. Tapi secara realistis, saya tidak pernah mengharapkan metode ".index ()" untuk mengembalikan angka negatif.

Ini dia satu baris (ok, ini adalah baris yang agak panjang ...), melewati daftar tepat sekali dan mengembalikan "Tidak ada" jika item tidak ditemukan. Akan mudah untuk menulis ulang untuk mengembalikan -1, jika Anda menginginkannya.

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

Cara Penggunaan:

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

Saya tidak tahu mengapa Anda harus berpikir itu kotor ... karena pengecualian? jika Anda ingin oneliner, ini dia:

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

tetapi saya akan menyarankan agar tidak menggunakannya; Menurut saya solusi Ross Rogers adalah yang terbaik, gunakan objek untuk merangkum perilaku Anda yang diinginkan, jangan mencoba memaksakan bahasa hingga batasnya dengan mengorbankan keterbacaan.

Saya sarankan:

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