Winsock2客户端无法连接指定IPv4服务器问题求助

Let's break down why you're having trouble connecting from external machines (even though localhost works) and fix the code and configuration issues step by step.

First: Critical Bugs in Your Server Code #

Your server code has several fatal issues that are preventing proper external connections, even if your network setup is correct:

1. Incorrect WSAStartup/WSACleanup Placement
   You're calling WSAStartup and WSACleanup inside your main while loop. This means every time a connection ends, you're tearing down the Winsock environment and reinitializing it on the next loop iteration. This is unnecessary and can cause unexpected behavior. These calls should be done once at the start and end of your server program.

2. Closing the Listening Socket After First Connection
   Right after accepting a client with accept(), you call closesocket(listening). This destroys the listening socket, so your server can only accept one single connection ever. After the first client disconnects, your server will never accept new connections again—this explains why external connections might fail (if another connection attempt happens after the first, or if the initial setup is broken by this).

3. Invalid String Comparison
   You're using if (buf == "testingfromclient") to compare the received buffer to a string. In C++, this compares the memory address of the buf array to the address of the string literal, not the actual content. This will never be true, so your server will always send "stranger" instead of "friend". Use strcmp(buf, "testingfromclient") == 0 or convert buf to a std::string for comparison.

4. Incorrect Send Length
   When sending senddata, you're passing bytesReceived as the length parameter. But senddata's length isn't related to the number of bytes received—you should use senddata.size() + 1 to include the null terminator, so the client can properly parse the string.

Fixed Server Code #

Here's the corrected version of your server code addressing all these issues:

#include <iostream>
#include <winsock2.h>
#include <ws2tcpip.h>
#include <string>
#include <cstring>
#include <windows.h>

#pragma comment(lib, "ws2_32.lib")

int main() {
    // Initialize Winsock ONCE at the start
    WSADATA wsaData;
    int wsOk = WSAStartup(MAKEWORD(2, 2), &wsaData);
    if (wsOk != 0) {
        std::cerr << "Can't initialize Winsock. Error: " << wsOk << std::endl;
        return 1;
    }

    // Create listening socket once
    SOCKET listening = socket(AF_INET, SOCK_STREAM, 0);
    if (listening == INVALID_SOCKET) {
        std::cout << "Can't create listening socket. Error: " << WSAGetLastError() << std::endl;
        WSACleanup();
        return 1;
    }

    // Bind the socket
    sockaddr_in hint;
    hint.sin_family = AF_INET;
    hint.sin_port = htons(/* Replace with your port number */); // Match client's port
    hint.sin_addr.S_un.S_addr = INADDR_ANY;

    int bindResult = bind(listening, (sockaddr*)&hint, sizeof(hint));
    if (bindResult == SOCKET_ERROR) {
        std::cout << "Bind failed. Error: " << WSAGetLastError() << std::endl;
        closesocket(listening);
        WSACleanup();
        return 1;
    }

    // Start listening
    listen(listening, SOMAXCONN);
    std::cout << "Server started, listening for connections..." << std::endl;

    while (true) {
        std::cout << "============= Waiting for new connection =============" << std::endl;

        // Accept client connection
        sockaddr_in client;
        int clientSize = sizeof(client);
        SOCKET clientSocket = accept(listening, (sockaddr*)&client, &clientSize);
        if (clientSocket == INVALID_SOCKET) {
            std::cout << "Accept failed. Error: " << WSAGetLastError() << std::endl;
            continue;
        }

        // Get client info
        char host[NI_MAXHOST];
        char service[NI_MAXSERV];
        ZeroMemory(host, NI_MAXHOST);
        ZeroMemory(service, NI_MAXSERV);

        if (getnameinfo((sockaddr*)&client, sizeof(client), host, NI_MAXHOST, service, NI_MAXSERV, 0) == 0) {
            std::cout << host << " connected on port " << service << std::endl;
        } else {
            inet_ntop(AF_INET, &client.sin_addr, host, NI_MAXHOST);
            std::cout << host << " connected on port " << ntohs(client.sin_port) << std::endl;
        }

        // Handle client communication
        char buf[4096];
        while (true) {
            ZeroMemory(buf, 4096);
            int bytesReceived = recv(clientSocket, buf, 4096, 0);

            if (bytesReceived > 0) {
                std::cout << "============= Received bytes from client =============" << std::endl;
                printf("Bytes received: %d\n", bytesReceived);

                std::string senddata;
                // Correct string comparison
                if (strcmp(buf, "testingfromclient") == 0) {
                    senddata = "friend";
                } else {
                    senddata = "stranger";
                }

                // Send with correct length
                int sendResult = send(clientSocket, senddata.c_str(), senddata.size() + 1, 0);
                if (sendResult == SOCKET_ERROR) {
                    std::cout << "Send failed. Error: " << WSAGetLastError() << std::endl;
                    break;
                }
                std::cout << "Sent data: " << senddata << std::endl;
                Sleep(1000);
            } else if (bytesReceived == 0) {
                printf("Connection closing...\n");
                break;
            } else {
                printf("Recv failed with error: %d\n", WSAGetLastError());
                break;
            }
        }

        // Close client socket after communication ends
        closesocket(clientSocket);
        std::cout << "============= Connection ended =============" << endl;
    }

    // Cleanup Winsock ONCE at the end (unreachable with infinite loop, but good practice)
    closesocket(listening);
    WSACleanup();
    std::cout << "Server ended" << std::endl;
    return 0;
}

Network Configuration Troubleshooting #

Even with fixed code, you need to verify these network settings:

• Local Network Test First: Before testing external connections, connect from another machine on the same local network using your server's local IPv4 address (e.g., 192.168.1.100). If this fails:

  • Double-check that your Windows firewall allows your server program (not just the port rule)—go to Windows Defender Firewall > Allow an app through firewall, and make sure your server executable is listed and allowed for private/public networks as needed.
  • Ensure no other program is using the same port on the server. Use netstat -ano | findstr :[your-port] in Command Prompt to check.

• External Network/Port Forwarding:

  • Confirm your router's port forwarding rule is pointing to your server's local IPv4 address and the correct TCP port (your code uses TCP, so UDP rules aren't needed for this setup).
  • Check if your ISP uses CGNAT (Carrier-Grade NAT)—many home ISPs do this, which means you don't have a public IPv4 address. In this case, port forwarding won't work, and you'll need to use a service like ngrok for testing, or switch to a public cloud server.
  • When testing from an external network, use your router's public IPv4 address (you can find this by searching "what's my IP" on the server machine).

Client Code Notes #

Your client code looks mostly okay, but make sure:

• The port variable matches the port the server is using.
• When connecting from an external network, you're using the router's public IP, not the server's local IP.

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