Question

//server side
void* s2(void *arg){
    info *s = (info*)arg;
    char buffer[MAXS];
    int k;
    sockaddr_in addr;
    socklen_t aSize = sizeof(sockaddr_in);
    int sfd = accept(s->fd,(sockaddr*)&addr,(socklen_t*)&aSize);
    if(sfd<0){
        s->current--;
        pthread_exit(0);
    }
    while(1){
        k = recv(sfd,buffer,MAXS,0);
        cout<<buffer<<"\n";
        //1. k ==0 socket has been closed by client 
        //2. k==-1 error in recv 
        //3. recv quit
        if((k==-1)||(!strncmp(buffer,"quit",4))||(k==0))break; 
        sprintf(buffer,"%d\n",(int)strlen(buffer)); //Convert length to  string using sprintf()
        send(sfd,buffer,strlen(buffer),0); //send buffer to client 
    }
    close(sfd);
    if(s->limit==s->current)
    FD_SET(s->fd,&sfds);
    s->current--; //decreament the client number 
    pthread_exit(0);
}

//client side
1. send(sockfd,"sadhdag",8,0);
2. send(sockfd,"ss",3,0);

首次调用recv。

时，服务器recv sadhdag

第二次调用recv服务器recv

ss
dag

服务器端：

函数s2由一个线程运行，并且传递参数，在那里接受套接字信息连接，并且在新接受的客户端上调用send和recv。

为什么会这样？或者如何避免这种情况？

Answer 1

您忽略recv()返回的计数。与评论中的建议相反，在memset()之前将recv()缓冲区设置为零是必要的，但必须仅在之后使用该缓冲区达到那个数量。例如：

printf("%s", buffer);

错了，

printf("%.*s", count, buffer);

是对的。

NB

if((k==-1)||(!strncmp(buffer,"quit",4))||(k==0))break;

也不正确。它应该是

if((k==-1)||(k==0)||(k >= 4 && !strncmp(buffer,"quit",4))) break;

除非k为正，否则查看缓冲区是无效的，除非，否则比较其中的四个字符是无效的。其中有四个字符。

Answer 2

您忘记实际实现任何类型的协议或应用程序级逻辑。总结一下：

    k = recv(sfd,buffer,MAXS,0);
    cout<<buffer<<"\n";

此代码建议您使用换行符终止消息：

    sprintf(buffer,"%d\n",(int)strlen(buffer)); //Convert length to  string using sprintf()
    send(sfd,buffer,strlen(buffer),0); //send buffer to client

如果是，那么在另一端解析该代码的代码在哪里？你扔掉了返回的长度，所以你甚至不知道要检查换行符的字符。

TCP是一种字节流协议，不保留应用程序级别的消息边界。如果要使用它来发送字符串或其他应用程序级消息，则必须指定并实现应用程序级协议才能执行此操作。

您调用recv来获取原始TCP数据，然后将其输出，就像它是一个字符串一样。如果您要定义并实现通过TCP发送和接收字符串的协议，则必须实际执行此操作。你不能想象它是通过魔法发生的。

如果您希望将换行终止的消息接收到C风格的字符串中，您当然可以这样做。但你必须实际编写代码才能完成它。

Berkeley Socket：recv系统调用

2 个答案: