C++ 网络，recv() 无缘无故失败（？）

Question

我正在用 C++ 编写一个聊天服务器以供自己消遣，但总的来说我对网络还很陌生。我的知识来自 Beej 的指南 this site，以及 Kurose&Ross 的 Top Down 方法。

我的问题是，我编写的代码与您在链接中找到的代码非常相似，但它不起作用。当我尝试在服务器上使用 recv() 函数时，程序失败。

我知道 recv() 可以返回任意数量的字节，但它只是给了我-1。我知道我需要一个循环来拼凑正在发送的消息，但目前还没有类似的东西。然而，客户端中的 send() 表示它发送了缓冲区中指定的数量（是的，我知道这可能有点矫枉过正，但不确定这是否是一个错误）。

在我看来，服务器似乎……以某种方式关闭了？我很确定我在某个地方搞砸了，有一行代码我真的不确定（寻找评论）。我昨天设法让它工作，但在那之后出现了问题。

我对套接字编程和任何类型的网络都很陌生，但是函数调用和诸如此类的东西来自指南，我有一种预感，问题将出在我指定端口和地址的部分，也许是字节没有到达服务器，因为我将它们发送到其他地方？但是如果accept() 有效，那不就意味着建立了TCP 连接，那之后我不应该能够使用它吗？

我正在使用最新版本的 Lubuntu linux，如果有帮助的话。

如果你发现问题，你能告诉我如何正确地做吗？无论如何，这是服务器的代码：

void start() {

    char message[1024] = "";

    int socketfd = socket( AF_INET, SOCK_STREAM, 0 ), opt = 1, new_socket;

    struct sockaddr_in address;
    struct sockaddr_storage cl_addr;
    socklen_t len = sizeof(address);

    if( socketfd != 0 ) {

        address.sin_family = AF_INET;
        address.sin_port = htons( PORT );
        address.sin_addr.s_addr = INADDR_ANY;

        if( setsockopt(socketfd, SOL_SOCKET, SO_REUSEADDR | SO_REUSEPORT, &opt, sizeof(int)) < 0 ) {
            perror("Setsockopt failed");
            exit(EXIT_FAILURE);
        }

        if( bind( socketfd, (struct sockaddr*)&address, len ) < 0 ) {           
            perror("Couldn't bind to port");
            exit(EXIT_FAILURE);
        } 

        if( listen( socketfd, 3 ) < 0 ) {
            perror("Listening on port failed");
            exit(EXIT_FAILURE);
        }

        if( (new_socket = accept( socketfd, (struct sockaddr *)&cl_addr, (socklen_t*)&len  ) < 0) ) {
            perror("Couldn't accept request");
            exit(EXIT_FAILURE);
        }

        /// This is where it fails
        std::cout << recv( new_socket, message, 1024, 0) <<  std::endl;
        close(new_socket);
    }
    else {
        perror("Couldn't open socket");
        exit(EXIT_FAILURE);
    }
    close( socketfd );
}

现在是客户端：

int main(void) {

int sockfd;

struct sockaddr_in serv_addr;
char hello[1024] = "Hello";

serv_addr.sin_family = AF_INET; 
serv_addr.sin_port = htons(PORT);

/// I'm not sure about this !! 
inet_pton(AF_INET, "127.0.0.1", &serv_addr.sin_addr);

sockfd = socket(AF_INET, SOCK_STREAM, 0);

if( sockfd == 0 ) {
    perror("Opening socket failed");
    exit(EXIT_FAILURE);
}

if( connect(sockfd, (struct sockaddr*)&serv_addr, sizeof(serv_addr)) != -1 ) {
    /// Says it sent 1024, but sever doesn't recieve it, what gives ?
    int test = send( sockfd, hello, sizeof(hello), 0);
    std::cout << test << std::endl;
}
else {
    perror("Error");
    exit(EXIT_FAILURE);
}

如您所见，它与链接中的代码相似。服务器代码只是一个函数，还有其他部分的代码，比如创建用户文件，但没有一个用于网络部分，所以我不想再白发800行代码。服务器的 main() 只是我创建了一个服务器类的对象并在其上调用 start()。

我想在力所能及的地方进行改进，所以如果您发现我可以做得更好，请告诉我，无论是一般的编码风格还是其他方面。另外，这是我在这里的第一个问题，我的第一语言不是英语，请放轻松:)

Answer 1

这是一个括号问题。

这一行：

if( (new_socket = accept( socketfd, (struct sockaddr *)&cl_addr, (socklen_t*)&len  ) < 0) ) {

< 运算符的计算优先级高于 = 运算符。

new_socket 被赋值为 accept() < 0，这是一个错误的表达式，因此 new_socket 被赋值为零。

这可能是你的意思：

if (new_socket = accept( socketfd, (struct sockaddr *)&cl_addr, (socklen_t*)&len ) < 0)

但这更不容易出错：

new_socket = accept( socketfd, (struct sockaddr *)&cl_addr, (socklen_t*)&len  ); 
if (new_socket < 0) {
        perror("Couldn't accept request");
        exit(EXIT_FAILURE);
}