discordbotのコード

Question

discordのbotで天気をlivedoorから取得しようとしたら

コード

import urllib.request
import json
import re
CHANNEL_ID = 602723871522553871
client = discord.Client()
citycodes = {
    "東京": 130010,
    "大阪": 270000,
    "神戸": 280010,
    "京都": 260010,
    "大津": 250010,
    "奈良": 290010,
}

@client.event
async def on_ready():
    print('login done')

@client.event
async def on_message(message):
  if message.author != client.user:

    reg_res = re.compile(u'Bot君、(.+)の天気は？').search(message.content)
    if reg_res:

      if reg_res.group(1) in citycodes.keys():

        citycode = citycodes[reg_res.group(1)]
        resp = urllib.request.urlopen('http://weather.livedoor.com/forecast/webservice/json/v1?city=%s'%citycode).read()
        resp = json.loads(resp.decode('utf-8'))

        msg = resp['location']['city']
        msg += "の天気は、/n"
        for f in resp['forecasts']:
          msg += f['dateLabel'] + 'が' + f['telop'] + '\n'
        msg += 'です。'

        await channel.send(message.author.mention + msg)

      else:
        await channel.send('なにそんな過疎地知らないよ^^')


client.run("")```

#内容
エラーは吐かないのですが、hiには反応するのにBot君、どこか(大阪とかすべて)の天気は？と聞いても反応しません。そして、場所に違うものを入れても、反応しません。ですので、コードの問題だと思うのですが、調べてもわからなかったのでここにあげました。回答していただけると嬉しいです。

#エラー
ご指摘ありがとうございます。エラーが出ないことは解決しましたが、エラーの解決方法がわかりません。
エラーはこれです

![イメージ説明](a5d5480f618c6b042fc8cb1e62f1b21c.png)
![イメージ説明](659626dab5ff3f5428826e0e4fdfa888.png)
![イメージ説明](937b32bbd401d4e6c8c1b507c1f0c16b.png)

Comments

[suama]

こんにちは。可能でしたらソースコードは ``` でくくっていただけると見やすくて助かります。
きになるところを回答に添えておきますね。

Answer 1

こんにちは。
Pythonだとインデントが重要なので、できればコードの投稿の仕方を参考にして貼り付けてくださいませ。

そのうえで、動かしていないのでちょっと確実とは言えないのですが、

def on_message(message) の処理が2つ書かれていますよね。

読み込みの順場で、後からの def on_message(message)のほうで上書きされてしまうと思います。
なので、hiの呼びかけには答えますがお天気の処理は頑張ってコードを書いていてもスルーされてしまうのではと...。

いったん後からの def on_message(message)のほうを＃をつけてコメントアウトして実行すると、どうなるでしょうか？

def on_message(message)の部分で条件分岐をさせる感じでやってくとのかなあと思います。
こちらの記事を参考に...。
https://qiita.com/1ntegrale9/items/9d570ef8175cf178468f

20190727追記：

コメントにも記載しましたが、回答に添えておきます。

キャプチャありがとうござます。重ねてお願いですが、ソースコードはと囲んだ形で貼り付けていいただけないでしょうか？
上記だと、「インデントが崩れているのでエラーにしかなりません」としか言えなくなってしまうので。
質問の際のテンプレートに出てきように記載してください。
アドバイスが難しくなってしまうので、今回のみ。

さて、一番シンプルなDiscordに接続するプログラムを書いてみて、そこから広げていくといいと思います。

最初に書かれていた、エラーは吐かないのですが、hiには反応するのに というのは、Discord上ではメッセージを返してくれていたということですよね。
このときは、メッセージを送信するのは message.channel.send(...) になっていたはずです。

上記のキャプチャだと、 Messageable.send(...) になっています。
エラーメッセージをみると、 「 Messageableなんて知らないよ！」と言われています。

もう一点、画面のエラーメッセーですが、"Unable to import discord" と表示されているので、そもそもDiscord用のライブラリが読めていますでしょうか？
なにか修正したり試したため、このエラーが出てくる様になったのであれば、まずはここを確認で。

その上で、Messageable.send(...) じゃなくて、message.channel.send(送信したい内容) ではないでしょうか？

Comments

[ficorajo]

ご指摘ありがとうございます。ご指摘の通りにすると、エラーをちゃんと吐いてくれました。エラーを加筆したので、それについてもお教え願いませんでしょうか。

[suama]

キャプチャありがとうござます。重ねてお願いですが、ソースコードは ``` .... ``` と囲んだ形で貼り付けていいただけないでしょうか？
上記だと、「インデントが崩れているのでエラーにしかなりません」としか言えなくなってしまうので。

さて、一番シンプルなDiscordに接続するプログラムを書いてみて、そこから広げていくといいと思うのですが、最初に書かれていた、 "エラーは吐かないのですが、hiには反応するのに" というのは、Discord上ではメッセージを返してくれていたということですよね。
このときは、メッセージを送信するのは message.channel.send(...) になっていたはずです。

上記のキャプチャだと、 Messageable.send(...) になっています。
エラーメッセージをみると、 「 Messageableなんて知らないよ！」と言われていますね。

もう一点、画面のエラーメッセーですが、"Unable to import discord" と表示されているので、そもそもDiscord用のライブラリが読めていますでしょうか？
編集されていたのであれば、まずはここを確認で。

その上で、Messageable.send(...) じゃなくて、message.channel.send(送信したい内容) ではないでしょうか？

[ficorajo]

最後のを変更すると、問題のほうは消えたのですが、
Traceback (most recent call last):
File "D:\Program Files (x86)\Microsoft Visual Studio\Shared\Python37_64\lib\site-packages\discord\client.py", line 270, in _run_event
await coro(*args, **kwargs)
File "discordbot.py", line 39, in on_message
await message.channel.send(message.channel,message.author.mention + msg)
TypeError: send() takes from 1 to 2 positional arguments but 3 were given
と、また新たなエラーが出ました。そして、await coro()あたりのエラーの理由がわからないのですが、お教えいただけますでしょうか

[suama]

現時点のソースがわからないので、助けようがないので、すみません。
（最初に質問をした時点から変わってますよね？）

とにかくエラーメッセージを読んで見てくださいね。
File "discordbot.py", line 39, in on_messageとあるので、とにかく3９行目を見ること。
で、エラーは英語ですけど、「send() メソッドにわたす引数の数が違ってます！3個渡ってきてるけど１個か2個だよ！」と言われているだけですね。