C: 使用 system() 命令

Question

我正在编写一个充当简单外壳的程序。用户从命令行调用程序并提示输入发送到操作系统以完成的命令。它应该一直运行到用户输入“完成”，此时程序应该中断。我在输入 done 时遇到问题 - 程序按原样退出，但打印出来

sh: -c: line 0: syntax error near unexpected token `done'
sh: -c: line 0: `done'

在完成执行之前到终端。这是我编写的适用的代码：

char isDone[] = "done\n"; //fgets stores the new line character from the command line

do {
    printf(">>"); //print the prompt each time through the loop
    fgets(command, 50, stdin); //get a line from the terminal
    system(command);
} while (strcmp(command, isDone) != 0); //break the loop if the user types in done

我认为该错误与“完成”不是有效的 UNIX 命令这一事实有关，但我不确定如何处理此错误。我尝试通过以下修复解决此问题：

if(system(command) == -1){
    printf("The OS doesn't recognize this command");
}
else
    system(command);

但这并没有解决问题或将错误打印到屏幕上，并产生了第二个问题，即两次打印命令/错误 - 一次在 if 条件块中，一次在 else 块中。我该如何解决这个问题？

编辑 这是一个家庭作业问题，需要使用 do-while。有没有使用 do-while 的解决方案？

Answer 1

do...while 构造在检查循环条件之前执行其主体。因此，当循环“意识到”用户输入了done 时，它已经尝试将该输入作为循环体内的命令执行。

解决此问题的最简单方法是使用break：

while (1)
{
    fgets(command, 50, stdin);
    if (!strcmp(command, isDone)) break;
    system(command);
}

以这种方式构造它的原因是每次迭代都包含应该在条件之前完成的操作（读取用户输入）和应该在条件之后完成的操作（使用system() 执行命令） .因此，do...while 或简单的while 都不允许您直观地构建代码。 break 关键字提供了一种将循环的终止条件放在循环体中间的方法。

Answer 2

执行顺序为：

fgets(command, 50, stdin); //get a line from the terminal
system(command);
strcmp(command, isDone) != 0

因此“完成”行被读取，发送到系统（尝试将其作为 shell 命令执行，打印错误），然后才对其进行检查。

你可以试试这样的：

for(;;){
    printf(">>"); //print the prompt each time through the loop
    fgets(command, 50, stdin); //get a line from the terminal
    if(!strcmp(command, isDone)) break; //break the loop
    system(command);
}

编辑：如果你想保留do-while：

printf(">>"); //print the prompt each time through the loop
fgets(command, 50, stdin); //get a line from the terminal
do {
    system(command);
    printf(">>"); //print the prompt each time through the loop
    fgets(command, 50, stdin); //get a line from the terminal
} while (strcmp(command, isDone) != 0); //break the loop if the user types in done

但break 版本显然更具可读性。

Answer 3

在 fgets() 之后，在 if 语句中执行 system() 调用：

if ( strcmp( isDone, command) != 0 ) {
    system( command );
}