2016-09-09 11 views
0

私は、ソケットを使用して別のマシンと通信するC言語の非常に基本的なチャットクライアントを作成しようとしています。関連するコードの簡単なスニペットがあります。私はしているよ何STDINと着信ソケットでのselect()

while(1) 
    { 
    FD_ZERO(&readfds); 
    FD_ZERO(&writefds); 

    FD_SET(STDIN, &writefds); 
    FD_SET(connectFD, &readfds); 

    select(connectFD+1, &readfds, &writefds, NULL, NULL); 

    if(FD_ISSET(connectFD, &readfds) != 0) 
    { 
     char buf[1024]; 
     memset(buf, 0, sizeof(buf)); 
     int lastBit; 

     lastBit = recv(connectFD, buf, sizeof(buf), 0); 
     if (lastBit > 0 && lastBit < 1024) 
     { 
     buf[lastBit] = '\0'; 
     } 
     else 
     { 
     close(connectFD); 
     break; 
     } 
     printf("%s\n", buf); 
    } 

    else if (FD_ISSET(STDIN, &writefds)) 
    { 
     char msg[1024]; 
     memset(msg, 0, sizeof(msg)); 
     read(STDIN, msg, sizeof(msg)); 
    } 
    } 
} 

は、私はEnterキーを押した後、彼らが到着するとすぐに処理される着信メッセージを持っている、とだけ送信メッセージが送られてきているが、私は代わりに、エンターキーを押した後、私が持っていることは、今だけの着信データを処理しますすぐに。私はそれが読み込みをブロックする呼び出しであると仮定し、バッファに任意のデータがあるときにselectが返されるのは、改行(読み返しが返ってくるとき)だけではなく、そうでなければこれを処理する方法を知らないからです。私に正しい道を導くためのアドバイスやヒント?

ありがとう!あなたは&readfdsなく&writefdsにSTDINを追加する必要がありますので、STDINから読みたい

+0

1,024バイトを実際に読み取るとエラーが発生する場合は、1,024バイトを読み込もうとします。 –

+0

これは、バッファオーバーフローの非常に基本的な(そして間違った)チェックに過ぎませんでした。私はプログラムの実際のロジックが働いたら後で修正するつもりです。 @DavidSchwartz – user3502205

答えて

4
FD_SET(STDIN, &writefds); 

。 STDINへの書き込みはほとんど常に可能ですので、あなたのコードはSTDINへの書き込みが可能であるという情報を効果的に得ますが、STDINから読み込みを試み、読み込みが実際に可能になるまでそこにハングします。

+0

ああ、私は何か基本的なものが欠けていることを知っていた。私はSTDINからの入力の結果として(ソケットに)書いていると思っていたので、writefdsセットに入る必要がありました。すべてが今、完璧に動作します。どうもありがとうございます! – user3502205

関連する問題