[英]Parent Process Waiting for Child to Finish in C
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#define MAX_LINE 80 /* 80 chars per line, per command, should be enough. */
/**
* setup() reads in the next command line, separating it into distinct tokens
* using whitespace as delimiters. It also sets the args parameter as a
* null-terminated string.
*/
void setup(char inputBuffer[], char *args[],int *background)
{
int length, /* Number of characters in the command line */
i, /* Loop index for inputBuffer array */
start, /* Index where beginning of next command parameter is */
ct; /* Index of where to place the next parameter into args[] */
ct = 0;
/* Read what the user enters on the command line */
length = read(STDIN_FILENO, inputBuffer, MAX_LINE);
start = -1;
if (length == 0)
exit(0); /* ^d was entered, end of user command stream */
if (length < 0){
perror("error reading command");
exit(-1); /* terminate with error code of -1 */
}
/* Examine every character in the inputBuffer */
for (i = 0; i < length; i++) {
switch (inputBuffer[i]){
case ' ':
case '\t' : /* argument separators */
if(start != -1){
args[ct] = &inputBuffer[start]; /* set up pointer */
ct++;
}
inputBuffer[i] = '\0'; /* add a null char; make a C string */
start = -1;
break;
case '\n': /* should be the final char examined */
if (start != -1){
args[ct] = &inputBuffer[start];
ct++;
}
inputBuffer[i] = '\0';
args[ct] = NULL; /* no more arguments to this command */
break;
case '&':
*background = 1;
inputBuffer[i] = '\0';
break;
default : /* some other character */
if (start == -1)
start = i;
}
}
args[ct] = NULL; /* just in case the input line was > 80 */
}
int main(void)
{
char inputBuffer[MAX_LINE]; /* Buffer to hold the command entered */
int background; /* Equals 1 if a command is followed by '&' */
char *args[MAX_LINE/2+1];/* Command line (of 80) has max of 40 arguments */
while (1){ /* program terminates normally inside setup */
background = 0;
printf("CSE2431Sh->");
fflush(0);
setup(inputBuffer, args, &background); /* get next command */
/* the steps are:
(1) fork a child process using fork()
(2) the child process will invoke execvp()
(3) if background == 0, the parent will wait,
otherwise returns to the setup() function. */
/* MY CODE HERE */
pid_t pid;
pid = fork();
if(pid == 0)
{
execvp(args[0],args);
/* If execvp returns, it must have failed. */
printf("Fork Failed\n");
exit(0);
}
else
{
if(&background == 0)
{
while( wait(&background) != pid)
{/* Do nothing, waiting */}
}
else
{
setup(inputBuffer, args, &background);
}
}
}
}
我试图派生一个子进程,让子进程调用execvp()并让父进程在后台等待。 我的错误来自与父代码的等待部分。 上面说我的代码的所有内容都已给出,不应进行编辑
if(&background == 0) ^
那条线没有多大意义。 当您可能想比较实际的存储值时,即在比较地址时,即您可能要删除&
。
否则,测试将永远不会为真,也就是说,变量background
的地址永远不会为0。
您的“前叉失败”消息应为“执行失败”(前叉有效;执行程序未执行)。 您还应该有一个单独的“ fork failed”错误报告,但此刻您仍未找到该报告。 错误消息应该写入stderr
而不是stdout
。
wait()
循环条件应为:
int corpse;
int status;
while ((corpse = wait(&status)) != -1 && corpse != pid)
;
调试时,在每次迭代中打印来自wait()
的信息。 注意, waitpid()
允许您在有尸体要收集时等待,而在没有死者要哀悼的情况下可以返回。
所有这些仅在处理了应该从编译器获得的警告之后才有意义。 如果没有收到有关if (&background == 0)
始终为false的警告,则需要提高编译器警告级别。 如果您使用的是GCC,则gcc -Wall
是一个好的开始, gcc -Wall -Wextra -Wstrict-prototypes -Wmissing-prototypes
更好。 并修复来自编译器的警告。
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.