Discord.pyのwait_forを実行しても望んだ結果が得られません

前提・発生している問題

Discord.pyでDiscordのbot作成を試みています。
wait_forを使いユーザーのリアクションを待って反応を返したいのですが、
リアクションを返してもタイムアウトとなってしまい望んだ結果が得られません。
お力を貸していただけないでしょうか。

該当のソースコード

python
1import discord
2import asyncio
3from discord.ext import commands
4
5emojis = ["{}\N{COMBINING ENCLOSING KEYCAP}".format(num) for num in range(1, 10)]
6
7@bot.event
8async def on_message(message):
9    if message.content.startswith('$thumb'):
10        channel = message.channel
11        await channel.send('Send me that' + emojis[0] + 'reaction, mate')
12
13        def check_emj(reaction, user):
14            emj = str(reaction.emoji)
15            return (user == message.author) and (emj == emojis[0]) or (emj == emojis[1])
16
17        try:
18            reaction, user = await bot.wait_for('reaction_add', timeout=60.0, check=check_emj)
19        except asyncio.TimeoutError:
20            await channel.send(emojis[2])
21        else:
22            if str(reaction.emoji) == emojis[0]:
23                await channel.send(emojis[5])
24            if str(reaction.emoji) == emojis[1]:
25                await channel.send(emojis[4])

試したこと

def check_emjのreturn以下に問題があるかと思い、括弧で括ってみたりしたのですがダメでした。

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

回答1件

ベストアンサー

これでどうでしょうか？
わざわざreturnに絵文字条件を含めなくてもelse内で分岐すればいいと思われます。
下に例を出しておきますのでご確認ください

python
1def check(reaction,user):
2    return user == message.author
3try:
4    reaction, user = await bot.wait_for('reaction_add', timeout=60.0, check=check)
5except asyncio.TimeoutError:
6    await channel.send('タイムアウト')
7else:
8    if str(reaction.emoji) == '1⃣': #どの絵文字を使いたいのか分かってません、おそらくこれ？
9        await channel.send('1の絵文字が押されたよ'):
10
11

一点質問とは関係ないですが、discord.botを使うならcommand使ったほうがやりやすいと思います

emojis = ["{}\N{COMBINING ENCLOSING KEYCAP}".format(num) for num in range(1, 10)]
これ本当にユニコードなんでしょうか？自分もそこまで詳しくないので何とも言えないですが、\NはPython内では改行として認識されていると思われます。(システム上逆スラッシュが\になってしまっています)

効率化するよりもelse内にif,elif...書いていくほうがいいです
どうしても効率化したいのなら

python
1def create_unicode(num):
2    return str(num)+r'\N{COMBINING ENCLOSING KEYCAP}'

python
1def check(reaction,user):
2    return user == message.author
3try:
4    reaction, user = await bot.wait_for('reaction_add', timeout=60.0, check=check)
5except asyncio.TimeoutError:
6    await channel.send('タイムアウト')
7else:
8    for i in range(1,10):
9        unicode = create_unicode(i)
10        if str(reaction.emoji) == 'unicode':
11            await channel.send(str(i)+'の絵文字が押されたよ')
12            continue
13        else:
14            pass
15

こんな感じで関数にすることをお勧めします

投稿2020/07/29 07:25

編集2020/07/29 07:27

Hyugopython

総合スコア148

crepscolo

2020/07/29 08:17

とても詳細かつ丁寧な回答、ありがとうございます。絵文字も分かりづらい表記をしてしまい、失礼いたしました。　先ず、ご回答頂きましたコードで思った通りの反応が得られました。　当方IDLEのみを使ってコード作成を行っており、1⃣の絵文字を使うとIDLEがクラッシュしてしまうためどこかのサイトで見かけた["{}\N{COMBINING ENCLOSING KEYCAP}".format(num) for num in range(1, 10)]を使って1⃣～9⃣を出力していました。　まだDiscord.botを作り始めて間もなく、関数に関してもまだ理解不足ですので、書いてくださった補足情報をじっくり読み込んで今後に活かせればと思っております。（elifの存在もここ数日で知りました…）　今回は本当にありがとうございました。