วิธีที่ดีที่สุดในการจัดการ list.index (อาจไม่มีอยู่จริง) ใน python?

ฉันมีรหัสที่มีลักษณะดังนี้:

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

โอเคมันง่ายมาก แต่คุณเข้าใจแล้ว ตอนนี้thingอาจจะไม่เป็นจริงในรายการซึ่งในกรณีนี้ผมต้องการที่จะผ่าน thing_index-1 ในภาษาอื่นนี่คือสิ่งที่คุณคาดหวังว่าindex()จะได้รับคืนหากไม่พบองค์ประกอบ ในความเป็นจริงมันพ่นไฟล์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]

มีวิธีที่สะอาดกว่าในการบรรลุสิ่งเดียวกันหรือไม่? สมมติว่ารายการไม่ได้เรียงลำดับ

ไม่มีอะไร "สกปรก" เกี่ยวกับการใช้ try-except clause นี่คือวิธี pythonic ValueErrorจะเพิ่มขึ้นโดย.indexวิธีการเท่านั้นเนื่องจากเป็นรหัสเดียวที่คุณมี!

ในการตอบความคิดเห็น:
ใน Python การขอการให้อภัยนั้นง่ายกว่าการได้รับอนุญาตอย่างดีและไม่ indexจะไม่เพิ่มข้อผิดพลาดประเภทนี้สำหรับปัญหาอื่น ๆ ไม่ใช่ว่าฉันจะคิดอะไรได้

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

หนึ่งบรรทัด เรียบง่าย ไม่มีข้อยกเว้น.

dictประเภทมีgetฟังก์ชั่นที่สำคัญถ้าไม่ได้อยู่ในพจนานุกรมอาร์กิวเมนต์ที่ 2 getคือค่าที่มันควรจะกลับ ในทำนองเดียวกันจะมีการsetdefaultส่งคืนค่าในdictif ที่มีคีย์อยู่มิฉะนั้นจะตั้งค่าตามพารามิเตอร์เริ่มต้นของคุณแล้วส่งกลับพารามิเตอร์เริ่มต้นของคุณ

คุณสามารถขยาย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

แต่ฉันจะไม่แนะนำให้ใช้มัน ฉันคิดว่าโซลูชันของ Ross Rogers นั้นดีที่สุดใช้วัตถุเพื่อห่อหุ้มพฤติกรรมที่คุณต้องการอย่าพยายามผลักดันภาษาให้ถึงขีด จำกัด โดยเสียค่าใช้จ่ายในการอ่าน

ฉันขอแนะนำ:

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