リストの中で前後より小さくなる値を取得したいです。

Question

前提・実現したいこと

list = [2,3,5,7,4,8,8,6,9,10]
このようなリストを順に見ていき、最初に前後の数字より小さくなる値を取得したいです。
このリストでは前後の数字より小さくなる4と6のうち、先に出てくる4にあたる数字です。

コードでは、「1つ前の数字より小さい」「1つ後の数字より小さい」をandで処理しようとしたのですが値もエラーも返ってきません。同じ向きの不等号はandで繋げられないのでしょうか？
どなたか解決方法を教えてください。

該当のソースコード

list = [2,3,5,7,4,8,8,6,9,10]
for i in list:
    if (list[i] <list[i+1]) and (list[i] < list[i-1]) :
        print(list[i])
    break

Answer 1

data = [2, 3, 5, 7, 4, 8, 8, 6, 9, 10]

work = [data[i] for i in range(1, len(data) - 1) 
        if data[i -1] > data[i]  and data[i] < data[i + 1]]

if len(work) > 0:
    print(work[0])
else:
    print("None")

Answer 2

原因については既に回答が付いているとおりです。

---

zipを使うと範囲外アクセスを考慮しなくて済むので楽です。
あとfor-elseを使うと見つからなかった際の挙動もすっきり書けますね。

src = [2, 3, 5, 7, 4, 8, 8, 6, 9, 10]

for a, b, c in zip(src, src[1:], src[2:]):
    if b < a and b < c:
        break
else:
    b = None

print(b)   # => 4

Comments

[mouccy]

見つからなかった際を考えてませんでした！ありがとうございます。
zipを使うとすっきり書くことができるんですね。勉強になります。

Answer 3

参考に内包表記を使用したコードを提示します。

lst = [2,3,5,7,4,8,8,6,9,10]

mid = [n for i, n in enumerate(lst)
       if 0 < i < len(lst)-1 and lst[i-1] > n < lst[i+1]][0]

print(mid)  # 4

Comments

[mouccy]

このような書き方も可能なんですね。勉強になります！

Answer 4

他の方の回答の通り、リストの添え字と、リストの要素との混同があります。
breakの位置もおかしいです。

先頭と末尾は対象外のようなので、2番目から最後から2番目まで繰り返します。

foo = [2,3,5,7,4,8,8,6,9,10]
for i in range(1,len(foo)-1):
    if foo[i] < foo[i+1] and foo[i] < foo[i-1]:
        print(foo[i])
        break

Comments

[mouccy]

さまざまな基本的なことができてませんでした、、、！ありがとうございます。

Answer 5

and が動いていないのではなく，問題は別のところにあります．

処理が止まることについてはbreakしてるからですね．
break外せばIndexError出しつつ出力が出ます．

それとは別に， i in list と要素を取得してますが，その i をリストのインデックスに使っているので先頭から探索するという意図通りには動きません．

先頭から探索したいならば range などで順にインデックスを作成する必要があります．

Comments

[mouccy]

break外したらエラーメッセージ出ました、、、
そのままでは先頭からみていくことができないんですね。勉強になります！

Answer 6

・「list」はpythonの組み込み型名なので変数名に使わない
・for i in lstとすると、lstの要素が一つずつ順番に出てきます。つまり、i = 2,3,5,7...となるのでこれをlst[i]とかインデックスに使うとややこしいことになります。
というあたりを修正しました。

lst = [2,3,5,7,4,8,8,6,9,10]
for i in range(1,len(lst)-1):
    if (lst[i] < lst[i+1]) and (lst[i] < lst[i-1]) :
        print(lst[i])  # -> 4
        break

あと、最初の「2」は2つ目の「3」とその一つ前＝最後の「10」と比較されてますが、その挙動で合っているんでしょうか。
4を出したいというのを忘れてました。

Comments

[mouccy]

listは変数名に使用するのは避けた方がいいのですね！
勉強になりました、ありがとうございます。