velvelhype commented 1 year ago

issue URL

27

joinの簡易実装

joinの実装のために、ユーザークラスとか、チャンネルクラス作ったりした

色々エラー処理とか、条件付けとかがばい

ほんとにユーザーリストに追加されてるかとか

コマンドの雛形として実装はいい感じか？　見てほしい

fujishit commented 1 year ago

コードだけとりあえず読んだ

関数と変数の命名がスネークになってる。修正は後でも出来るし面倒だったらほっといていいけど一応。
Server::joinの命名がもう少し違ってもいいと思う。なんとなく、コードを読んだときにサーバーにjoinしてる感じがしちゃう気がする。
issueにするか微妙だけど、ログ系を提出時にも出力しっぱなしにするのか、ファイル出力にするのか、全部消すのかによって、ログ出力の仕方を変えたほうがいいような気もする。今のままだと、どれがログの出力なのかわからないよね。

velvelhype commented 1 year ago

スネーク修正しまーーす

join_channelにする

ログね〜〜〜　現状適当すぎる断捨離してみるわ　ファイルは見たくないし、断捨離してデバッグ系は色変えるとかにとりあえずしてみる

fujishit commented 1 year ago

あざすあざす。

ログね〜〜〜　現状適当すぎる断捨離してみるわ　ファイルは見たくないし、断捨離してデバッグ系は色変えるとかにとりあえずしてみる

最後に消すなら、簡単にコメントアウトとかできるように関数にラップしておいてコメントアウト簡単にできるようにしとくのがいいと思う。方針的には色とかで出力するって感じにいったんしてもいいと思う。

fujishit commented 1 year ago

実行は問題なさそう。 joinできてるしirssiも認識してる。limechatは認識しないけど、なんで認識しないのかはよくわかんね。 privmsg出来たらもうほぼircとして完成してる説すらあるよね、結構いい感じだと思う。

hyudai28 commented 1 year ago

挙動問題なしーログについて私はあっていいと思う、ただ整理する必要はあるね。最優先ではないから後回しでいいと思う。