Discord Bot VCに居た時間を測定して表示する

Question

#目的
discordのボイチャに入ったら、入った時間だけ時間を表示されるbotを作りたい。
#言語
python

#バージョン・ソフト
python3.9.4

vscode

#参考にしたサイト
https://qiita.com/tokkq/items/311aa297175b9cf7f946

#コード
import discord
import datetime

client = discord.Client()
pretime_dict = {}

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

@client.event
async def on_voice_state_update(member, before, after):
print("ボイスチャンネルで変化がありました")

if((before.self_mute is not after.self_mute) or (before.voice.self_deaf is not after.voice.self_deaf)):
print("ボイスチャンネルでミュート設定の変更がありました")
return

if(before.voice_channel is None):
pretime_dict[after.name] = datetime.datetime.now()
elif(after.voice_channel is None):
duration_time = pretime_dict[before.name] - datetime.datetime.now()
duration_time_adjust = int(duration_time.total_seconds()) * -1

reply_channel_name = "Study-time"
reply_channel = [channel for channel in before.server.channels if channel.name == reply_channel_name][0]
reply_text = after.name + "　が　"+ before.voice_channel.name + "　から抜けました。　通話時間：" + str(duration_time_adjust) +"秒"

await client.send_message(reply_channel ,reply_text)

client.run("token")

#追記2

import discord
import datetime

client = discord.Client()
pretime_dict = {}

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

@client.event
async def on_voice_state_update(member, before, after):
    print("ボイスチャンネルで変化がありました")

    if((before.self_mute is not after.self_mute) or (before.voice.self_deaf is not after.voice.self_deaf)):
        print("ボイスチャンネルでミュート設定の変更がありました")
        return

    if(before.voice_channel is None):
        pretime_dict[after.name] = datetime.datetime.now()
    elif(after.voice_channel is None):
        duration_time = pretime_dict[before.name] - datetime.datetime.now()
        duration_time_adjust = int(duration_time.total_seconds()) * -1

    reply_channel_name = "#Study-time"
    reply_channel = [channel for channel in before.server.channels if channel.name == reply_channel_name][0]
    reply_text = after.name + "　が　"+ before.voice_channel.name + "　から抜けました。　通話時間：" + str(duration_time_adjust) +"秒"

    await Client.send_message(reply_channel ,reply_text)

client.run("token")

#エラー　※起動してボイチャに入って抜けて終わる動作です。
ボイスチャンネルで変化がありました
Ignoring exception in on_voice_state_update      
Traceback (most recent call last):
  File "C:\Program Files\Python39\lib\site-packages\discord\client.py", line 343, in _run_event   
    await coro(*args, **kwargs)
  File "c:\Users\user\Desktop\nob\Virtual_currency_of_the_tool\noob.py", line 15, in on_voice_state_update
    if((before.self_mute is not after.self_mute) 
or (before.voice.self_deaf is not after.voice.self_deaf)):
AttributeError: 'VoiceState' object has no attribute 'voice'
ログインしました
ボイスチャンネルで変化がありました
Ignoring exception in on_voice_state_update      
Traceback (most recent call last):
  File "C:\Program Files\Python39\lib\site-packages\discord\client.py", line 343, in _run_event   
    await coro(*args, **kwargs)
  File "c:\Users\user\Desktop\nob\Virtual_currency_of_the_tool\noob.py", line 15, in on_voice_state_update
    if((before.self_mute is not after.self_mute) 
or (before.voice.self_deaf is not after.voice.self_deaf)):
AttributeError: 'VoiceState' object has no attribute 'voice'
Exception ignored in: <function _ProactorBasePipeTransport.__del__ at 0x00000255F0F0ADC0>
Traceback (most recent call last):
  File "C:\Program Files\Python39\lib\asyncio\proactor_events.py", line 116, in __del__
    self.close()ボイスチャンネルで変化がありました
Ignoring exception in on_voice_state_update      
Traceback (most recent call last):
  File "C:\Program Files\Python39\lib\site-packages\discord\client.py", line 343, in _run_event   
    await coro(*args, **kwargs)
  File "c:\Users\user\Desktop\nob\Virtual_currency_of_the_tool\noob.py", line 15, in on_voice_state_update
    if((before.self_mute is not after.self_mute) 
or (before.voice.self_deaf is not after.voice.self_deaf)):
AttributeError: 'VoiceState' object has no attribute 'voice'
ログインしました
ボイスチャンネルで変化がありました
Ignoring exception in on_voice_state_update      
Traceback (most recent call last):
  File "C:\Program Files\Python39\lib\site-packages\discord\client.py", line 343, in _run_event   
    await coro(*args, **kwargs)
  File "c:\Users\user\Desktop\nob\Virtual_currency_of_the_tool\noob.py", line 15, in on_voice_state_update
    if((before.self_mute is not after.self_mute) 
or (before.voice.self_deaf is not after.voice.self_deaf)):
AttributeError: 'VoiceState' object has no attribute 'voice'
Exception ignored in: <function _ProactorBasePipeTransport.__del__ at 0x00000255F0F0ADC0>
Traceback (most recent call last):
  File "C:\Program Files\Python39\lib\asyncio\proactor_events.py", line 116, in __del__
    self.close()
コード

Comments

[K_3578]

回答が得られなかった等の理由により、同様の質問を投稿するのは禁止です。
https://teratail.com/questions/333104　は削除リクエストを送ってください。

そして、上記質問でも指摘のあるとおり、teratailではソースコードはMarkDownの<code>ブロック内に
貼り付けてください。インデントがされていないので非常に見づらいです。

[munya24]

すいませんでした

[K_3578]

対応して頂ければ問題ありませんよ。解決したくて焦る気持ちは分かりますが、
厚意で成り立っているコミュニティなので、マナーとモラルを持って質問して頂けると。

<code>ブロックに関しては、使い方がわからなかったら下記質問が参考になると思います。
一度見てみて下さい。
https://teratail.com/questions/238564

[munya24]

見てみます
インテントは1個以上できるんですか？

[K_3578]

インテント→インデントですかね？

ちょっと言ってる意味が理解出来ませんが、恐らく<code>ブロックだと仮定しますが、
<code>ブロック自体は複数使用できます。

ただし、複数使用する場合はブロック間で1行以上スペースを空けてあげないと一つのブロックとして
認識されてしまうので注意してください。

[munya24]

できました
インデントです。すいません

[K_3578]

謝らなくていいです。あまり卑屈になられてもこちらも困惑するので。

[munya24]

はい

Answer 1

voice_state_updateイベントのイベントハンドラには3つの位置引数が渡されるのに before と after しか受け取らないように記述されているからですよ。

@client.event
async def on_voice_state_update(member, before, after):

でどうでしょう。

（追記）

AttributeError: 'VoiceState' object has no attribute 'voice'

参考サイトのコードは投稿日を見る限りかなり古いので参考になりません。
全体的に書き直すことになりそうです。

この AttributeError は、before、afterにはそれぞれVoiceStateオブジェクトが入るのでvoice属性を参照できないために発生します。

before.voice.self_mute などvoice属性を参照する部分は before.self_mute というように置き換えてみてください。

---

###（追記２）

エラーの読み方を知ったほうがよいですね

AttributeError: '〇〇' object has no attribute '▲▲'

このエラー文はそのまま 「▲▲」という属性は 〇〇 オブジェクトにはない、ということです

AttributeError: 'VoiceState' object has no attribute 'voice'

ゆえにこれは VoiceState オブジェクトは voice 属性を持っていないという意味になります。
VoiceState はここでは before と after の型です
どこで voice 属性を参照しているでしょうか？

ここです：

if((before.self_mute is not after.self_mute) or (before.voice.self_deaf is not after.voice.self_deaf)):

before.voice.self_deaf is not after.voice.self_deaf

before や after にvoice属性はないことを念頭に置いてください：

before.self_deaf is not after.self_deaf

###（追記３）

Pythonでは、同じレベルのブロックの中ではインデントの数は同じにする必要があります。
というより、インデントの数によってブロックの深さを表現しています。

    print("ボイスチャンネルで変化がありました")

if((before.self_mute is not after.self_mute) or (before.voice.self_deaf is not after.voice.self_deaf)):
    print("ボイスチャンネルでミュート設定の変更がありました")
    return

  if(before.voice_channel is None):
    pretime_dict[after.name] = datetime.datetime.now()

インデントの数がどれもバラバラなので、これはエラーになります。

ブロックというのは、「:」記号以降に記述された一連の処理です。

# ここは一番浅く

def f():
    #  関数宣言には「:」がつくので1段下げる
    for i in range(5):
        #  for文には「:」がつくので1段下げる

このように、「:」でブロックが発生したら1段下げるように覚えればよいと思います
（インデントは半角4個のスペースで表現するのが一般的）

ゆえに、

@client.event
async def on_voice_state_update(member, before, after):
    print("ボイスチャンネルで変化がありました")

    if((before.self_mute is not after.self_mute) or (before.voice.self_deaf is not after.voice.self_deaf)):
        print("ボイスチャンネルでミュート設定の変更がありました")
        return

    if(before.voice_channel is None):
        pretime_dict[after.name] = datetime.datetime.now()
    elif(after.voice_channel is None):
        duration_time = pretime_dict[before.name] - datetime.datetime.now()
        duration_time_adjust = int(duration_time.total_seconds()) * -1

このように修正するとよいです。

Comments

[munya24]

ボイスチャンネルで変化がありました
Ignoring exception in on_voice_state_update
Traceback (most recent call last):
File "C:\Program Files\Python39\lib\site-packages\discord\client.py", line 343, in _run_event
await coro(*args, **kwargs)
File "C:\Users\user\Desktop\DISCORD\Virtual_currency\bot.py", line 11, in on_voice_state_update
if((before.voice.self_mute is not after.voice.self_mute) or (before.voice.self_deaf is not after.voice.self_deaf)):
AttributeError: 'VoiceState' object has no attribute 'voice'
ボイスチャンネルで変化がありました
Ignoring exception in on_voice_state_update
Traceback (most recent call last):
File "C:\Program Files\Python39\lib\site-packages\discord\client.py", line 343, in _run_event
await coro(*args, **kwargs)
File "C:\Users\user\Desktop\DISCORD\Virtual_currency\bot.py", line 11, in on_voice_state_update
if((before.voice.self_mute is not after.voice.self_mute) or (before.voice.self_deaf is not after.voice.self_deaf)):
AttributeError: 'VoiceState' object has no attribute 'voice'
Exception ignored in: <function _ProactorBasePipeTransport.__del__ at 0x00000166C9D99DC0>
Traceback (most recent call last):
File "C:\Program Files\Python39\lib\asyncio\proactor_events.py", line 116, in __del__
File "C:\Program Files\Python39\lib\asyncio\proactor_events.py", line 108, in close
File "C:\Program Files\Python39\lib\asyncio\base_events.py", line 746, in call_soon
File "C:\Program Files\Python39\lib\asyncio\base_events.py", line 510, in _check_closed
RuntimeError: Event loop is closed
こういうふうになりました。

[munya24]

多分「ボイスチャンネルで変化がありました」というところは、ボイチャに入って抜けたときです。

[coolwind0202]

追記しました。

[munya24]

追記しました。

[munya24]

追記しました。

[coolwind0202]

このエラーはインデントの数が正しくないという意味なのですが、直接コードを貼ると行頭のインデントが表示されないので、

コードを貼ったあと ``` 記号で囲ってください。
Pythonの場合は以下のように：

```py
print("Hello world")
```

[munya24]

はい
わかりました。

[coolwind0202]

追記しました。

[munya24]

追記しました。

[coolwind0202]

なぜbeforeやafterからvoice属性を参照したままなのでしょうか？