try文とその他の処理について

Question

python（selenium）のtry文で『この要素がなかったら17時にシャットダウンをする、要素があればクリックをしてクリックをすると新しいタブが開くから新しいタブに切り替えて17時にシャットダウンをする』というコードを書きたいのですが上手くいきません
どこが間違っておりますでしょうか？
何卒ご教授よろしくお願いいたします

try:
    driver.find_element_by_xpath('//*[@id="root"]/div/div/div[5]/div[2]/div[4]/div[5]/div/div[1]/table/tbody/tr[2]/td[2]/div/div[1]').click() 
except NoSuchElementException:
    def shutdown():
        os.system('shutdown -s')
    
        schedule.every().monday.at("17:00").do(shutdown)
        schedule.every().tuesday.at("17:00").do(shutdown) 
        schedule.every().wednesday.at("17:00").do(shutdown) 
        schedule.every().thursday.at("17:00").do(shutdown) 
        schedule.every().friday.at("17:00").do(shutdown) 

        while True:
            schedule.run_pending()
            time.sleep(60)
            break
else:
    driver.switch_to.window(driver.window_handles[1]) 


    def shutdown():
        os.system('shutdown -s')

        schedule.every().monday.at("17:00").do(shutdown)
        schedule.every().tuesday.at("17:00").do(shutdown) 
        schedule.every().wednesday.at("17:00").do(shutdown) 
        schedule.every().thursday.at("17:00").do(shutdown) 
        schedule.every().friday.at("17:00").do(shutdown) 

        while True:
            schedule.run_pending()
            time.sleep(60)
            break                
     

Answer 1

try``exceptは他にに方法がない時に使います。

この要素がなかったら～～、要素があれば～～

であれば、普通は、

elements = driver.find_elements_by_xpath("～～～")
if len(elements)>0:
    elements[0].click()
    driver.switch_to.window(driver.window_handles[1]) 

シャットダウンの処理

シャットダウンの部分はインデントがおかしいので、インデントだけ直すと、

def shutdown():
    os.system('shutdown -s')

schedule.every().monday.at("17:00").do(shutdown)
schedule.every().tuesday.at("17:00").do(shutdown) 
schedule.every().wednesday.at("17:00").do(shutdown) 
schedule.every().thursday.at("17:00").do(shutdown) 
schedule.every().friday.at("17:00").do(shutdown) 

while True:
    schedule.run_pending()
    time.sleep(60)

でしょうが、シャットダウンしてしまえばそれ以降の処理は動かないので、everyにしてwhileでループする意味ないのでは？また、なぜ曜日指定？

Comments

[help-man]

やはりwhile文は必要ないですよね？
ネットに転がってたのを適当にコピペしてみてみただけなので「これ必要かな？」と私も思っていました
曜日指定にしたので平日のみ使いたかったからです
これもネットで『python pc シャットダウン 方法』で調べてみたら「scheduleを使うといい」と書いてあったのでscheduleを使いました

[otn]

意味をわからずにコードを書いていると、エラーになったり、意図通り動かないのは当然です。
意味をわかった上でコードを書きましょう。

「平日であれば17にシャットダウンし、平日でなければ何もしない」ということであれば、その通りにストレートに書いた方がいいです。

[help-man]

分かりました
毎回勉強不足で申し訳ございません

[otn]

謝る話ではないでしょう。

[help-man]

ご指示のとおりにやってみたらできました
ありがとうございます！
ただシャットダウンに関しては
『while True:
schedule.run_pending()
time.sleep(60)』
がないとシャットダウンはできないようです
なぜかは分かりませんが

[help-man]

ターミナルの最後に『Passthrough is not supported, GL is disabled, ANGLE is 』と表示されますが関係ありませんよね？

[otn]

失礼しました。run_pending()は該当がないとすぐ終了するようなので、やはりwhileは必要でした。訂正します。

[help-man]

そうなんですか
訂正ありがとうございます

Answer 2

とりあえず、defの中に その関数の呼び出しが入ってしまっているので、except側に入ってもelse側に入っても何かが動作するようにはなっていません。

schedule.every 以下のインデントは間違えているでしょう。

Comments

[help-man]

勉強不足で大変恐縮なのですがschedule.every 以下のインデントとはwhileのところでしょうか？