c++ - 绑定(bind) : Socket operation on non-socket

Question

我编写了一个服务器和一个客户端，并不断收到“绑定(bind):非套接字上的套接字操作”。

我研究了这个问题，让其他代码在另一个应用程序中运行，并且已经耗尽了 8 个小时试图找到这个错误。

代码是:

void TCPSocket::buildTCPSocket(int port)
{
    initializeSocket1();
    getSocket();
    bindSocket();
    listenToSocket();
    acceptSocket();
         // now you can send() and recv() with the
        // connected client via socket connectedTCPSocket
}

void TCPSocket::getSocket()
{
        // Get an internet domain socket AF_INET
    if(socket1 = socket(AF_INET, SOCK_STREAM,0) == -1)
    {
        perror("socket");
        exit(1);
    }    
}


void TCPSocket::bindSocket()
{
  // Bind to a port on the host
    int myAddressSize = sizeof(myAddress);
    int bindReturnValue = bind(socket1, (struct sockaddr *) &myAddress, AddressSize);
    if (bindReturnValue == -1)
    {
        perror("bind");  // <== Error message generated here
        exit(1);
    }
    printf("Socket for TCP bound to port %d\n", port);    
}

此外，在此之前，我使用此功能对内存块进行了 memset。

void TCPSocket::initializeSocket1()
{
    // Fill tcpSocket struct with 0's

    memset(&myAddress, '\0', sizeof(myAddress));
    myAddress.sin_family = AF_INET;
    myAddress.sin_addr.s_addr = INADDR_ANY;
   // Conver PORT to big-endian if necessary
    myAddress.sin_port = htons(this->port);
}

变量在类的头文件中声明。

public:
    struct sockaddr_in myAddress, clientAddress;

    void buildTCPSocket(int newPort);

private:
    int port;
    int socket1, socket2;

    socklen_t clientAddressLength;

-- 编辑代码现在应该更清楚了。 socket1 在 getSocket() 中初始化。

我看到很多人在 if 中漏掉了括号，但我想我通过声明 myAddressSize 和 bindReturnValue 消除了这个错误。

欢迎任何意见。
谢谢，泰德·S

好的，问题解决了。当然，问题永远不在于你正在寻找的地方，你会发现它。这是更正后的代码。问题出在对 socket() 的调用中缺少一组括号。

void TCPSocket::getSocket()
{
        // Get an internet domain socket AF_INET
    if((socket1 = socket(AF_INET, SOCK_STREAM,0)) == -1)
    {
        perror("socket");
        exit(1);
    }    
}

再次感谢!

Answer 1

我几乎可以向您保证您会收到该错误，因为您从未初始化 socket1。

通常你必须做这样的事情:

 int socket1 = socket(AF_INET, SOCK_STREAM, 0);
 bind(socket1, ...);

我在那里的任何地方都没有看到任何用于设置 socket1 的代码。毕竟，这就是错误消息告诉您的内容。 socket1 不是套接字，因此失败。

编辑:作为后续，这是我尽量避免使用语法的原因之一

if ((foo = bar()) == ERROR)
{
   // handle me
}

而是坚持:

void TCPSocket::getSocket()
{
        // Get an internet domain socket AF_INET
    socket1 = socket(AF_INET, SOCK_STREAM, 0);
    if (socket == -1)
    {
        perror("socket");
        exit(1);
    }    
}