Closed fujishit closed 1 year ago
Server::loopのリファクタ 分割したりなどした。
エラー時の動作を変更。 エラーファイルを分け、エラー処理の内容を別ファイルに移したなど。
Server::loopの中身を関数に分けて、呼び出すだけにした。
Server/Error.cppにエラー処理を移動し、エラー処理は呼び出すだけにした。
エラー処理については出力をするだけで具体的にどう処理するのか。 エラーでexitしたときにusersが適切にdeleteされるか確かめていない。 Userクラスのエラー処理も同じようにしたいけどしてない。 Server::start()を分けるみたいな話が持ち上がってたけどやってない。
動作は変わりません。 いつも通り接続と受信ができるか。
あくまで目安です。
issue URL
28
対応内容・対応背景・妥協点
Server::loopのリファクタ 分割したりなどした。
エラー時の動作を変更。 エラーファイルを分け、エラー処理の内容を別ファイルに移したなど。
やったこと
Server::loopの中身を関数に分けて、呼び出すだけにした。
Server/Error.cppにエラー処理を移動し、エラー処理は呼び出すだけにした。
やってないこと
エラー処理については出力をするだけで具体的にどう処理するのか。 エラーでexitしたときにusersが適切にdeleteされるか確かめていない。 Userクラスのエラー処理も同じようにしたいけどしてない。 Server::start()を分けるみたいな話が持ち上がってたけどやってない。
テスト
動作は変わりません。 いつも通り接続と受信ができるか。
レビュー観点
あくまで目安です。
補足