selectを使った文がうまくいきません

質問

C言語でselectを使ってチャットサーバを実現したいのですが、うまくいきません。

以下にメッセージの受信と新しいクライアントの受付を処理するループのコードを示します。

以下のコードのif(select(FD_SETSIZE,&rfds,NULL,NULL,&tv)>0) {
...
}

の分岐に入って1回目の処理を行った後、以降のループでメッセージを送ってもこの分岐に入らなくなります。

どなたか原因がわかるかたよろしくお願いします。

while(1){
FD_ZERO(&rfds); /* rfds を空集合に初期化*/
FD_SET(sock,&rfds); /* 接続要求を待つソケット*/
/* クライアントを受け付けたソケット*/
/* 監視する待ち時間を1 秒に設定*/
tv.tv_sec = 1;
tv.tv_usec = 0;
/* 標準入力とソケットからの受信を同時に監視する*/
max = sock;
for(i = 0;i = k ; i++){
if(max > csock[i]){
max = csock[i];
}
}
if(select(FD_SETSIZE,&rfds,NULL,NULL,&tv)>0) {
/* s3 */

if(FD_ISSET(sock,&rfds)){
/* s4 */
/* クライアントの受付*/
clen = sizeof(clt);
if ( ( csock[k] = accept(sock,(struct sockaddr *)&clt,&clen) ) <0 ) {
perror("accept");
exit(2);
}
FD_SET(csock[k],&rfds);
if(k < MAXCLIENTS){

strcpy(jusin,"REQUEST ACCEPTED\n");
write(csock[k],jusin,strlen(jusin));

/* s5 */
read(csock[k],nbuf,sizeof(nbuf));
check = 0;
for(i = 0;i < 5;i++){
for(j = 0; j < 100;j++){
if(name[i][j] != nbuf[j]){
check = 1;
break;
}
if(nbuf[j] == '\n'){
break;
}
}
}
if(check != 1){
strcpy(userkyohi,"USERNAME REJECTED\n");
write(csock[k],userkyohi,strlen(userkyohi));
write(csock[k],nbuf,strlen(nbuf));
close(csock[k]);
}
else{
i = 0;
while(rbuf[i] != '\n'){
name[k][i] = nbuf[i];
i++;
}
name[k][i]= '\n';
strcpy(toroku,"USERNAME REGISTERED\n");
write(csock[k],toroku,sizeof(toroku));
k++;
}
}else{
strcpy(kyohi,"REQUEST REJECTED\n");
write(csock[k],kyohi,strlen(kyohi));
close(csock[k]);
}

}

for(i = 0;i < k;i++){

if(FD_ISSET(csock[i],&rfds)){
read(csock[i],mbuf,sizeof(mbuf));

if(mbuf[0] == 'E'&& mbuf[1]=='O' && mbuf[2] == 'F'){
/* s7 */
for(;i == k ; i++){
csock[i]=csock[i+1];
memset(name[i],0,sizeof(name[i]));
j = 0;
while(name[i+1][j] != '\n'){
name[i][j] = name[i+1][j];
}
}
close(csock[k]);
k--;
}
else{
memset(msg, 0, sizeof(msg));
j = 0;

while(name[i][j] != '\n'){
msg[j] = name[i][j];
j++;
}

msg[j] = ' ';j++;
msg[j] = '>';j++;
l = 0;

while(mbuf[l]!='\0'){
msg[j]= mbuf[l];
j++;
l++;
}
msg[j] = '\0';

for(t = 0; t < k;t++){

write(csock[t],msg,sizeof(msg));
}
}
}
}
}
}

回答No.4

trapezium · ベストアンサー

全体的な話として、どこかでブロックしてる可能性。
非ブロッキングならread,write,send,recvすべてでエラーチェックとフォローが必須。
クライアントとのプロトコルで、どうしても時間がかかる処理を実装するならサーバ側もfork()する。
場合によってはwrite処理もselect()でチェックして、書き込むデータはqueueから読み出す。

回答No.3

trapezium

ループの頭で
>FD_ZERO(&rfds);
してて、すべてのfdをFD_SET()してますか？
accept()した後のfdはどこで？

補足

確かに、セットし忘れていました；

/* クライアントを受け付けたソケット*/
for(i=0;i < k;i++){
FD_SET(csock[i],&rfds);
}

と書き加えました。
ありがとうございました。

ただ、これが原因ではないようです；

回答No.2

asuncion

＞変数max自体使うのをやめたのに消すの忘れてました

肝心なところですから、現在お手元にあるコードを載せてください。

補足

以前までif(select(max,&rfds,NULL,NULL,&tv)>0)とかいていたのを

if(select(FD_SETSIZE,&rfds,NULL,NULL,&tv)>0) と書き直してmaxを使わなくなったので

max = sock;
for(i = 0;i = k ; i++){
if(max > csock[i]){
max = csock[i];
}
}

このfor文自体に意味がなくなったということです。

回答No.1

asuncion

本題と関係があるかどうかはわかりませんが、

＞for(i = 0;i = k ; i++){

このfor文は思ったとおりに動いていますか？

補足

i == k; ですね；
というか変数max自体使うのをやめたのに消すの忘れてました

汚いコードで申し訳ないです；