discordbotで入力を複数回受け取るには

Question

from asyncio.tasks import wait_for
from datetime import MAXYEAR
import datetime
from time import sleep
import discord
from discord import user
from discord import message
from dotenv import load_dotenv
import os
import time
import main

#env読み込み
load_dotenv()
TOKEN = os.getenv('token')

# 接続に必要なオブジェクトを生成
client = discord.Client()

@client.event
async def on_ready():
    print('ログインしました')



@client.event
async def on_message(message):
    if message.author.bot:
        return
    
    if message.content.startswith('!join') or message.content.startswith('!j'):
        await message.channel.send(f'Hello {message.author.mention}. Please choose timer(!timer or !t) or stopwatch(!stopwatch or !sw)')

    
        def select(s):
            global slt
            if s.content == '!timer' or s.content == '!t':
                slt = 'timer'
                return slt
            elif s.content == '!stopwatch' or s.content == '!sw':
                slt = 'stopwatch'
                return slt
            return 
        
        msg = await client.wait_for('message', check=select)
        await message.channel.send(f'{msg.author.mention} selects a {slt}.')


        #timer
        if slt == 'timer':
            await message.channel.send('Please set the time : 10m(!10m) 20m(!20m) 30m(!30m)') 
            global settime

            def set_time(st):
                if st.content == '!10m':
                    global settime
                    settime = 600
                    return settime
                elif st.content == '!20m':
                    settime = 1200
                    return settime
                elif st.content == '!30m':
                    settime = 1800
                    return settime
                elif st.content == '!3m': #test 3s
                    settime = 3
                    return settime
                return

            s_msg = await client.wait_for('message', check=set_time)    
            await message.channel.send(f'{s_msg.author.mention} sets {settime}m.\n Start with !start ot !s')

            
           
            if message.content == '!start' or message.content == '!s':
                await message.channel.send(f'{s_msg.author.mention} Start timer')

                for i in range(settime):
                    await message.channel.send(f'{s_msg.author.mention} {settime}')
                    sleep(settime)
                    settime -= 1
                    if settime == 0:
                        await message.channel.send(f'{s_msg.author.mention} Thank you for your hard work.')
                        break
 


            

discordbotでタイマーを作ろうとしています。
if message.content == '!s'のところで!sを入力してもなにも帰ってきません。どのようにしたらタイマーが実装されますか。
ご教授ください

【追記】
!jでbotを読み込みます。その後、タイマー(!t)かストップウォッチ(!sw)の入力を受け取り時間を選択後(/10m)、!sで開始という流れを想定しています。
入力を受け取るとは、discord上で入力したもの(!tや!10mなど)のことで、それをもとに条件分岐を行おうと考えていました。
この場合、複数回入力を受け取ることが不可能ということでしょうか。

Answer 1

上の方の
if message.content.startswith('!join') or message.content.startswith('!j'):
でTrueの場合のみそこから下を実行するコードなので、message.contentが!sならそこはFalseになるから何もされないのでは？

Comments

[kazulztk]

おっしゃる通りです。見落としていました。
では、入力を受け取ってからタイマーを開始するにはどのようにしたらいいですか。

[udon-ken]

ちょっと意味が分からないです。

> discordbotでタイマーを作ろうとしています。

これだけでは実際のところどういう仕様のものを作ろうとしているのかさっぱりわかりません。
「discordでタイマーと言えばおおよそこんなもの」みたいな共通認識があるようなものでしたらこれだけでもおおまかに推測がつくのでしょうけど・・・
例でいうと、「リアクションロール」と言えばボット制作界隈ではほぼどういうものか共通認識があると思いますのである程度通じますが「タイマー」では無理です。

> 入力を受け取ってからタイマーを開始するにはどのようにしたらいいですか。

仕様がわかりませんので、そもそも「入力を受け取って」の意味すら分かりません。

[kazulztk]

!jでbotを読み込みます。その後、タイマー(!t)かストップウォッチ(!sw)の入力を受け取り時間を選択後(/10m)、!sで開始という流れを想定しています。
入力を受け取るとは、discord上で入力したもの(!tや!10mなど)のことで、それをもとに条件分岐を行おうと考えていました。
この場合、複数回入力を受け取ることが不可能ということでしょうか。

[udon-ken]

うーんと・・・多分ですが、初歩的な部分ができていないのにあれもこれもと欲張りすぎてる感じがします。

あと、質問するなら必要最低限のコードかつ他人が見た時に分かりやすいコードにしないと、読むほうは訳がわかりません。
大量のimportとかログインメッセージとか、時間の入力が択一式（？）であろうが自由入力であろうが動作に関係ないのだから質問段階では自由入力でよくないですかね？

まあ、それはいいとして、なぜ初歩的な部分が～となるかというと「!jでbotを読み込みます。」この時点で既に意味不明だからです。
読み込みが起動という意味なら既に起動していますし、もし起動していないなら!jのメッセージが受け取れません。
その辺、ボット作成者ならユーザーにどう説明するかは悩んでも仕様について表す場合にそういう表現は起こりえません。

で、一応、なんとなく読み取りましたが、なんとなく仕様がおかしい。
以下のような仕様が普通に思いつくものです。

１．!j でボットが反応し「ボットが起動した。ついてはタイマーかストップウォッチか指定しろ」とのメッセージを送信する。
※※ ここの「起動した」はユーザーに理解しやすいよう伝えるだけの方便で、実際は全くの嘘であり最初からボットは起動している ※※
２．ユーザーがタイマーかストップウォッチか指定したなら、「時間を送信しろ」とのメッセージを送信する。
３．ユーザーが時間を入力したなら、「スタートしたい時に!sと送信しろ」とのメッセージを送信する。
４．ユーザーから!sの送信があったならタイマーの処理を開始する。

これの１と２、２と３、３と４の間にwait_forで入力を待つというのが常道と思います。
条件分岐の件もよくわかりませんが、普通に可能と思います。
wait_forで受け取った値を元に分岐するだけでは？

いずれにしても、あれやこれや最初から盛り込もうとせず、シンプルなものを作って（ただし、拡張する事は考慮しながら）それを拡張いくほうがいいと思います。
自分なら
user:!timer
bot:何秒？（好きな数字を入れろ）
user:30
bot:開始しました
から始めて、次はuserが秒数を答えたらboが「開始する時はsを入力しろ」と返答して任意タイミングにスタートできるようにして…
のようにして拡張していきます。

あと、仕様を考える時、使うのは「ユーザーである」というのを強く意識ましょう。
また、今回はどうか分かりませんが、大半が日本人であるというのも重要です。
・何回も何回も入力させる（プログラム的に必要かもしれないが、ユーザーは最小限を望む）
・意味の分からないコマンド（また、文字種が入り混じった入力の強制）
・意味の分からないメッセージ
が最低限になるよう考える必要があります。

今回でいうと、!j(join）とか・・・
ネイティブなら理解するのかも知れませんが、日本人ならjoinってなに？または、jointって参加だよね？となり、とても動作開始のコマンドと思わない（＝覚えられない）と思います。
途中で!を含む文字列を送信させる事についても、まずはめんどくさいです。「何秒？」って問われているのなんで!なんか付けないといけない？めんどくさい。となります。
そして、一般的には!で始めるのはコマンドですので、確実に混乱の元です。

[kazulztk]

最初から難解なことをしようとしていたかもしれません。シンプルなものから作ってみたいと思います。
ありがとうございます。