hyudai28 / ft_irc

0 stars 0 forks source link

Make ircd #19

Closed fujishit closed 1 year ago

fujishit commented 1 year ago

issue URL

15

対応内容・対応背景・妥協点

やったこと

bircdをリファクタした。 Serverクラスを作成し、Server.start()とServer.loop()で動くようにした。

やってないこと

Server.loop()の中身はほぼコピペなので改善の余地あり。 エラー処理がexitなので、エラー処理の方法にも改善の余地あり。

テスト

ircクライアントから接続できるか。 コマンドを送信したときにターミナルで出力されるか。

レビュー観点

上記動作に問題がないか。 server.loop()をもう少し綺麗にしたいね。

補足

bircdと同じようにselectで実装しているが、pollで実装することも考えられそう。 詳しくは調べてないがpollだとfdの上限がない?みたいな要素があるらしい。要確認。

fujishit commented 1 year ago

makefileも一応作ってあるけど適当なので、不満あればください

velvelhype commented 1 year ago

>>ircクライアントから接続できるか。 >>コマンドを送信したときにターミナルで出力されるか。

動作確認しました〜 俺もうみたんでhyudaiさん確認後マージおなしゃす