核心 Bug:客户端与服务器端口不匹配(导致请求无法送达)

Bug 分析报告

1. 核心 Bug:客户端与服务器端口不匹配(导致请求无法送达)
  • 问题描述 :客户端代码中定义的服务器端口为 69(static const int PORT = 69),而服务器代码中绑定的端口为 6969(static const int PORT=6969)。由于 UDP 通信中客户端必须发送数据到服务器实际监听的端口,端口不匹配会导致服务器无法接收客户端的下载 / 上传请求,最终表现为功能失败。
  • 定位代码
    • 客户端端口定义(TFTPClient类私有成员):

      cpp

      运行

      复制代码
      static const int PORT = 69;  // 客户端使用的服务器端口
    • 服务器端口定义(TFTPServer类私有成员):

      cpp

      运行

      复制代码
      static const int PORT=6969;  // 服务器实际监听的端口
2. 次要 Bug:客户端输入缓冲区处理不当(可能导致文件名读取失败)
  • 问题描述 :客户端run()函数中,用户输入选择(1/2/3)后调用waitForInput()清空缓冲区,但在doDownload()doUpload()中使用getline(cin, filename)读取文件名时,可能因缓冲区残留数据导致读取空字符串或错误内容。
  • 定位代码
    • run()中输入选择后调用waitForInput()

      cpp

      运行

      复制代码
      cin >> choice;   // 输入选项
      waitForInput(); // 吸收垃圾字符
    • waitForInput()实现(仅清空到换行符,但未考虑cin的状态):

      cpp

      运行

      复制代码
      void TFTPClient::waitForInput() {
          while (getchar() != '\n'); // 纯清空输入缓冲区直到换行符
      }
    • 若用户输入选择后有多余字符(如1 abc\n),waitForInput()会将abc也清空,导致后续getline需要重新输入,但更严重的是,若cin状态异常(如输入非字符),可能导致getline直接失败。

Bug 解决方案

1. 修复端口不匹配问题
  • 修改客户端端口 :将客户端TFTPClient类中的PORT修改为与服务器一致的 6969,确保客户端请求能发送到服务器监听的端口。

    cpp

    运行

    复制代码
    // 客户端TFTPClient类私有成员修改
    static const int PORT = 6969;  // 与服务器端口保持一致
2. 优化输入缓冲区处理(避免文件名读取失败)
  • 改进waitForInput()函数 :在清空缓冲区前先检查cin的状态,确保输入流正常,避免因输入异常导致后续getline失效。

    cpp

    运行

    复制代码
    void TFTPClient::waitForInput() {
        // 若cin处于错误状态,先清空错误标志
        if (cin.fail()) {
            cin.clear();
        }
        // 清空缓冲区直到换行符
        while (getchar() != '\n');
    }
  • getline前手动清空缓冲区 :尽管waitForInput()已处理,但为确保getline能正确读取文件名,可在doDownload()doUpload()getline前再做一次缓冲区检查(可选增强):

    cpp

    运行

    复制代码
    // doDownload()中读取文件名前添加
    cout << "请输入要下载的文件的文件名称:";
    cin.ignore(numeric_limits<streamsize>::max(), '\n'); // 强制清空残留缓冲区
    getline(cin, filename);
    
    // doUpload()中同理
    cout << "请输入要上传的文件名:";
    cin.ignore(numeric_limits<streamsize>::max(), '\n');
    getline(cin, filename);
3. 补充错误处理(增强鲁棒性)
  • 服务器路径拼接容错 :服务器handleReadRequesthandleWriteRequest中拼接文件路径时,处理root_dir/结尾的情况(避免出现//):

    cpp

    运行

    复制代码
    // 服务器拼接路径时修改
    string full_path;
    if (!root_dir.empty() && root_dir.back() == '/') {
        full_path = root_dir + filename;
    } else {
        full_path = root_dir + "/" + filename;
    }
  • 客户端超时处理 :客户端recvfrom时添加超时机制,避免因服务器无响应导致程序卡死(使用setsockopt设置SO_RCVTIMEO):

    cpp

    运行

    复制代码
    // 客户端构造函数中添加超时设置(如5秒超时)
    struct timeval timeout = {5, 0}; // 5秒
    setsockopt(sfd, SOL_SOCKET, SO_RCVTIMEO, &timeout, sizeof(timeout));

总结

主要问题是客户端与服务器端口不匹配,导致请求无法送达,修复端口一致即可解决核心功能失败问题。输入缓冲区处理和错误处理的优化可进一步提升程序稳定性。

相关推荐
哈基咪怎么可能是AI4 小时前
为什么我就想要「线性历史 + Signed Commits」GitHub 却把我当猴耍 🤬🎙️
linux·github
十日十行21 小时前
Linux和window共享文件夹
linux
木心月转码ing1 天前
WSL+Cpp开发环境配置
linux
崔小汤呀2 天前
最全的docker安装笔记,包含CentOS和Ubuntu
linux·后端
何中应2 天前
vi编辑器使用
linux·后端·操作系统
何中应2 天前
Linux进程无法被kill
linux·后端·操作系统
何中应2 天前
rm-rf /命令操作介绍
linux·后端·操作系统
何中应2 天前
Linux常用命令
linux·操作系统
葛立国2 天前
从 / 和 /dev 说起:Linux 文件系统与挂载点一文理清
linux
DianSan_ERP3 天前
电商API接口全链路监控:构建坚不可摧的线上运维防线
大数据·运维·网络·人工智能·git·servlet