pythonでループを抜け出す方法

Question

現在discordのbotを作成していまして、
!startというコマンドでループする動作を
!stopというコマンドで止めたいのですが
whileを使用して、もしくは別の方法でいいやり方がないでしょうか。

async def on_message(message):
    if message.content.startswith('!start'):
        while True:    
            try:
                aaa = 1
                break
            except:
                if message.content.startswith('!stop'):
                    stop = "終わります"
                    await client.send_message(message.channel, stop)
                    break
                else: 
                    time.sleep(30)

Comments

[y_waiwai]

このコード、ちゃんと動くんでしょうか

[soniroji]

いえ、これ自体は動かないです。実際はもっと長いのですが、tryの部分に色々と個人情報が入っていたため省略しました。

Answer 1

Threadを使えばよいのではないでしょうか

このサンプルは２回目の "!start" メッセージは考慮しておりません

import threading
import time

def on_running():
    while not stop_event.is_set():
        # なんかの処理
        time.sleep(1)

stop_event = threading.Event()
running_thread = threading.Thread(target = on_running)

async def on_message(message):
    if message.content.startswith('!start'):
        # Thread開始
        running_thread.start()
    elif message.content.startswith('!stop'):
        # Thread停止
        stop_event.set()
        running_thread.join()

Comments

[soniroji]

ありがとうございます！
調べてみると思ていたことができそうです。
ちなみにこれに対して
if message.content.startswith('!start'):
これが返信になるのですが
await client.send_message(message.channel, stop)
これはwhileの中に入れたままでいいのでしょうか？

[magichan]

スミマセン。一部コードにミスがありましたので修正しました。

上記のコメントに関しては、仕様がよくわからないのですが、 on_message() に "!stop" メッセージを受信して処理が終了した際に、（誰かに）メッセージを送りたいということでしょうか？
であればループの中ではなくて
running_thread.join() # ←スレッドが終了するまで待つ処理
の次の行でも良い気がします

[soniroji]

ありがとうございます。
一応やりたいことはbeautifulsoupを使用してブログを更新したらdiscordに流す(メッセージを送る)というようなことです・・。
初心者なので、常に調べながらなんですが例えばwhileのtryで変数への代入を行ったとします。
（例えばそのブログのタイトル）
それをon_message(message):以下で処理するメッセージで送りたい場合はどういった記述をしたらいいでしょうか。
そのまま処理しようとすると変数にUndefined variableエラーが出てしまいます。

[magichan]

遅くなりました。

確認ですが、
・client.send_message() は discord に対してメッセージを送信する為のAPI
・"!stop" がきた時に上記のAPIにてメッセージ送信をしたい
ということでよろしいですかね。
であれば、 "!stop" を受信して "stop_event.set()" を行うことにより、on_running() 関数のループ処理を抜けるので、ループを抜けた後の処理で行えばよいのではないでしょうか。