Pythonでlist.index（might-not-exist）を処理する最良の方法？

次のようなコードがあります。

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句の使用については、「汚い」ことは何もありません。これはpythonicの方法です。ValueErrorが提起され.index、それはあなたがそこに持っている唯一のコードなので、唯一の方法！

コメントに答えるために：
でのPythonを、許可を取得するよりも、許しを頼むが容易哲学は十分に確立されていない、と何が index任意の他の問題のため、このタイプのエラーは発生しません。何でも思いつくわけではない。

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

1行。シンプル。例外なく。

dictタイプがありgetファンクションキーが辞書に存在しない場合、第2引数には、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)

この問題は言語哲学の1つです。たとえばJavaでは、例外は実際には「例外的な状況」、つまりフロー制御ではなくエラーが発生した場合にのみ使用されるべきであるという伝統があります。最初はJavaの例外が遅いためこれはパフォーマンス上の理由でしたが、今ではこれが受け入れられるスタイルになっています。

対照的に、Pythonは常に例外を使用して、ValueErrorここで説明しているようにaを発生させるなど、通常のプログラムフローを示します。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)

関数インターフェイスのリストインデックスのように実装に依存するものを公開するのではなく、コレクションと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つのライナー（大丈夫ですが、かなり長い行です...）を実行し、リストを1回だけ通過し、アイテムが見つからない場合は「なし」を返します。必要に応じて、-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
>>>

なぜ汚いと思うべきなのかわからない…例外のため？ワンライナーが必要な場合は、次のとおりです。

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)