以下是我的C / S代码和输出仅用于测试
我有麻烦。
很明显,我只是通过客户端发送两次。但是,服务器只有3条消息。而第二个似乎是一个NULL。
#define MP_MAXLINE 4096
/* client */
int main(int ac, char *av[])
{
char buf[MP_MAXLINE];
int clifd;
struct sockaddr_in cliaddr;
if((clifd = socket(AF_INET, SOCK_STREAM, 0)) == -1)
perror("socket error");
memset(&cliaddr, 0, sizeof(struct sockaddr_in));
cliaddr.sin_family = AF_INET;
cliaddr.sin_port = htons(atoi(av[2]));
inet_pton(AF_INET, av[1], &cliaddr.sin_addr);
if(connect(clifd, (struct sockaddr *)&cliaddr, sizeof(cliaddr)))
perror("can not connect to server");
printf("enter data to send: ");
while(fgets(buf, MP_MAXLINE, stdin) != NULL){
write(clifd, buf, sizeof(buf));
printf("enter data to send: ");
}
}
/* server */
static void cli_service(int clifd){
int lclifd, rcv_cnt = 0;
char data[MP_MAXLINE];
for(;;){
if(read(clifd, data, sizeof(data)) == 0)
exit(1);
rcv_cnt++;
printf("cnt: %d, data: %s", rcv_cnt, data);
}
}
int main(int ac, char *av[])
{
...
for(;;){
clifd = accept(svrfd, (struct sockaddr *)&cliaddr, &clisocklen);
if((clipid = fork()) < 0){
printf("fork error\n");
continue;
}
else if(clipid == 0){
close(svrfd);
cli_service(clifd);
}
close(clifd);
continue;
}
}
客户的输出在这里:
输入要发送的数据:1111
输入要发送的数据:2222
输入要发送的数据:
服务器的输出在这里:
cnt:1,数据:1111
cnt:2,数据:cnt:3,数据:2222
答案 0 :(得分:3)
发生的事情是您即将发现TCP是一个面向流的协议,这意味着它可以保证您发送的字节将按正确顺序传递,但映射单个write()
/ read()
调用的字节数是任意的 - 也就是说,每次调用read()
时写入接收器数组的字节数肯定是不是保证与发送程序先前传递给write()
调用的字节数相同。
因此,在您的客户端中,每次调用write()
都会发送4096个字节,其中前几个将包含用户输入的以0结尾的文本字符串,其余部分将未初始化数据(即它可能是任何东西,因为你从未写过数组的那一部分)。
然后在您的服务器中,您正在调用read()
,它读取了一些字节数,但您不知道它读入了多少字节,因为您所做的就是检查read()
是否返回0或不。所以你不知道read()是将4096字节的数据放入你的data
数组还是一些较小的字节数;如果它是一个较小的数字,那么您的接收代码现在与您的发送代码不同步,因为后续调用read()
可能会将未初始化的数据写入data
的开头数组,以及用户的下一个字符串位于数组中间的某个位置。
当然,这不是有用的行为。为了解决这个问题,您需要做的是密切注意read()
返回的值;如果它大于零但小于sizeof(data)
,你将需要以某种方式处理它。处理它的一种方法是保持data
数组中当前有效字节数的计数,并始终read(clifd, data+numValidBytes, sizeof(data)-numValidBytes)
直到(numValidBytes == sizeof(data))
...然后才解析数据和将numValidBytes
设置为零。另一个更简单的选择是调用recv()
而不是read()
,并在最后一个参数中设置MSG_WAITALL - 在这种情况下,recv()
将保证在读取全部sizeof(data)
之前不返回{ {1}}字节,然后您不必自己处理部分读取。