Socket发送函数返回0但客户端已收到消息的问题排查（附SendStr代码）

嘿，我一眼就揪出你代码里的关键问题了——这是个运算符优先级的经典坑！你看到的write返回0其实是个假象，实际write已经成功发送了数据，只是你的代码错误地处理了它的返回值。

核心问题拆解 #

看你循环里的这行关键代码：

if (nsend = (write(connfd, sendByte + nsend, nleft)) < 0)

在C++中，<运算符的优先级比=更高，所以这行代码的实际执行顺序是：

1. 先调用write函数，得到实际发送的字节数（比如你测试时一次性发完了所有数据，返回值是正数）
2. 判断这个返回值是否小于0，得到布尔结果false（也就是0）
3. 把这个布尔值（0）赋值给nsend

所以你看到的nsend == 0并不是write的真实返回值，而是比较结果的赋值！这就导致你进入了else if (nsend == 0)分支直接break，但实际上write已经成功发送了数据，所以客户端能正常接收。如果你的数据量更大，需要多次write才能发完，这个错误就会导致数据发送不完整，客户端只能收到部分内容。

修复方案 #

解决方法很简单，给赋值表达式加上括号，让=先执行，再判断结果是否小于0：

if ((nsend = write(connfd, sendByte + nsend, nleft)) < 0)

这样nsend就能正确拿到write的实际返回值，循环也能正常处理剩余未发送的数据。

额外优化建议 #

还有两个小细节可以优化，避免后续踩坑：

• 字节序问题：直接发送主机序的uint16_t长度，跨平台（比如小端主机和大端主机通信）时会解析错误，应该用htons()转换成网络字节序：

  uint16_t len = htons(static_cast<uint16_t>(str.size()));
  

  客户端接收后再用ntohs()转回来。
• 内存安全问题：手动new/delete容易出现内存泄漏，建议用std::vector自动管理内存：

  void SendStr(int connfd, std::string str) {
      uint16_t len = htons(static_cast<uint16_t>(str.size()));
      std::vector<char> sendByte(sizeof(len) + str.size(), 0);
      memcpy(sendByte.data(), &len, sizeof(len));
      memcpy(sendByte.data() + sizeof(len), str.c_str(), str.size());
  
      size_t nleft = sendByte.size();
      int nsend = 0;
      while (nleft > 0) {
          if ((nsend = write(connfd, sendByte.data() + nsend, nleft)) < 0) {
              if (errno == EINTR) continue;
              ERROR_EXIT("send");
          } else if (nsend == 0) {
              break;
          }
          nleft -= nsend;
      }
  }
  

  这样不用手动释放内存，更安全可靠。

内容的提问来源于stack exchange，提问作者h53