我正在尝试创建一个连接到多个客户端的服务器。基本上,我不想做的是每当有人连接到服务器时向所有客户端发送提示。但是,send()命令不会发送数据。:(我想知道代码有什么问题。
int main(int argc, char **argv){
int listenfd,connfd;
struct sockaddr_in servaddr,cliaddr;
socklen_t len = sizeof(cliaddr);
char cli_ip[32];
char mesg[1024];
char broadcast[500];
int maxconn,counter = 0;
// struct addrinfo hints, *ai, *p;
socklen_t addrlen;
fd_set master;
fd_set read_fds;
int fdmax;
int yes = 1;
int i, j;
char remoteIP[INET6_ADDRSTRLEN];
FD_ZERO(&master);
FD_ZERO(&read_fds);
listenfd = socket(PF_INET,SOCK_STREAM,0);
bzero(&servaddr, sizeof(servaddr));
servaddr.sin_family = AF_INET;
servaddr.sin_addr.s_addr = htonl(INADDR_ANY);
servaddr.sin_port = htons(54321);
setsockopt(listenfd, SOL_SOCKET,SO_REUSEADDR, &yes, sizeof(int));
if(bind(listenfd, (struct sockaddr*) &servaddr, sizeof(servaddr)) < 0 ){
perror(NULL);
exit(1);
}
maxconn = 2;
if((listen(listenfd,5)) == -1){
perror("listen");
exit(2);
}
FD_SET(listenfd, &master);
fdmax = listenfd;
while(1){
read_fds = master;
if((select(fdmax + 1,&read_fds, NULL, NULL, NULL )) == -1){
perror("select");
exit(3);
}
for(i = 0; i <= fdmax; i++){
if(FD_ISSET(i, &master)){
if(i = listenfd){
if(counter<= maxconn){
addrlen = sizeof(cliaddr);
connfd = accept(listenfd, (struct sockaddr *)&cliaddr, &addrlen);
counter++;
inet_ntop(AF_INET,(struct in_addr *)&cliaddr.sin_addr, cli_ip, sizeof(cli_ip));
if(connfd == -1){
perror("accept");
exit(4);
}else{
FD_SET(connfd, &master);
if(connfd > fdmax)
fdmax = connfd;
sprintf(broadcast,"Player %d from %s has connected.\n",connfd, cli_ip);
for(j = i+ 1; j <= connfd; j++){
if(FD_ISSET(j, &master)){
/*somehow this doesn't work :(*/ if((send(j, broadcast, sizeof(broadcast), 0))){
perror("send");
exit(5);
}
}
}
printf("%s,%d\n",broadcast,counter);
printf("%d,%d\n",i,fdmax);
break;
}
}else{
printf("server full\n");
close(connfd);
break;
}
}
}
}
}
return 0;
}
正如您所看到的,此代码与Beej指南中的代码略有不同。那么为什么客户端不接收广播?
答案 0 :(得分:0)
我不知道这是否是您的确切问题,但这不会像您期望的那样有效。
if ((send(j, broadcast, sizeof(broadcast), 0)))
{
perror("send");
exit(5);
}
send
将返回发送的字节数或错误的-1,对于0以外的任何值,这将被评估为true,因此您至少应该看到{{1}的输出即使它具有误导性,因为它只是errno中的一些随机值,因为perror
实际上没有失败,接着是程序终止。
并且您不希望发送send
,这将是整个500字节,其中大多数是随机垃圾,但是sizeof(broadcast)
。