エコーサーバー・クライアントの実装（いただいたコメントの対応）

[thonda28]

Description

#2 の続き（#2 に続けると煩雑になると判断したため、PR を分離）

2025/01/27 に小西さんにいただいたコメント（このコメント以降のもの）に対する対応を行う（詳細は以下に記入）

• SocketManager まわりの修正
  • メモリ確保を new_socket_manager() に、メモリ解放を free_socket_manager() に修正
  • find_socket() は O(N) かかるため、epoll_event->data->ptr に SocketData のポインタをもたせることで性能改善
• SocketData まわりの修正
  • SocketType を設定し、Type を見て処理を分岐
    • 元々 SocketData を find_socket(client_socket_manager, events[i].data.fd) のように取得したためどんなソケットかが判断できたが、ポインタから直接辿れるようになると listen_socket なのか client_socket かの判断がつかなくなったため
    • PIPE もまとめて Socket という扱いに入れている部分は微妙なので命名あたりを修正したほうがよさそう
  • SocketData に buffer_start と buffer_end を追加し、それらを使って recv()/send() 時のバッファを管理
• SIGINT がきたときの安全な終了
  • listen_socket_manager と client_socket_manager を解放してから break（epoll_wait() がブロックしないようにタイムアウトも設定）
  • socket_manager が解放されていなくても一定時間経過後に break
• その他比較的小さめな修正
  • Makefile で $@ や $^ という特殊変数を使用
  • エラー時の puts() は fputs() を使って標準エラー出力へ出力
    • puts() は改行あり、fputs() は改行なしという差分に気づいたので、そのあたりも整理
  • int 型にもかかわらず is_xxx となっていた変数名を修正
  • ほぼ同じコードの共通化
  • ひとまとまりの処理を関数化して分離
  • SIGPIPE の対応をしなくてよいように MSG_NOSIGNAL フラグを send() に設定
  • などなど

コミットを細かめに分けたので、コミット単位で見ていただくのがわかりやすいかもです。

Note

3点わかっていない点があり、それは #2 の コメント に返信（以下詳細）

• remove_socket() を O(1) にする方法
  • エコーサーバー・クライアントの実装（追加機能） #2 (comment)
• 送り返すデータがない場合には EPOLLOUT を受け取らない方法
  • エコーサーバー・クライアントの実装（追加機能） #2 (comment)
• SIGINT を受信した際に EWOULDBLOCK が起き、再度ループに入ったときに何が起こるか
  • エコーサーバー・クライアントの実装（追加機能） #2 (comment)