コーディングの変数に関して

Question

コーディングの際の変数に関して、後々のことを考えてきれいなコーディングをしたいと思っています。

現在、seleniumを使ってスクレイピングをしようとしています。
その際、変数が無駄に増えてしまい、収拾がつかなくなってしまいます。
現状、以下のようにelement elemtnt1 ...　と無限に増えていっています。

from selenium import webdriver

driver = webdriver.Chrome(chromedriver)
chromeOptions = webdriver.ChromeOptions()

driver = webdriver.Chrome(chromedriver, chrome_options=chromeOptions)

driver.get("https://www.aaa.co.jp/")

element = driver.find_element_by_id("username")
element.send_keys("aaaa")
 
element2 = driver.find_element_by_link_text("次へ")
element2.click()

・・・

これは例えば以下のように変えたほうがいいのでしょうか。

from selenium import webdriver

driver = webdriver.Chrome(chromedriver)
chromeOptions = webdriver.ChromeOptions()

driver = webdriver.Chrome(chromedriver, chrome_options=chromeOptions)

driver.get("https://www.aaa.co.jp/")


driver.find_element_by_id("username").send_keys("aaaa")

driver.find_element_by_link_text("次へ").click()

しかしこうすると逆にコードがわかりづらくなってしまうような気がします。

こういうものなのでしょうか？
完全に独学なので、ほかの人に聞くこともできないため、質問しました。
どうぞよろしくお願いします。

Answer 1

下の方が綺麗じゃないでしょうか。メソッドチェーンと呼ばれるような技法です。

あとはelementとelement2を分ける理由が特にない場合（つまりあとからelementにアクセスしてどうこう……とは想定しない場合）は、elementに再代入する手もあります。一時的な変数であることは頭の中に入れておかないと駄目ですが……（こういうときはelemなど少し適当な変数名にする傾向があるように思います）

elem = driver.find_element_by_id("username")
elem.send_keys("aaaa")

elem = driver.find_element_by_link_text("次へ")
elem.click()

Answer 2

第一歩としては「適切な名前をつける」ところからじゃないでしょうか。
element1, element2...だと、後から見てその中身が何なのか想像がつきにくくなります。

例えばusername_elementとかlink_to_next_elementとか。（長いですが）

後は、その変数の生存期間が短い（コード上使われている場所が狭い）のであれば、その変数を使って行っている「処理」の単位で関数（メソッド）化していくとか。

Comments

[hayataka2049]

細かいですが、pythonは公式でコーディング規約を出しており（PEP8）、変数名はスネークケース推奨です。その観点からするとusernameElementとかlinkToNextElementよりはusername_elementとかlink_to_next_elementの方が推奨されるでしょう。
常に従うべきというものでもありませんが、参考までに。

[gentaro]

あ、なるほど。ごもっともです。

[gentaro]

回答修正しました。ご指摘ありがとうございます。（たぶん書いてるうちに頭がPythonじゃなくJavaScriptになってました）