1

我正在制作一个不同版本的猜谜游戏。这一次，子进程必须将它的猜测发送给父进程，然后父进程对其进行评估。我认为我做错的是我的孩子只跑了一次，但在找到正确的数字之前无法弄清楚如何猜测。

#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/types.h>
#include <sys/ipc.h>
#include <sys/msg.h>
#include <string.h>
#include <sys/wait.h>
#include <time.h>

#define KEY 19950914
#define FLAG 0666

struct message {
    long mtype;
    int szam;
};

int main()
{
    int number, false=1, guess=0;
    int mqid;
    struct message buf;
    struct msqid_ds statbuff;

    mqid = msgget(KEY, FLAG | IPC_CREAT);

    if (mqid < 0)
            perror("msgget"), exit(EXIT_FAILURE);

    srand(time(NULL));
    number = rand() % 256;

    if (fork() == 0)
    {
            srand(time(NULL));
            buf.mtype = 2;
            buf.szam = rand() % 256;
            msgsnd(mqid, &buf, sizeof(struct message), 0);
            msgctl(mqid, IPC_STAT, &statbuff);

    exit(EXIT_SUCCESS);
    }

    while ( guess != number )
    {
            if (guess > number)
                    printf("Too high!\n");
            else if (guess < number)
                    printf("Too low!\n");

            guess = msgrcv(mqid, &buf, sizeof(struct message), 2, 0);
    }

    printf("Winner! Yes, the answer was %d \n",number);

    wait(NULL);

    exit(EXIT_SUCCESS);
}
4

1 回答 1

2

一种方法是将孩子放入循环中，然后在得到正确答案后删除消息队列，这将导致msgsnd失败并EIDRM退出循环：

#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/types.h>
#include <sys/ipc.h>
#include <sys/msg.h>
#include <string.h>
#include <sys/wait.h>
#include <time.h>

#define FLAG 0666

struct message {
    long mtype;
    int szam;
};

int main()
{
    int number, false=1, guess;
    int mqid;
    struct message buf;

    mqid = msgget(IPC_PRIVATE, FLAG | IPC_CREAT);

    if (mqid < 0)
            perror("msgget"), exit(EXIT_FAILURE);

    srand(time(NULL));
    number = rand() % 256;

    if (fork() == 0)
    {
            buf.mtype = 2;
            int sndres;
            do {
                    buf.szam = rand() % 256;
                    sndres = msgsnd(mqid, &buf, sizeof(struct message), 0);
            } while(sndres == 0);

    exit(EXIT_SUCCESS);
    }

    do {
            msgrcv(mqid, &buf, sizeof(struct message), 2, 0);
            guess = buf.szam;
            if (guess > number)
                    printf("Too high!\n");
            else if (guess < number)
                    printf("Too low!\n");
    } while ( guess != number );

    printf("Winner! Yes, the answer was %d \n",number);

    msgctl(mqid, IPC_RMID, NULL);

    wait(NULL);

    exit(EXIT_SUCCESS);
}

我也在你的程序中修复了一些其他的东西：

  • 我没有使用 fixed KEY，而是将其更改为IPC_PRIVATE，从而避免了键冲突的可能性。由于您没有尝试在其他地方打开相同的队列，因此没有理由使用固定的队列。
  • 我摆脱了statbuff你的IPC_STAT电话。他们没有做任何有用的事情。
  • 我删除了您对 的第二次调用srand。通过将两个如此靠近，两次time(NULL)都是相同的，因此您的子程序将具有相同的随机数状态，因此每次第一次尝试时都会猜对。
  • 成功的返回值msgrcv是消息的大小，它总是相同的（可能是 16）。我更改它以检查实际猜测，在buf.szam.
  • 你的第一次检查guess是在你的第一次检查之前msgrcv，这导致了一个不是来自孩子的虚假猜测。我将您的while循环更改为do-while循环以避免这种情况。

这里还有一些应该修复的东西，但我留给读者作为练习：

  • 摆脱你实际上不使用的所有东西，比如false（顺便说一下，一个可怕的变量名称）
  • 不要对逗号如此“聪明”，如perror("msgget"), exit(EXIT_FAILURE);. 只需使用大括号和分号。
  • 您应该将结果保存fork()到一个变量中，这样您就可以检查它是否为负数，这表明失败。
  • 您传递给msgsnd的msgrcv大小应该是消息结构的第二个成员的大小（即，不包括mtype或紧随其后的填充），而不是整个结构的大小。
  • 您应该检查返回msgrcv以确保它不会失败。
  • 像我一样在一个恒定的循环中运行孩子是最简单的方法，但不一定是最有效或最好的。考虑让父级向子级发送消息，以便它一次只发出一个猜测，而不是尽可能多地填充队列。（即使您确实进行了此更改，您仍然应该让父级在最后删除消息队列，否则它不会消失，直到您重新启动或使用手动清理它ipcrm。）
于 2020-04-28T23:28:43.767 回答