Question

我已经完成了一个简单的分叉程序，其中一个进程使用mq_send和mq_receive向其父进程发送一个chars数组，并输出反向字符串。但是，出于某种原因，当我有时输入要发送的字符串时，程序的接收端已经添加了字符。更重要的是，这似乎是随机的，因为再次发送相同的字符串将提供正确的输出。

这是整个代码，以及之后的示例输出。

#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:

输出中的两个整数值是我用来检查长度的值，第一个是发送字符串的strlen，第二个是接收字符串的strlen。

我确保在初始化和printf之后立即清空字符串，所以我不知道这些神秘字符来自哪里。

Answer 1

在 void serverfunction（）和 void clientFunction（）

替换memset(str1,0,strlen(str1))。

memset(str1,0,sizeof(str1))。

sizeof将给出数组的总大小

现在可能你不会得到垃圾数据。

Answer 2

正如您所看到的，第一次收到的字符似乎比发送的字符多四个。原因是字符串str1不是NULL终止，因为 memset初始化字符数组str1。尝试将其更改为：

memset(str1, 0, MAX_STR_LEN);

并且您的程序应该按预期工作。

第二个想法，如果你正确地发送你的字符串，你根本不需要memset。您的gets会在'\0'的末尾追加str。但是len(str)不会算这个。这意味着您只需发送真实字符，而不是最终'\0'。

因此，有两种更好的方法可以解决您的问题：

另一方在收到字符串后附加最终'\0'。您可以通过添加以下行来执行此操作：
```
str1[receive] = '\0';
```
在mq_receive。
也会发送最终的'\0'。您可以在len+1。{/ p>

mq_send

通过这两种修复，您可以摆脱代码中的memset次调用。

在为POSIX发送消息队列时接收额外的字符

2 个答案: