discordのbotを一度呼び出すとそれ以降動作しなくなるを解決したい

Question

前提・実現したいこと

身内のサーバーで使う目的でDiscordのBOT製作を開始し、ひとまずVoice Channelに参加した状態でBOTを動かせるようにはなったのですが、一度動かすとBOTプログラムを再起動しないとコマンドを受け付けてくれないといった状態になってしまい、それを解決したいと思い質問させていただきました。

発生している問題・エラーメッセージ

エラーメッセージ

Ignoring exception in on_message
Traceback (most recent call last):
  File "C:\Program Files\Python36\lib\site-packages\discord\client.py", line 307, in _run_event
    yield from getattr(self, event)(*args, **kwargs)
  File "suumobot.py", line 27, in on_message
    voice = await client.join_voice_channel(message.author.voice_channel)
  File "C:\Program Files\Python36\lib\site-packages\discord\client.py", line 3177, in join_voice_channel
    raise ClientException('Already connected to a voice channel in this server')
discord.errors.ClientException: Already connected to a voice channel in this server

該当のソースコード

import discord
import youtube_dl

# ログイン処理
client = discord.Client()
@client.event
async def on_ready():
    print('以下のログイン情報でBOTにログインしています。↓↓↓')
    print(client.user.name)
    print(client.user.id)
    print('起動完了')

# 音割れスーモ
@client.event
async def on_message(message):
    if message.author.bot:
        return
    if message.author.voice_channel is None:
        await self.bot.say('ボイスチャンネルに参加した上で、もう一度実行してください。')
        return False


    voice = await client.join_voice_channel(message.author.voice_channel)
    if message.content == ("スーモ"):
        player = await voice.create_ytdl_player('https://www.youtube.com/watch?v=ZiAOvG82xN4')
        player.start()

client.run("token") # 「token」の部分は都合により変更しています。

試したこと

特になし

補足情報（FW/ツールのバージョンなど）

Windows10 Home x64 build 17134.165
Python 3.6.5
aiohttp 1.0.5
async-timeout 3.0.0
cffi 1.11.5
chardet 3.0.4
discord 0.0.2
discord.py 0.16.12
multidict 4.3.1
PyNaCl 1.2.1
six 1.11.0
websockets 3.4
youtube-dl 2018.7.10

Answer 1

voice = None
player = None
@client.event
async def on_message(message):
    global voice, player
    if message.author.bot:
        return
    if message.author.voice_channel is None:
        await client.send_message(message.channel ,'ボイスチャンネルに参加した上で、もう一度実行してください。')
        return
    if voice == None:
        voice = await client.join_voice_channel(message.author.voice_channel)
    # 接続しているかを確認
    elif(voice.is_connected() == True):
        # 接続先で音声を流しているか確認
        if(player.is_playing()):
            # 流していれば停止させる
            player.stop()
        await voice.move_to(message.author.voice_channel)
    player = await voice.create_ytdl_player('https://www.youtube.com/watch?v=ZiAOvG82xN4')
    player.start()

これでどうでしょうか。

---

send_message()は基本的に以下のようにして使います。

await client.send_message(メッセージを送信するチャンネル, '送信するメッセージ')

次に接続の確認ですが以下のような実装になるかと思います。

voice = None
player = None
@client.event
async def on_message(message):
    if message.author.bot:
        return
    if message.author.voice_channel is None:
        await client.send_message(message.channel ,'ボイスチャンネルに参加した上で、もう一度実行してください。')
        return
    if voice == None:
        voice = await client.join_voice_channel(message.author.voice_channel)
    # 接続しているかを確認
    elif(voice.is_connected() == True):
        # 接続先で音声を流しているか確認
        if(player.is_playing()):
            # 流していれば停止させる
            player.stop()
        await voice.move_to(message.author.voice_channel)
    player = await voice.create_ytdl_player('https://www.youtube.com/watch?v=ZiAOvG82xN4')
    player.start()

動かしてはいないので、間違っているかもしれません。

---

いくつか間違いがあるので訂正をさせていただきますね。

まず、肝心の動かなくなる原因ですが、一度コマンドを実行した際にボイスチャンネルにボットが参加すると思います。
二回目にコマンドを実行した場合、もう一度ボイスチャンネルへの接続処理を行うのですが、このときに既にボイスチャンネルへ接続済みであるため、このエラーが発生していると思われます。
なのでコードに接続確認を行う処理を加えてみましょう。

次に、この部分なんですが、self.bot.say()ではなくclient.send_message()を使いましょう。

if message.author.voice_channel is None:
        await self.bot.say('ボイスチャンネルに参加した上で、もう一度実行してください。')
        return False

say()はBotクラスのメソッドでClientクラスには無いので、ここを通ったときに必ずエラーになり動かなくなります。

Comments

[shx64a]

回答ありがとうございます。
>>self.bot.say()ではなくclient.send_message()を使いましょう。

```python
await client.send_message('ボイスチャンネルに参加した上で、もう一度実行してください。')
```
これで合っていますでしょうか？

>>コードに接続確認を行う処理を加えてみましょう。

回答を頂いてから数時間Google等で調べているのですが私の拙い検索スキルでは上手く目的のページまでたどり着けず、またPythonは両手で数えられる位しか触った事が無いという状況です。お手数ですがどのようにして実装すれば良いのかご伝授頂けないでしょうか。何卒宜しくお願いします。

[shx64a]

追記ありがとうございます。
先程実行してみたのですが、以下のエラーメッセージが出力されましたのでご報告いたします。

```
Ignoring exception in on_message
Traceback (most recent call last):
File "C:\Program Files\Python36\lib\site-packages\discord\client.py", line 307, in _run_event
yield from getattr(self, event)(*args, **kwargs)
File "suumobot_1.py", line 27, in on_message
if voice == None:
UnboundLocalError: local variable 'voice' referenced before assignment
```

[shx64a]

再度の追記ありがとうございます。
無事実行する事が出来ました。