Pythonを用いてのIRC botととの通信ができない

Question

#エラー概要
IRCとの通信ができない。

詳細

こちらのチュートリアルを基にPythonを用いてのIRCとの通信接続を試みようとしていますが、型違反？と思われるエラーが発生してうまくいきません。

import socket
import sys

class IRC:
    irc = socket.socket()

    def __init__(self):
        self.irc = socket.socket(socket.AF_INET, socket.SOCK_STREAM)

    def send(self, chan, msg):
        self.irc.send("PRIVMSG" + chan + " " + msg + "n")

    def connect(self, server, channel, botnick):
        # defines the socket
        print("connecting to:" + server)
        self.irc.connect((server, 6667))     # connects to the server
        self.irc.send("USER " + botnick + " " + botnick + " " + botnick + " :This is a fun bot!n")   # user authentication
        self.irc.send("NICK " + botnick + "n")
        self.irc.send("JOIN " + channel + "n")  # join the channel

    def get_text(self):
        text=self.irc.recv(2040)    # receive the text

        if text.find('PING') != -1:
            self.irc.find('PONG' + text.split()[1] + 'rn')
        return text

from irc import *
import os
import random

channel = "#testit"
server = "irc.freenode.net"
botnick = "reddity"

irc = IRC()
irc.connect(server, channel, botnick)

while 1:
    text = irc.get_text()
    print(text)

    if "PRIVMSG" in text and channel in text and "hello" in text:
        irc.send(channel, "HEllo!")

bot.pyを実行した所、以下のエラーが発生しました。

/home/yuta/Desktop/work/python/PycharmProjects/TestPJ/venv/bin/python /home/yuta/Desktop/work/python/PycharmProjects/TestPJ/networking/bot.py
connecting to:irc.freenode.net
Traceback (most recent call last):
  File "/home/yuta/Desktop/work/python/PycharmProjects/TestPJ/networking/bot.py", line 10, in <module>
    irc.connect(server, channel, botnick)
  File "/home/yuta/Desktop/work/python/PycharmProjects/TestPJ/networking/irc.py", line 17, in connect
    self.irc.send("USER " + botnick + " " + botnick + " " + botnick + " :This is a fun bot!n")   # user authentication
TypeError: a bytes-like object is required, not 'str'

Process finished with exit code 1

TypeError: a bytes-like object is required, not 'str'
↑はbyte型で指定しないといけないという意味はなんとなくわかるのですが、具体的にどこを修正すればよろしいのかイマイチ理解できておりません。

わかる人がいましたらご教授お願いいたします。

2020/4/26追記
指摘点を基に修正しましたが新たなエラーが出ました。

import socket
import sys

class IRC:
    irc = socket.socket()

    def __init__(self):
        self.irc = socket.socket(socket.AF_INET, socket.SOCK_STREAM)

    def send(self, chan, msg):
        self.irc.send("PRIVMSG " + chan + " " + msg + "\n")

    def _send_raw(self, line, encoding="utf-8"):
        self.irc.send(line.encode(encoding) + b"\r\n")

        def connect(self, server, channel, botnick):
            # defines the socket
            print("connecting to:" + server)
            self.irc.connect((server, 6667))     # connects to the server
            self.irc.send("USER " + botnick + " " + botnick + " " + botnick + " :This is a fun bot!\n")   # user authentication
            self.irc.send("NICK " + botnick + "\n")
            self.irc.send("JOIN " + channel + "\n")  # join the channel

    def get_text(self):
        text=self.irc.recv(2040)    # receive the text

        if text.find(b'PING') != -1:
            self._send_raw('PONG' + text.split()[1])
        return text

from irc import *
import os
import random

channel = "#testit"
server = "irc.freenode.net"
botnick = "reddity"

irc = IRC()
irc.connect(server, channel, botnick)

while 1:
    text = irc.get_text()
    print(text)

    if "PRIVMSG " in text and channel in text and "hello" in text:
        irc.send(channel, "HEllo!")

/home/yuta/Desktop/work/python/PycharmProjects/TestPJ/venv/bin/python /home/yuta/Desktop/work/python/PycharmProjects/TestPJ/networking/bot.py
Traceback (most recent call last):
  File "/home/yuta/Desktop/work/python/PycharmProjects/TestPJ/networking/bot.py", line 10, in <module>
    irc.connect(server, channel, botnick)
AttributeError: 'IRC' object has no attribute 'connect'

Process finished with exit code 1

connect属性を持っていないとエラーがでました。
irc.pyの該当属性を入れ子にしてタブをずらしたことで属性を見つけられなくなったと推測できたのですが、この場合botの呼び出しの引数をどのようにirc.pyの方へ渡せば良いのでしょうか？
わかる方がいらしたらぜひアドバイスをお願い致します。

2020/04/16追記
指摘を基に修正したところ新たなエラーがでました。

import socket
import sys

class IRC:
    irc = socket.socket()

    def __init__(self):
        self.irc = socket.socket(socket.AF_INET, socket.SOCK_STREAM)

    def send(self, chan, msg):
        self.irc._send_raw("PRIVMSG " + chan + " " + msg + "\n")

    def _send_raw(self, line, encoding="utf-8"):
        self.irc._send_raw(line.encode(encoding) + b"\r\n")

    def connect(self, server, channel, botnick):
        # defines the socket
        print("connecting to:" + server)
        self.irc.connect((server, 6667))     # connects to the server
        self.irc._send_raw("USER " + botnick + " " + botnick + " " + botnick + " :This is a fun bot!\n")   # user authentication
        self.irc._send_raw("NICK " + botnick + "\n")
        self.irc._send_raw("JOIN " + channel + "\n")  # join the channel

    def get_text(self):
        text=self.irc.recv(2040)    # receive the text

        if text.find(b'PING') != -1:
            self._send_raw('PONG' + text.split()[1])
        return text

/home/yuta/Desktop/work/python/PycharmProjects/TestPJ/venv/bin/python /home/yuta/Desktop/work/python/PycharmProjects/TestPJ/networking/bot.py
connecting to:irc.freenode.net
Traceback (most recent call last):
  File "/home/yuta/Desktop/work/python/PycharmProjects/TestPJ/networking/bot.py", line 10, in <module>
    irc.connect(server, channel, botnick)
  File "/home/yuta/Desktop/work/python/PycharmProjects/TestPJ/networking/irc.py", line 20, in connect
    self.irc._send_raw("USER " + botnick + " " + botnick + " " + botnick + " :This is a fun bot!\n")   # user authentication
AttributeError: 'socket' object has no attribute '_send_raw'

Process finished with exit code 1

クラスsocketが_send_raw属性を持たないとエラーが出ていますが、なぜクラスsocketがここで出てくるのかよくわかりません。
IRCクラス内の属性だと思ったのですが、この辺の仕組みがよく理解できていないように思います。
エラーの原因がわかりましたらご教授お願いいたします。

2020/4/17追記

/home/yuta/Desktop/work/python/PycharmProjects/TestPJ/venv/bin/python /home/yuta/Desktop/work/python/PycharmProjects/TestPJ/networking/bot.py
connecting to:irc.freenode.net
b':barjavel.freenode.net NOTICE * :*** Looking up your hostname...\r\n'
Traceback (most recent call last):
  File "/home/yuta/Desktop/work/python/PycharmProjects/TestPJ/networking/bot.py", line 16, in <module>
    if "PRIVMSG " in text and channel in text and "hello" in text:
TypeError: a bytes-like object is required, not 'str'

Process finished with exit code 1

from irc import *
import os
import random

channel = "#testit"
server = "irc.freenode.net"
botnick = "reddity"

irc = IRC()
irc.connect(server, channel, botnick)

while 1:
    text = irc.get_text()
    print(text)

    if "PRIVMSG " in text and channel in text and "hello" in text:
        irc.send(channel, "HEllo!")

呼び出す側のbotで型エラーがおきました。

strをbyteに直す必要がありますが、どこを指しているのか不明です。
自分では全然検討もつかなくものすごく申し訳ないですが、わかる人がいましたらご教授お願いいたします。

Answer 1

class IRC:
    # ここは不要です
    # irc = socket.socket()

    def __init__(self):
        self.irc = socket.socket(socket.AF_INET, socket.SOCK_STREAM)

    def send(self, chan, msg):
        self._send_raw("PRIVMSG " + chan + " " + msg + "\n")

    def _send_raw(self, line, encoding="utf-8"):
        # 文字列→バイト型にしてから送信
        self.irc.send(line.encode(encoding))

    def connect(self, server, channel, botnick):
        # defines the socket
        print("connecting to:" + server)
        self.irc.connect((server, 6667))     # connects to the server
        self._send_raw("USER " + botnick + " " + botnick + " " + botnick + " :This is a fun bot!\n")   # user authentication
        self._send_raw("NICK " + botnick + "\n")
        self._send_raw("JOIN " + channel + "\n")  # join the channel

    def get_text(self):
        text=self.irc.recv(2040)    # receive the text

        if text.find(b'PING') != -1:
            self._send_raw('PONG' + text.split()[1])
        return text

修正箇所

• _send_raw メソッドの中身
• self.irc._send_raw -> self._send_raw
• もう一点追加で、末尾の改行文字が2重に追加されるので

self.irc.send(line.encode(encoding)) の b'\r\n' を外しました。

コードの修正箇所を少なくする為。_send_raw内のを変更しましたが、
呼び出し側で追加するか、_send_raw内で追加するか、どちらか片方で良いです。

---

追記

from irc import *
import os
import random

channel = "#testit"
server = "irc.freenode.net"
botnick = "reddity"

irc = IRC()
irc.connect(server, channel, botnick)

while 1:
    text = irc.get_text().decode("utf-8")
    print(text)

    if "PRIVMSG" in text and channel in text and "hello" in text:
        irc.send(channel, "HEllo!")

変更点:
text = irc.get_text().decode("utf-8")

text がバイト型の場合

• b"PRIVMSG" in text
• b"hello" in text
• とできますが、 channel in text の channel は 文字列なので変換が必要

但し、irc.send は文字列を要求されるので。元のtextを文字列に変換するようにしました。
エンコードは固定で utf-8 としてます。

Comments

[Yuta_for]

度々ありがとうございます。
申し訳ない事にまだ解決出来なくて本文中に追記いたしました。
textとchannelに問題があるのでしょうか？

[teamikl]

ここも受信したデータは文字列ではなくバイト型になってるので、
型を合わせる為に b"RRIVMSG" b"hello" のようにします

[teamikl]

channel in text も型を合わせる必要がありました。
もう一点修正、回答に追記します

[Yuta_for]

できました！
何度もアドバイスくださりありがとうございます！

Answer 2

参考もとにしたコードが Python2.x 系のものなのだと思います。
2.x -> 3.x への大きな変更で、文字列とバイト列を区別するようになったので、
文字コードを指定してバイト型に変換する必要があります。

• また、エラーの内容とは別の部分ですが、行末の "n" や "rn" に関しては、
  エスケープシーケンスが消えてしまってるので "\n", "\r\n" に。
• PONGを返すところはtypoで、find ではなく send です。
• "PRIVMSG " <- スペースが必要です

修正方法の一例を挙げると。

• b"..." はバイト型のリテラル
• "..." の文字列は、.encode("utf-8") で変換。

IRCプロトコルの規定に文字コードの規約はなかったので、
文字コードはサーバーの設定に依存します。
(freenodeは未テストなので解りません) クライアントの設定確認utf-8 でした。
例えば、他の日本語のサーバーならiso-2022-jp等がありました。

- self.irc.find('PONG ' + text.split() [1] + 'rn') 
+ self.irc.send(b'PONG ' + text.split()[1].encode("utf-8") + b'\r\n') 

送受信は全部変更する必要があるので、メソッドにしておくとよいです。

# send() メソッドは使われていたので、
# メソッド名は、何か適当に内部で使われるもの風な名前に。

    def _send_raw(self, line, encoding="utf-8"):
        self.irc.send(line.encode(encoding) + b"\r\n")

# 呼び出し側: 例)
self._send_raw('PONG ' + text.split()[1])

受信するデータもバイト型になっています。
値チェック等は文字列と同じ find が使えますが、
バイト型のデータが要求されます。

- if text.find('PING') != -1:
+ if text.find(b'PING') != -1:

Comments

[Yuta_for]

ご回答ありがとうございます。
ご指摘頂いた箇所を基に修正したら新たにエラーが発生しましたので、もしよろしければこちらの問題点もお願いしてもよろしいでしょうか？

[teamikl]

socketのconnect と区別が必要ですが、

IRCクラス内の def connect が全体的にインデントが１段階ずれてます。

_send_raw内部で定義するようになってるので、他の関数と同じ位置に合わせると直るはずです。
タブ文字を使うと見かけは同じでも、タブ幅4や8だったりして、
インデントが合ってないことも有るので、(Pythonでは)基本インデントはスペースにして下さい。

追記: 他の self.irc.send を直接呼んでいる箇所も、TypeErrorとなるので、
_send_raw を用いるように修正が必要です。

[Yuta_for]

何度もご指摘ありがとうございます。
大変申し訳ないですが、私の理解が足りておらず新しいエラーへの対処ができずまた助けが必要になりました。
何度もお願いしてしまい大変恐縮なのですが、こちらもご確認いただけますでしょうか？

[teamikl]

最初に修正した箇所と同じ PONGの所と同様で
self.irc._send_raw -> self._send_raw です

[teamikl]

IRCクラスの中と外にircという変数があるので混同されたのだと思います。
irc = IRC() これは IRCクラスのインスタンス
class IRC内の self.irc = socket.socket() これはソケット

[teamikl]

それから、 def _send_raw の中身も見直してください。

ここはソケットに対してバイト型のデータを送るので self.irc.send です
他の箇所は、直接ソケットにデータを送らず、_send_raw に文字列を渡して
文字列 -> バイト型変換してから送るようにします