2

1 つのプロセスが mq_send と mq_receive を使用して親プロセスに文字の配列を送信し、反転した文字列を出力する単純な分岐プログラムをほぼ完成させました。しかし、送信する文字列を入力すると、なぜかプログラムの受信側で文字が追加されてしまいます。さらに、同じ文字列を再度送信すると適切な出力が得られるため、これはランダムに見えます。

コード全体と、その後の出力例を次に示します。

#include  <stdio.h> /* printf */
#include  <sys/types.h> /* pid_t */
#include <unistd.h> /* get_pid */
#include <stdlib.h>     /* exit, EXIT_FAILURE */
#include <sys/wait.h>   /* wait */
#include <mqueue.h>
#include <errno.h>
#include <fcntl.h>
#include <sys/stat.h>
#include <string.h>

#define MAX_STR_LEN 500

void reverse(char s[]);
void clientFunction(const char *msgqname);
void serverFunction(const char *msgqname);
int main(void)
{
    //A
    char msgQName[] = "/queue";
    //B
    pid_t pid;
    pid = fork();
    if (pid == 0){      
        clientFunction(msgQName);
    }
    else {
        usleep(10L);
        serverFunction(msgQName);
    }
    return(0);

}
void clientFunction(const char *msgqname){
    //C

    mqd_t mqident = mq_open(msgqname, O_WRONLY|O_CREAT ,  0666, NULL);
    //D
    if (mqident == -1){
        printf("Error opening message queue. Exiting program. \n");
        exit(0);
    }
    char str[MAX_STR_LEN];  //Keep the string that has been read from input and written on the pipe
    memset(str,0,strlen(str));  
    while(1){

        printf("Enter an string:");
        gets(str);  //Reads the string from input
        printf("%i\n",strlen(str));
        unsigned len = (unsigned) strlen(str); //Finds the length of the string
        int sent = mq_send(mqident,str,len,0);
        if (sent == -1){
            printf("Error sending message. Exiting program. \n");
            exit (0);
        }
        usleep(10L);
        memset(str,0,strlen(str));
    }

}
void serverFunction(const char *msgqname){
    //F
    mqd_t mqident = mq_open (msgqname,O_RDONLY|O_CREAT , 0666, NULL);

    if (mqident == -1){
        printf("Error opening message queue. Exiting program. \n");
        printf( "Error : %s\n", strerror( errno ) );
        exit(0);
    }
    char str1[MAX_STR_LEN];
    memset(str1,0,strlen(str1));
    struct mq_attr attr;
    while(1){
        //G
        mq_getattr(mqident,&attr);
        //H
        ssize_t receive = mq_receive(mqident,str1,attr.mq_msgsize,0);
        if (receive == -1){
            printf("Error receiving message. Exiting program. \n");
            exit(0);
        }
        printf("%i, %i\n",strlen(str1), attr.mq_msgsize);
        reverse(str1);
        printf("What you wrote in reverse was:%s\n", str1); 
        memset(str1,0,strlen(str1));
    }

}
void reverse(char s[])
{
      int length = strlen(s) ;
      int c, i, j;

      for (i = 0, j = length - 1; i < j; i++, j--)
     {
     c = s[i];
     s[i] = s[j];
     s[j] = c;
      }
}

ターミナルで -o および -lrt を使用して gcc でコンパイルした場合の出力例

Enter an string:tester 1234567890     
17
21
What you wrote in reverse was:1��0987654321 retset
Enter an string:tester 1234567890
17
17
What you wrote in reverse was:0987654321 retset
Enter an string:

出力の 2 つの整数値は、長さを確認するために使用したものです。最初の値は送信文字列の strlen で、2 番目は受信文字列の strlen です。

初期化とprintfの直後に文字列を空にしたので、これらの謎の文字がどこから来ているのかわかりません.

4

2 に答える 2

1

ご覧のとおり、初回は、送信された文字よりも受信された文字の方が 4 つ多いようです。その理由は、文字配列を初期化するstr1ため、文字列が NULL で終了していないためです。次のように変更してみてください。memsetstr1

memset(str1, 0, MAX_STR_LEN);

プログラムは期待どおりに動作するはずです。

memsetよく考えてみると、文字列を適切に送信していれば、 はまったく必要ないはずです。の最後にagetsを追加します。しかし、それはカウントしません。これは、最終的な文字ではなく、実際の文字のみを送信することを意味します。 '\0'strlen(str)'\0'

したがって、問題を解決するためのより良い方法が 2 つあります。

  1. '\0'反対側は、文字列を受け取ると、最後の , を追加します。これを行うには、次の行を追加します。

    str1[receive] = '\0';

    あなたの直後mq_receive。

  2. ファイナル'\0'も配信。len+1で文字を送信することでこれを行うことができますmq_send。

memset両方の修正により、コード内の両方の呼び出しを取り除くことができます。

于 2013-10-03T19:46:44.653 に答える