我对这个着名的问题有疑问。 我必须分叉两个通过管道进行通信的子（生产者和消费者）。 第一个子（生产者）必须从stdin读取字符串，将它们发送给必须将它们转换为大写并打印到stdout的第二个子（消费者）。
我写了一段代码，但它不起作用。
（我试着在这里写代码，但也许我不明白如何正确地做。解释我！谢谢你）
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <signal.h>
#include <fcntl.h>
#include <ctype.h>
#include <string.h>
#include <errno.h>
#include <sys/wait.h>
#define MAXC 30
static void signalHandler(int signo)
{
return;
}
pid_t pids[2];
int main()
{
int fd[2], i, nW;
size_t l;
char string[MAXC+1], stringtoup[MAXC+1], tmp[MAXC+1];
signal(SIGUSR1, signalHandler);
if(pipe(fd)==0)
{
pids[0]=fork();
if(pids[0]==0)
{
fprintf(stdout, "PID=%d PRODUCER\n", getpid());
close(fd[0]); //produttore
sleep(3);
fprintf(stdout, "Insert strings (max 30 chars), finish with 'end':\n");
fscanf(stdin, "%s", string);
while(1)
{
if(strcmp(tmp, "end")==0)
break;
l=strlen(string)+1;
sprintf(tmp, "%2lu%s", l, string);
printf("%s\n", tmp);
nW=write(fd[1], tmp, (l+2));
printf("nW(PRODUCER)=%d", nW);
if(nW!=(l+2))
{
perror("wrote not whole string");
exit(1);
}
sleep(5);
kill(pids[1], SIGUSR1);
pause();
fprintf(stdout, "Insert string:\n");
fscanf(stdin, "%s", string);
}
exit(0);
}
pids[1]=fork();
if(pids[1]==0)
{
fprintf(stdout, "PID=%d CONSUMER\n", getpid());
close(fd[0]); //consumer
while(1)
{
pause();
read(fd[0], tmp, MAXC+1);
printf("tmp(CONSUMER)=%s\n", tmp);
sscanf(tmp, "%2lu%s", &l, stringtoup);
printf("lenght string(CONSUMER)=%2lu\n", l);
printf("stringtoup=%s\n", stringtoup);
for(i=0; i<l; i++)
stringtoup[i]=toupper(stringtoup[i]);
fprintf(stdout, "%s\n", stringtoup);
fflush(stdout);
sleep(1);
kill(pids[0], SIGUSR1);
}
exit(0);
}
sleep(4);
for(i=0; i<2; i++)
{
waitpid(pids[i], NULL, 0);
fprintf(stdout, "PID=%d exited\n", pids[i]);
}
}
return(0);
}
编辑：代码已修复
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <signal.h>
#include <fcntl.h>
#include <ctype.h>
#include <string.h>
#include <errno.h>
#include <sys/wait.h>
#define MAXC 30
pid_t pids[2];
int main()
{
int fd[2], i, nW;
size_t l;
char string[MAXC+1], stringtoup[MAXC+1], tmp[MAXC+1];
if(pipe(fd)==0)
{
pids[0]=fork();
if(pids[0]==0)
{
fprintf(stdout, "PID=%d PRODUCER\n", getpid());
close(fd[0]); //produttore
fprintf(stdout, "Insert strings (max 30 chars), finish with 'end':\n");
fscanf(stdin, "%s", string);
while(1)
{
if(strcmp(string, "end")==0)
break;
l=strlen(string)+1;
sprintf(tmp, "%2lu%s", l, string);
nW=write(fd[1], tmp, (l+2));
if(nW!=(l+2))
{
perror("wrote not whole string");
exit(1);
}
sleep(1);
fscanf(stdin, "%s", string);
}
kill(pids[1], SIGINT);
exit(0);
}
pids[1]=fork();
if(pids[1]==0)
{
fprintf(stdout, "PID=%d CONSUMER\n", getpid());
close(fd[1]); //consumer
while(1)
{
read(fd[0], tmp, MAXC+1);
sscanf(tmp, "%2lu%s", &l, stringtoup);
for(i=0; i<l; i++)
stringtoup[i]=toupper(stringtoup[i]);
fprintf(stdout, "%s\n", stringtoup);
fflush(stdout);
sleep(1);
}
exit(0);
}
for(i=0; i<2; i++)
{
waitpid(pids[i], NULL, 0);
fprintf(stdout, "PID=%d exited\n", pids[i]);
}
}
return(0);
}
答案 0 :(得分：2)
通过阅读评论，我想问题已经改变，并且涉及杀戮等待程序。
问题应该是孩子们从分娩时起与父母的记忆毫无关系。
第一个子（生产者）将pids数组填充为：
pids[0] == 0;
pids[1] == undefined;
pids[1]的值永远不会改变，因为这个过程永远不会改变它自己记忆的那部分。
第二个孩子（消费者）将pids数组填充为：
pids[0] == first_child's pid;
pids[1] == 0;
总而言之，只有父母可以杀死第二个孩子（一个pids[1]），而不是第一个孩子。因此，“生产者”用一些你不知道的pid来杀死一个进程（如果不是幸运的话，可能会导致更广泛的系统问题），所以“消费者”永远不会得到它。
当管道上没有编写器时，read()函数返回零（0）。这就是你应该利用的东西，当它发生时就会中断并退出。另一方面，当有作者时read()函数会阻塞，直到有东西要读，所以不需要sleep(1)。
正如我所看到的，当执行杀死该随机进程时程序失败，这就是父进程不打印任何子进程的原因。但是，如果删除kill(...)，请关闭管道的写入端并检查程序运行的read(...)的返回值。
另外，一个好的模式是关闭你不需要的所有fds（即使退出函数会在某个时刻执行），所以在生成所有孩子后父母可以关闭2个fds（它必须关闭写入端才能让读者破解！），读者可以在退出前关闭剩余的fd。下面是需要修复的代码部分。
...
if(pids[0]==0)
{
fprintf(stdout, "PID=%d PRODUCER\n", getpid());
close(fd[0]); //produttore
fprintf(stdout, "Insert strings (max 30 chars), finish with 'end':\n");
fscanf(stdin, "%s", string);
while(1)
{
if(strcmp(string, "end")==0)
break;
l=strlen(string)+1;
sprintf(tmp, "%2lu%s", l, string);
nW=write(fd[1], tmp, (l+2));
if(nW!=(l+2))
{
perror("wrote not whole string"); exit(1);
} sleep(1);
fscanf(stdin, "%s", string);
}
close(fd[1]);
exit(0);
}
pids[1]=fork();
if(pids[1]==0)
{
fprintf(stdout, "PID=%d CONSUMER\n", getpid());
close(fd[1]); //consumer
while(1)
{
if (read(fd[0], tmp, MAXC+1) == 0)
break;
sscanf(tmp, "%2lu%s", &l, stringtoup);
for(i=0; i<l; i++)
stringtoup[i]=toupper(stringtoup[i]);
fprintf(stdout, "%s\n", stringtoup);
fflush(stdout);
sleep(1); // this could be removed
}
close(fd[0]);
exit(0);
}
close(fd[0]);
close(fd[1]);
for(i=0; i<2; i++)
...