Question

我正在开发一个程序，该程序充当外壳解释程序，该程序读取带有参数的命令，并创建一个使用execvp()执行命令的子级。我一直在做一些字符串操作来收集字符数组*args[]，特别是使用fgets和strtok时。

这是我的代码的MCVE。

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

#define MAX_LINE 80

int main(void){
    //initialize variables
    char *args[MAX_LINE/2 + 1];
    char input[MAX_LINE];
    //char input[MAX_LINE] = "some sentence unknown"; // <-- this line works fine..

    int counter = 0;
    printf("COMMANDER>");
    fflush(stdout);

    //receive input
    fgets(input,MAX_LINE,stdin);

    //parse input
    char *parser;
    parser = strtok(input," \r\t");

    //parse line
    while(parser != NULL){
                args[counter] = parser;
        counter++;  
        parser = strtok(NULL," ");
    }

    //print results
    int i = 0;
    for(i = 0; i < counter + 1;i++){
        printf("1");
        printf(" - %d: %s\n",i,args[i]);    
    } 

    return 0;
}

这里的问题是输出。当我尝试运行此命令时，得到以下输出：

COMMANDER>some sentence unknown
1 - 0: some
1 - 1: sentence
1 - 2: unknown

1 - 3: (null)

我的问题是空白。无论我做什么，我都无法分辨它的来源。

据我所知，它可能是字符串末尾的\ n字符或其他内容，但是将其作为execvp传递给execvp(args[0],args)会产生错误，因为它解释了此空白行作为“”的参数。

我注释掉了一行，仅是main开头的字符串分配。如果使用此分配而不是fgets，则程序会运行，并且我会得到所需的输入：

COMMANDER>some sentence unknown
1 - 0: some
1 - 1: sentence
1 - 2: unknown
1 - 3: (null)

感谢您的阅读。我对C有点生疏，所以我一个人呆了几个小时，仍然找不到解决方法。

Answer 1

如果您阅读例如this fgets reference，您会看到它说

如果发生文件末尾或找到换行符，分析将停止，在这种情况下，str 将包含该换行符 。

_{[重点突出]}

您看到的“空白处”是fgets在字符串末尾添加的换行符。

但是您的代码中有一个更糟糕的问题。

使用

char *args[MAX_LINE/2 + 1];

您定义了一个指针数组，但未初始化该数组。在C中，未初始化的局部（和非静态）变量确实未初始化。它们的内容将不确定，并且看起来几乎是随机的。

更具体地说，您碰巧在args[counter]处获得了一个空指针，这纯粹是运气。

尝试不经初始化就以任何方式使用此指针将导致undefined behavior。

简单的解决方案是将数组显式初始化为充满空指针：

char *args[MAX_LINE/2 + 1] = { NULL };

上面将对所有元素进行“零初始化”，这对于指针意味着它们将为NULL。

Answer 2

由fgets()读取的行包含尾随换行符'\n'。您必须将其包含在strtok()识别为定界符的字符列表中。此外，您必须将相同的列表传递给两个呼叫。

您还忘记将偏移量count的条目设置为NULL。具有自动存储功能的本地对象未初始化。还要注意，您不应该为printf转换说明符传递指向%s的空指针：尽管某些实现测试该参数并输出特定的字符串，但它具有未定义的行为。

这是更正的版本：

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

#define MAX_LINE 80

int main(void){
    char *args[MAX_LINE/2 + 1];
    char input[MAX_LINE];
    int counter;

    printf("COMMANDER>");
    fflush(stdout);

    //receive input
    if (fgets(input, sizeof input, stdin)) {
        //parse input
        char *parser;
        parser = strtok(input, " \f\n\r\t");

        //parse line
        count = 0;
        while (parser != NULL) {
            args[counter] = parser;
            counter++;  
            parser = strtok(NULL, " \f\n\r\t");
        }
        args[counter] = NULL;

        //print results
        int i;
        for (i = 0; i <= counter; i++) {
            printf("1 - %d: %s\n", i, args[i] ? args[i] : "(null)");
        } 
    }
    return 0;
}

strtok和fgets的奇数数组行为

2 个答案: