Question

我的问题是关于Linux系统编程，特别是关于读写API。

我正在编写一个复制shell的程序。它接受一个字符串参数并用空格分隔符标记它。根据第一个令牌的命令，它使用剩余的令牌作为参数执行操作。到目前为止，我只是为了添加＆＃39;命令。代码循环运行，直到用户输入＆＃39; n＆＃39;为了＆＃39;继续？ [Y / N]＆＃39 ;.但是，在第一次迭代之后，我的程序在第一次write（）调用后跳过read（）以输入命令，并最终在＆＃39; continue？＆＃39; write（）调用。为什么它会在第一次write（）之后立即跳过read（）调用？

int main (int argc, char *argv[]) {
int true=0;
while (true==0) {
    char buff1[]="Please enter your command\n";
    int count1= strlen(buff1);
    write (STDOUT_FILENO, buff1, count1);
    char buff2[100];
    int count2=read (STDIN_FILENO, buff2, 100);
    buff2[count2-1]='\0';
    char *list[30]; //This creates an array of character pointers (strings)
    /*
    * Begin tokenization and entering tokens in list
    */
    const char delim[]=" ";
    char *token;
    token=strtok(buff2, delim);
    const char newline[]="\n";
    int i=0;
    while (token!= NULL) {
        write (STDOUT_FILENO, newline, strlen(newline));
        list[i]=token;
        write (STDOUT_FILENO, list[i], strlen(list[i]));
        i++;
        token=strtok(NULL,delim);
    }
    /*
    * End tokenization
    */

    /*
    * Begin Addition operation
    */
    const char add[]="add";
    if (strcmp(list[0], add)==0) {
        int result=0;
        for (int j=1; j<i; j++) {
            result+=atoi(list[j]);
        }
        char sum[50];
        int sumcount=sprintf(sum, "%d", result);
        write (STDOUT_FILENO, newline, strlen(newline));
        write (STDOUT_FILENO, sum, sumcount);
    }
    /*
    * End Addition operation
    */


    char *truefalse;
    char endmessage[]="Continue: [y/n]\n";
    write (STDOUT_FILENO, endmessage, strlen(endmessage));
    read (STDIN_FILENO, truefalse, 1);
    if (*truefalse=='n') {
        true=1;
    }

}
return 0;
}

As this output image shows, in the second iteration, after asking me to enter a command, the code skips to asking me to continue rather than actually reading my command

Answer 1

您的程序有不确定的行为。

使用尚未初始化的指针指向任何有效的指针。

该行

char *truefalse;

声明一个指针，但它尚未初始化为指向任何有效的指针。你继续在行

中使用它

read (STDIN_FILENO, truefalse, 1);

而不是

char *truefalse;
char endmessage[]="Continue: [y/n]\n";
write (STDOUT_FILENO, endmessage, strlen(endmessage));
read (STDIN_FILENO, truefalse, 1);

使用

char truefalse; // Make it an object instead of a pointer.
char endmessage[]="Continue: [y/n]\n";
write (STDOUT_FILENO, endmessage, strlen(endmessage));
read (STDIN_FILENO, &truefalse, 1); // Use the address of the variable.

更新

您的代码不等待您在第二次迭代中输入任何内容的原因是换行符仍留在输入流中。第二个调用只是读取换行符。

在阅读问题的答案后，您需要使用代码跳过剩下的部分。

最简单的方法是使用：

int c;
while ((c = fgetc(stdin)) != EOF && c != '\n');

为什么以下C代码跳过read（）系统调用来执行下一个write（）系统调用？

1 个答案:

更新