リストの中から、最初に見つかった負の値を表示したい

Question

forとbreakを使って―21を表示したいのですが、どこが間違っていますか？

vals = [1, 30, 30, 0, 2, -21, 0, 12, -2]  

for number in vals:
    if number > 0:
        break
    else:
        print(number)
        

Answer 1

-21が見つかったらその値を出力してbreakということと解釈しますと、

vals = [1, 30, 30, 0, 2, -21, 0, 12, -2]  

for number in vals:
    if number < 0: #負の値が欲しいなら当然'>'ではなく'<'
        print(number) #先にbreakの条件に引っかかった数値をprint()して
        break

-21

という風にします。提示されたコードだと、一つ目のifでvals[0]=1が引っかかって何もしないでプログラムが終了します。

Answer 2

どこというか・・・
最初に見つかった負の数を出力して終了ですよね？

であれば、
numberが負の値（0未満）であればnumberを出力して終了（break）
そうでなければ、何もしない（勝手にループ先頭に戻るので）というプログラムを書く必要があります。

現状は、1以上であれば終了となっていますので、多分いきなり終わりますよね？