c - 通过管道将字符从一个程序发送到另一个程序不起作用

Question

我正在尝试运行执行以下操作的程序:

1. fork 一个子进程，它将启动另一个已编译的 c 程序-抽出
2. 然后父进程等待用户从键盘输入
3. 然后输入通过管道传递给 draw.out 程序
4. draw.out 从管道读取并使用输入

这是两个程序，draw.out 工作正常(俄罗斯方 block 游戏)但是没有从管道获取输入，我的错误是什么？

文件 1:

#include <stdio.h>
#include <unistd.h>
#include <sys/types.h>
#include <signal.h>
#include <sys/wait.h>
#include <errno.h>
#include <stdlib.h>
#include <curses.h>
#include <string.h>


int main(void)
{
    int     fd[2], childpid;
    //pid_t   childpid;
    char    input = 'o';
    char    readbuffer[80];

    pipe(fd);

    if ((childpid = fork()) == -1)
    {
        perror("fork");
        exit(1);
    }

    if (childpid == 0)
    {
        close(0);
        dup(fd[0]);
        execl("draw.out", NULL);


    }
    else
    {
        close(fd[0]);
        while (input != 'q')
        {
            input = getch();
            if (input == 'a' || input == 's' || input == 'd' || input == 'w' || input == 'q')
            {
                write(fd[1], &input, 1);
                kill(getpid() + 1, SIGUSR2);
            }

        }
        exit(0);

    }

    return 1;
}

文件 2:

#include <signal.h>
#include <stdio.h>
#include <sys/types.h>
#include <unistd.h>
#include <string.h>
#include <stdlib.h>



void print_screen(int a, int b, int c, int t, int flat);
void my_handler(int signum);

int a = 9, b = 10, c = 11, t = 0, flat = 1;
int newfd = 0;

int main(void)
{


    if (signal(SIGUSR2, my_handler) == SIG_ERR)
    {
        printf("Pærent: Unable to create handler for SIGUSR2\n");
    }
    while (t < 20)
    {
        system("clear");
        print_screen(a, b, c, t, flat);
        sleep(1);
        t++;
    }



}

void my_handler(int signum)
{
    if (signum == SIGUSR2)
    {
        printf("signal arrived");
        char input;
        fgets(&input, sizeof(input), stdin);
        switch (input)
        {
        case 'a':
            if (a > 1)
            {
                a--;
                b--;
                c--;
            }
            break;
        case 's':
            t++;
            break;
        case 'd':
            if (c < 18)
            {
                a--;
                b--;
                c--;
            }
        case 'w':
            if (flat == 1)
            {
                flat = 0;
            }
            else
            {
                if (b<18 && b>1)
                {
                    flat = 1;
                }
            }
            break;
        case 'q':
            exit(0);

        }
    }
}


void print_screen(int a, int b, int c, int t, int flat)
{
    char str[420] = "\n";
    int i, j;
    for (i = 0; i < 20; i++)
    {
        for (j = 0; j < 20; j++)
        {
            if (i != 19 && !(j>0 && j < 19))
            {
                strcat(str, "*");
            }
            else if (i == 19)
            {
                strcat(str, "*");
            }
            else
            {

                if (flat == 1 && i == t && (j == a || j == b || j == c))
                {
                    strcat(str, "-");
                }
                else if (flat == 0 && j == b && (i == t || i == t - 1 || i == t + 1))
                {
                    strcat(str, "-");
                }
                else
                {
                    strcat(str, " ");
                }
            }
        }
        strcat(str, "\n");
    }
    puts(str);
}

Answer 1

首先:您不能使用fgets 读取一个字节。来自 man fgets:

fgets() reads in at most one less than size characters from stream

由于 sizeof(input) 为 1，“比 size 小一”为 0。因此 fets 不会读取任何内容。例如，将其更改为 fgetc。

第二:kill(getpid() + 1, SIGUSR2); 语句显然是错误的，因为你假设你的 child 将拥有你进程的 pid + 1。这是错误的假设，并且有不需要这样的构造，因为您已经正确设置了 childpid 变量。

第三:您没有使用 initscr() 初始化 ncurses，因此 getch() 将不起作用。因为这也会重新配置您的终端，所以在每行末尾添加“\n\r”而不是“\n”。并且不要忘记在退出前调用 endwin() 取消初始化 ncruses。

第四:在信号处理程序中做任何比获取/设置简单变量更复杂的事情通常是一个非常糟糕的主意。您应该使用 select 函数在主循环中轮询输入，如下所示:

struct timeval time;
time.tv_sec = 0;
time.tv_usec = 0;
fd_set set;
FD_ZERO(&set);
FD_SET(0, &set);
if (select(1, &set, NULL, NULL, &time) > 0) {
    handle_input();
}

第五:你在你的案例中缺少一个 break 语句，而且 d 和 a 键都是递减变量，而 d 我猜应该是递增的。

这是有效的代码:

#include <signal.h>
#include <stdio.h>
#include <sys/types.h>
#include <unistd.h>
#include <string.h>
#include <stdlib.h>

void print_screen(int a, int b, int c, int t, int flat);
void handle_input(void);

int a = 9, b = 10, c = 11, t = 0, flat = 1;
int newfd = 0;

int main(void) {
    int t = 0;
    while (t < 20) {
        system("clear");
        print_screen(a, b, c, t, flat);
        sleep(1);
        struct timeval time;
        time.tv_sec = 0;
        time.tv_usec = 0;
        fd_set set;
        FD_ZERO(&set);
        FD_SET(0, &set);
        if (select(1, &set, NULL, NULL, &time) > 0) {
            handle_input();
        }
        t++;
    }
    return 0;
}

void handle_input() {
    char input;
    input = fgetc(stdin);
    switch (input) {
    case 'a':
        printf("a\n");
        if (a > 1) {
            a--;
            b--;
            c--;
        }
        break;
    case 's':
        t++;
        break;
    case 'd':
        if (c < 18) {
            a++;
            b++;
            c++;
        }
        break;
    case 'w':
        if (flat == 1) {
            flat = 0;
        } else {
            if (b<18 && b>1) {
                flat = 1;
            }
        }
        break;
    case 'q':
        exit(0);
    }
}


void print_screen(int a, int b, int c, int t, int flat) {
    char str[4200] = "\n";
    int i, j;
    for (i = 0; i < 20; i++) {
    for (j = 0; j < 20; j++) {
        if (i != 19 && !(j>0 && j < 19)) {
            strcat(str, "*");
        }
        else if (i == 19) {
            strcat(str, "*");
        } else {
            if (flat == 1 && i == t && (j == a || j == b || j == c)) {
                strcat(str, "-");
            } else if (flat == 0 && j == b && (i == t || i == t - 1 || i == t + 1)) {
                strcat(str, "-");
            } else {
                strcat(str, " ");
            }
        }
    }
    strcat(str, "\n\r");
    }
    puts(str);
}

第二个文件:

#include <stdio.h>
#include <unistd.h>
#include <sys/types.h>
#include <signal.h>
#include <sys/wait.h>
#include <errno.h>
#include <stdlib.h>
#include <string.h>
#include <ncurses.h>

int main(void) {
    int     fd[2];
    pid_t   childpid;
    char    input = 'o';

    pipe(fd);

    if ((childpid = fork()) == -1) {
        perror("fork");
        exit(1);
    }

    if (childpid == 0) {
        close(0);
        dup(fd[0]);
        execl("draw.out", "draw.out", NULL);
    } else {
        close(fd[0]);
        initscr();
        while (input != 'q') {
            read(0, &input, 1);
            if (input == 'a' || input == 's' || input == 'd' || input == 'w' || input == 'q') {
                write(fd[1], &input, 1);
            }
        }
        endwin();
        exit(0);
    }

    return 0;
}