Question

我想在UDP中为4个人做一个聊天室。这是代码：

<code>
#include<stdio.h>
#include<sys/socket.h>
#include<sys/types.h>
#include<arpa/inet.h>
#include<unistd.h>
#define PORT 9999
#define SIZE 1024
int c;
int i=0;
int k=0;
char name[4][20];
char ip[4][16];
FILE * txt;
struct sockaddr_in seraddr,cliaddr[3],getcliaddr[3];

void gettxt()
{
    txt=fopen("ip.txt","r");
    for(k=0;k<4;k++)
    {
        c=fgetc(txt);
        while(c!=' ')
        {
            name[k][i]=(char)c;
            i++;
            c=fgetc(txt);
        }
        name[k][i]='\0';
        i=0;
        c=fgetc(txt);
        while(c<'1'||c>'3')
            c=fgetc(txt);
        while(c!='\n')
        {
            ip[k][i]=c;
            c=fgetc(txt);
            i++;
        }
        ip[k][i]='\0';
        i=0;
    }
    for(k=0;k<4;k++)
        printf("<%s>%s\n",name[k],ip[k]);
}

int compare(struct sockaddr_in whichcli)
{
    int w=1;
    for(w=1;w<4;w++)
    {
        if(whichcli.sin_addr.s_addr==cliaddr[w].sin_addr.s_addr)
            break;
    }
    return w;
}

int main()
{
    int com;//compare return value (just the "k")
    int qq;
    int ret;
    int ser;
    int maxsock;
    char bufrecv[SIZE];
    char bufsend[SIZE];
    socklen_t clilen=sizeof(cliaddr[1]);//the same wa ~~I guess

    gettxt();

    fd_set readfds;

    seraddr.sin_family=AF_INET;
    seraddr.sin_addr.s_addr=inet_addr(ip[0]);
    seraddr.sin_port=htons(PORT);

    for(qq=1;qq<4;qq++)
    {
        cliaddr[qq].sin_family=AF_INET;
        cliaddr[qq].sin_addr.s_addr=inet_addr(ip[qq]);
        cliaddr[qq].sin_port=htons(PORT);
    }
    ser=socket(AF_INET,SOCK_DGRAM,0);
    bind(ser,(struct sockaddr*)&seraddr,sizeof(seraddr));

    if(STDIN_FILENO>ser)
        maxsock=STDIN_FILENO;
    else 
        maxsock=ser;

    while(1)
    {
        FD_ZERO(&readfds);  
        FD_SET(STDIN_FILENO,&readfds);
        FD_SET(ser,&readfds);
        ret=select(maxsock+1,&readfds,NULL,NULL,0); 
        if(ret>0)
        {
            if(FD_ISSET(STDIN_FILENO,&readfds))
            {
                fgets(bufsend,SIZE,stdin);
                for(qq=1;qq<4;qq++)
                    sendto(ser,bufsend,SIZE,0,(struct sockaddr*)&cliaddr[qq],clilen);
            }
            if(FD_ISSET(ser,&readfds))
            {
                for(qq=1;qq<4;qq++)
                {
                    recvfrom(ser,bufrecv,SIZE,0,(struct sockaddr*)&getcliaddr[qq],&clilen);
                    com=compare(getcliaddr[qq]);//
                    printf("<%s>%s\n",name[com],bufrecv);
                }
            }
        }
    }
    return 0;
}
</code>

文件“ip.txt”只是名称-IP文件，如下所示：

<txt>
I 192.168.1.2
Sun 192.168.1.4
Jerry 192.168.1.5
Peter 192.168.1.6
</txt>

第一个包含我自己的信息，以下3个是其他人的信息。但是当我只用一个人运行程序时，首先，我们可以通过非阻塞来互相聊天。经过几句话，它没有奏效。我跑了gcc，我猜想当“sendto”缓冲区已满时，它阻止了另一个“recvfrom”buf。我写的程序是4个人，但我只用一个人运行它，另外两个不能从它中恢复（仍然是“recvfrom”中的缓冲区我猜，我是对的吗？）。所以“sendto”缓冲区已满并被阻止。这就是我的想法，我是对的吗？如果是真的，怎么解决呢？我的意思是如何定期清理“sendto”缓冲区？还是有其他方式？非常感谢~~ ;-)

Answer 1

我认为问题在于您的逻辑，当客户向您发送消息，您循环通过所有客户并致电recvfrom时，您应该每次只调用recvfrom一次select回报。

if(FD_ISSET(ser,&readfds))
{    
    struct sockaddr_in src_addr;
    socklen_t addrlen = sizeof(src_addr);
    recvfrom(ser, bufrecv, SIZE, 0, (struct sockaddr*)&src_addr, &addrlen);
    com=compare(src_addr);//
    printf("<%s>%s\n",name[com],bufrecv);    
}

修改您似乎在getcliaddr中使用recvfrom作为src_addr，这意味着每次您从客户端收到消息时，覆盖另一个客户端的地址，对于一个客户端来说这不是问题，但对于多个客户端，您可以用发件人的地址覆盖第一个客户端，如果再次调用recvfrom，它会阻止，因为您认为它正在等待第一个客户端实际上正在等待第二个。

Answer 2

你误解了src_addr参数对recvfrom（2）的意义：

ssize_t recvfrom(int sockfd, void *buf, size_t len, int flags,
                 struct sockaddr *src_addr, socklen_t *addrlen);

src_addr是来自此函数的OUTPUT，而不是输入。它应该指向一个空的struct sockaddr，它将填入收到的数据包的源地址。这意味着你不能选择一个特定的客户端来接收一个数据包（你真的不想，因为你不知道接下来会打字的人）。相反，你只是收到一个数据包，然后弄清楚它是谁。

当缓冲区已满时sendto（）被阻塞，如何解决？

2 个答案: