一連の流れの途中での例外チェックについて

前提・実現したいこと

Seleniumを使用してGoogle Chromeを操作するプログラムを作成しています。

プログラムの内容は用意してある複数のURLを指定しそのページに移動し、ボタンを押すというものです。

処理を順番にしていく際に画面に表示されたボタンがクリックできない場合があります。
その時だけ別の処理を行いプログラムを続行したいのですが、現状ではボタンのクリック毎にTry, Exceptの行を追加しないといけません。

対象のボタンがすべて同じ場合はループさせれば済むのですが、printf()で文章を表示してみたり異なるボタンを押したいためその手段が取れません。エラーの内容はページ毎に同じものなので、プログラムの行数を短くするために便利な書き方などありましたら教えていただけると嬉しいです。

Python
1
2# ボタンA,Bを押す、クリックできない場合は別のボタン(btn_Error)を押す
3Try:
4   driver.get(url_1)
5   btn = driver.find_element_by_class_name("btn_A").click()
6   btn = driver.find_element_by_class_name("btn_B").click()
7   print("ボタンを押しました")
8
9Except selenium.common.exceptions.ElementClickInterceptedException:
10   btn = driver.find_element_by_class_name("btn_Error").click()
11
12
13# ボタン(btn_A)を押す、クリックできない場合は別のボタン(btn_Error)を押す
14Try:
15   driver.get(url_2)
16   btn = driver.find_element_by_class_name("btn_A").click()
17
18Except selenium.common.exceptions.ElementClickInterceptedException:
19   btn = driver.find_element_by_class_name("btn_Error").click()
20
21
22# ボタンC,A,Bを押す、クリックできない場合は別のボタン(btn_Error)を押す
23Try:
24   driver.get(url_2)
25   btn = driver.find_element_by_class_name("btn_C").click()
26   btn = driver.find_element_by_class_name("btn_A").click()
27   btn = driver.find_element_by_class_name("btn_B").click()
28   printf("ボタン")
29
30Except selenium.common.exceptions.ElementClickInterceptedException:
31   btn = driver.find_element_by_class_name("btn_Error").click()
32
33・
34・
35・
36・
37

udon-ken

2021/08/23 16:49

全体に質問の意味と用語の使いか方がよくわかりません。なので、その保守性の悪いプログラムを提示しないと回答が付かないと思います。想像で回答して、「質問の意図と全然違っていた」となると完全に無駄になるからです。

toomato

2021/08/23 17:10

失礼しました。再度書き直そうと思います。文章を書くのは苦手なので編集しても分かりずらいと思いますので無視していただいて構いません。ご指摘ありがとうございました。

行動規範の内容に同意します

回答1件

ベストアンサー

ボタン操作の関数を作って行うのでは駄目でしょうか？

python
1def btn_click(driver, url, btns, cleanup = None):
2  try:
3    driver.get(url)
4    for btn in btns:
5       driver.find_element_by_class_name(btn).click()
6    if callable(cleanup):
7       cleanup()
8  except selenium.common.exceptions.ElementClickInterceptedException:
9    btn = driver.find_element_by_class_name("btn_Error").click()
10
11btn_click(url_1, ["btn_A", "btn_B"], lambda: print("ボタンを押しました"))
12btn_click(url_2, ["btn_A"])
13btn_click(url_3, ["btn_C", "btn_A", "btn_B"], lambda: print("ボタン"))

動作確認していませんが、こんな感じでしょうか？

投稿2021/08/26 06:08

TANAKAKazuyoshi

総合スコア96

toomato

2021/08/26 06:20

アドバイスありがとうございます。確かに関数に入れると余計に何度も書かなくてすっきりして使いやすいと感じました。自分のプログラムもボタン操作を関数にまとめたいと思います。一点気になることがありまして、callableに引数として渡すcleanupという関数(?)変数(?)はどのような意味があるのでしょうか？callable については調べて実行可能かを判定すると分かったのですが、cleanupの役割について補足いただけないでしょうか。