绑定(bind):非套接字的套接字操作。

5

我正在编写一个服务器和客户端,但一直出现“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);    
}

在此之前,我使用了这个函数对内存块进行了清零。

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来消除了这个错误。
欢迎任何意见反馈。 谢谢, Ted 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);
    }    
}

再次感谢!

你将socket1初始化为什么?那就是你的问题所在。 - Mike Bailey
谢谢您的快速回复。我不确定如何再次包含代码,但这是更完整的代码。 void TCPSocket :: buildTCPSocket(int port) { initializeSocket1(); getSocket(); bindSocket(); listenToSocket(); acceptSocket(); //现在您可以通过连接的套接字connectedTCPSocket与已连接的客户端进行send()和recv()操作 } - Ted Spradley
只需点击您的帖子上的“编辑”按钮即可。但是,这并没有什么帮助。我更想知道您实际上在哪里使用socket1。更具体地说,您能否编辑您的问题以包括getSocket()函数? - Mike Bailey
当您插入新的代码片段时,请突出显示所添加的代码,然后按下花括号按钮,以便它正确缩进。否则,您的代码将以未格式化的形式显示在您的问题中。 - Mike Bailey
你还没有展示如何创建/打开套接字 - 使用 socket() 调用。因此,我们无法确定 socket1 是如何初始化的,但系统知道它是什么值(很可能是0,与标准输入相同),它不是一个套接字。并且它正在告诉你这个信息。 - Jonathan Leffler
显示剩余3条评论
1个回答

9
我几乎可以保证,您之所以出现这个错误,是因为您从未初始化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);
    }    
}

2
你说得太对了。在getSocket()中,if(socket1 = socket(AF_INET, SOCK_STREAM,0) == -1)的调用缺少括号设置。我会发布正确的代码以获得正确的格式。非常感谢你们所有人。浪费了这么多时间真是让人恼火。干杯! - Ted Spradley
这正是我不喜欢使用C++该特性的原因。虽然它可以省略一行代码,但我发现将初始化和错误检查分别放在两行更易读且出错几率更低。 - Mike Bailey
谢谢Mike。虽然是一个艰难的教训,但还是学到了。 - Ted Spradley
@Ted:没问题。欢迎来到本网站! - Mike Bailey

网页内容由stack overflow 提供, 点击上面的
可以查看英文原文,
原文链接