while文を使って変数に変数を代入するには

Question

python初心者で、様々なサイトをかじりながらコードを書いています。
あるコードを書いている際に疑問に思い、実装できなかった＆検索しても解決しなかったので質問させていただきます。

前提・実現したいこと

あるサイトでseleniumを用いて「ラジオボタンをクリック→OKボタンをクリック」を複数回行いたいと考えています。

その際、ラジオボタン/okボタンのソースコードが

<input id="ohayo_1" type="radio" value="1">
<input type="submit" name="hogehoge_1" value="OK" id="hogehog_1">

<input id="ohayo_2" type="radio" value="1">
<input type="submit" name="hogehoge_2" value="OK" id="hogehog_2">

<input id="ohayo_3" type="radio" value="1">
<input type="submit" name="hogehoge_3" value="OK" id="hogehog_3">

のようになっているので、seleniumのfind_elementを用いて「ラジオボタンをクリック→予約をクリック」を要素がある分だけループさせたいです。

ソースコード

現在このようなコードを書いています。

number = 0
radio_id = f'ohayo_{number}'
OK_id = f'hogehog_{number}'

while len(driver.find_element_by_id(radio_id)) == 0:
    driver.find_element_by_id(radio_id).click()
    driver.find_element_by_id(OK_id).click
    number = number + 1

このhtmlではラジオボタン/OKボタンが3つしかないですが、本番環境ではそれらの個数が可変のため（ラジオボタンとOKボタンは同数）、find_elementsとlenで要素の存在を判定し、無かった際には終了するというようにしたいと考えています。

発生している問題・エラーメッセージ

しかし、上記のコードだと思ったように動作しません。

while処理の後にradio_idやOK_idのnumber部分が変更されているのかどうか、チェックするためにprint関数で出力すると、

ohayo_0
ohayo_0
ohayo_0
ohayo_0
ohayo_0

のようになっておりました。

なので、ページに要素が見つからないため強制終了しているということなのか？と考えています。

どなたか解決できる方がいらっしゃったら、教えていただきたいです。よろしくお願いします。

環境

Windows10
Python3.9

Comments

[meg_]

> あるサイトでseleniumを用いて「ラジオボタンをクリック→OKボタンをクリック」を複数回行いたいと考えています。
念のため確認ですが対象のサイトはスクレイピング禁止ではないサイトでしょうか？

[snsh]

スクレイピング禁止ではなかったはずです。

Answer 1

while処理の後にradio_idやOK_idのnumber部分が変更されているのかどうか、チェックするためにprint関数で出力すると、

ohayo_0
ohayo_0
ohayo_0
ohayo_0
ohayo_0
のようになっておりました。

なので、ページに要素が見つからないため強制終了しているということなのか？と考えています。

ページに要素が見つからないため強制終了しているというよりも、
結果の通り全部 ohayo_0 になっていたのですから、
radio_idやOK_idのnumber部分が変更されていない ということが考えられます。

---

最初に

radio_id = f'ohayo_{number}'

と定義して、その後 while文の中で

while ～:
   （略）
    number = number + 1

としても、radio_id や OK_id の内容は変更されません。

下記のように、whileの中で radio_id と OK_id を更新する必要があります。

number = 0
radio_id = f'ohayo_{number}'
OK_id = f'hogehog_{number}'

while ～:
（略）
    number = number + 1
    # 更新   
+   radio_id = f'ohayo_{number}'
+   OK_id = f'hogehog_{number}'

さらに、find_element_by_id はidが見つからない場合に例外を送出するため、
例外処理を挟む必要があります。

# IDが1始まりの連番であると仮定
number = 1
radio_id = f'ohayo_{number}'
OK_id = f'hogehog_{number}'

while True:
    try:
        driver.find_element_by_id(radio_id).click()
        driver.find_element_by_id(OK_id).click()
        
        number = number + 1
        radio_id = f'ohayo_{number}'
        OK_id = f'hogehog_{number}'

    except selenium.common.exceptions.NoSuchElementException:
        # 見つからない場合はループを抜ける。
        break

Comments

[snsh]

whileループの中で変数の記述がないために更新されない、ということを知らなかったため、今回大変勉強になりました！
また、find elementの例外処理も少し疑問な部分があったので、今回そちらも解決していただいたqnoirさんをベストアンサーにさせていただきました。ほかの方も質問にご回答いただき、大変ありがとうございました！

Answer 2

連番で変数を作る、なんていう手段は、ほとんどの場合に好ましくありません。

リスト（配列）を使いましょう。

Comments

[退会済みユーザー]

このケースでは、変数を使って**探索対象である、連番となっている数字を含むid(文字列)を作る**というアプローチ自体は特に間違いではないと思いますがいかがでしょうか。
たとえば、idの終端がページによって不定であるような場合、あらかじめリスト（配列）にすることはできません。
質問文に「本番環境ではそれらの個数が可変のため」と記載されています。

[maisumakun]

> たとえば、idの終端がページによって不定であるような場合、あらかじめリスト（配列）にすることはできません。

いえ、Pythonのリストはあとから動的に要素を追加できます。

Answer 3

while処理の後にradio_idやOK_idのnumber部分が変更されているのかどうか、

whileループの中で更新する処理が書かれていないので、更新されるわけがないです。
ループの中で更新（代入）しましょう。

number = 0
radio_id = f'ohayo_{number}'
OK_id = f'hogehog_{number}'

while len(driver.find_element_by_id(radio_id)) == 0:
    driver.find_element_by_id(radio_id).click()
    driver.find_element_by_id(OK_id).click
    number = number + 1
    radio_id = f'ohayo_{number}'
    OK_id = f'hogehog_{number}'

Comments

[snsh]

whileループの中で変数の記述がないので更新がされない、ということは知りませんでした...！
大変勉強になりました！ありがとうございます！

[otn]

「更新しないと更新されない」「代入しないと代入されない」と言うことを知らなかったと言うことですか？
書いてない処理は実行されるはずがないのですが。

[snsh]

その通りです。

正直に申し上げますと、pythonの基礎を全く学んでおらずwhile文もいまいち理解していない状態でした。
そのような状況で質問してしまい大変申し訳なかったです。

[otn]

whileは直接は関係ないです。
whileの理解が足りないのでなく、「プログラムを実行するとは何か」「代入とは何か」が理解できてないと思われます。
a=1
b=a
a=2
で、bも2になると思っていたと言うことですよね？

[snsh]

はい、そうです。

一度初めから学び直してみたいと思います。ありがとうございました。

Answer 4

radio_idが最初に"ohayo_0"という値の文字列で初期化されてしまっているのが原因だと思います．f-stringで定義した値はただの文字列であって，その後numberの値を変えた時にradio_idの値を連動して変えてくれるような機能はありません．

radio_idの値をwhile文の中でnumberの値が変わるたびに都度変えていけばうまくいくと思います．例えば，以下のような感じです．

number = 0
while len(driver.find_element_by_id(radio_id)) == 0:
    radio_id = f'ohayo_{number}'
    OK_id = f'hogehog_{number}'
    driver.find_element_by_id(radio_id).click()
    driver.find_element_by_id(OK_id).click
    number = number + 1

Comments

[snsh]

whileループの中で変数の記述がないために更新がされないことを存じ上げていなかったので、今回大変勉強になりました！ありがとうございます！