1

UNIX ドメイン ソケットを介してデータを共有する 2 つの簡単なプログラムをセットアップしました。1 つのプログラムが Queue からデータを読み取り、それを他のアプリケーションに送信します。データが送信される前に、各データの前に長さが 4 バイト追加されます。4 バイト未満の場合、残りのバイトは '^' 記号です。

次に、クライアント アプリケーションは最初の 4 バイトを読み取り、バッファーを適切なサイズに設定してから、残りを読み取ります。私が抱えている問題は、最初のメッセージが完全に送信されることです。その後は 1 回おきに余分なデータが送信されるため、「なんて素敵な日ですか??X??」というようなメッセージが表示されます。そのため、バッファが正しくクリアされていないように感じますが、見つけられないようです。

クライアントコード:

listen(sock, 5);
for (;;) 
{
    msgsock = accept(sock, 0, 0);
    if (msgsock == -1)
        perror("accept");
    else do 
    {
        char buf[4];
        bzero(buf, sizeof(buf));
        if ((rval = read(msgsock, buf, 4)) < 0)
        perror("reading stream message");

        printf("--!%s\n", buf);

        string temp = buf;
        int pos = temp.find("^");
        if(pos != string::npos)
        {
            temp = temp.substr(0, pos);
        }

        int sizeOfString = atoi(temp.c_str());
        cout << "TEMP STRING: " << temp << endl;
        cout << "LENGTH " << sizeOfString << endl;
        char feedWord[sizeOfString];
        bzero(feedWord, sizeof(feedWord));

        if ((rval = read(msgsock, feedWord, sizeOfString)) < 0)
              perror("reading stream message");

          else if (rval == 0)
              printf("Ending connection\n");
          else
              printf("-->%s\n", feedWord);
              bzero(feedWord, sizeof(feedWord));
              sizeOfString = 0;
              temp.clear();
      } 
        while (rval > 0);
      close(msgsock);
  }
  close(sock);
  unlink(NAME);

サーバーコード

                pthread_mutex_lock(&mylock);
                string s;
                s.clear();
                s = dataQueue.front();
                dataQueue.pop();
                pthread_mutex_unlock(&mylock);

                int sizeOfString = strlen(s.c_str());
                char sizeofStringBuffer[10];

                sprintf(sizeofStringBuffer, "%i", sizeOfString);
                string actualString = sizeofStringBuffer;
                int tempSize = strlen(sizeofStringBuffer);

                int remainder = 4 - tempSize;
                int x;
                for(x =0; x < remainder; x++)
                {
                    actualString = actualString + "^";
                }

                cout << "LENGTH OF ACTUAL STRING: " << sizeOfString << endl;

                actualString = actualString + s;

                cout << "************************" << actualString << endl;
                int length = strlen(actualString.c_str());

                char finalString[length];
                bzero(finalString, sizeof(finalString));
                strcpy(finalString, actualString.c_str());

                           if (write(sock, finalString, length) < 0)
                           perror("writing on stream socket");
4

3 に答える 3

2

パケットの長さを ' ' でパディングするよりも^、次のようにする方がはるかに良いでしょう:

snprintf(sizeofStringBuffer, 5, "%04d", sizeOfString);

値が 0 で埋められるようにします。受信者コードで '^' 文字を解析する必要はありません。

デバッグ コードも編集してください。write()現在のコードには 1 つしかなく、プロトコルの説明と一致しません。

理想的には、送信ルーチンを独自の関数に分割します。を利用しwritev()て、「長さ」フィールドを保持する文字列を実際のデータを保持するバッファーと合体させ、それらを単一のアトミックとして送信することもできますwrite()。

テストされていないコードは次のとおりです。

int write_message(int s, std::string msg)
{
     struct iovec iov[2];
     char hdr[5];

     char *cmsg = msg.c_str();
     int len = msg.length();

     snprintf(hdr, 5, "%04d", len);  // nb: assumes len <= 9999;

     iov[0].iov_base = hdr;
     iov[0].iov_len = 4;

     iov[1].iov_base = cmsg;
     iov[1].iov_len = len;

     return writev(s, iov, 2);
}
于 2009-05-21T18:11:31.013 に答える
2

両方の戻り値をチェックするだけでwriteなく、短い (要求されたよりも少ない) 書き込み/読み取りについてもチェックする必要があります。また、エラーを出力した後も続行するようです- do an or something そこで。read-1perrorexit(2)

于 2009-05-21T18:12:21.700 に答える