将一行解析为 ***char 时出现段错误

Question

我的代码应该将一个字符数组解析为 ***char，因此它首先将其拆分为 '|' char 然后由空格、换行符等转换成单词。示例输入/输出：

I = ls -l | sort | unique

O =

  *cmds[1] = {"ls", "-l", NULL};
  *cmds[2] = {"sort", NULL};
  *cmds[3] = {"unique", NULL};

上面是指向char数组的指针，所以按单词分割，下面是***char，上面有指向上面指针的指针

  char **cmds[] = {1, 2, 3, NULL};

现在，我没有看到我的错误（可能是因为我对 C 语言不太熟练），但是程序在我从 parsePipe() 内部调用 parse(..) 函数的第二次给出了段错误。有人可以帮忙吗？

void parse(char *line, char **argv)
{
        while (*line != '\0') {     
        while (*line == ' ' || *line == '\t' || *line == '\n')
            *line++ = '\0'; 
        *argv++ = line;     
        while (*line != '\0' && *line != ' ' && *line != '\t' && *line != '\n'){
            line++;
        }
    }
    *argv = '\0';  
}

void parsePipe(char *line, char ***cmds)
{
    char *cmd = strtok(line, "|");
    int word_counter = 0;

    while (cmd != NULL) 
    {
       printf("Printing word -> %s\n", cmd);
       word_counter++; 
       parse(cmd, *cmds++);

       cmd = strtok(NULL, "|");
    }

    printf("This string contains %d words separated with |\n",word_counter);
}


void  main(void)
{
    char  line[1024];          
    char  **cmds[64]; 
    while (1) {    
        printf("lsh -> ");   
        gets(line);     
        printf("\n");
        parsePipe(line, cmds);
    }
}

Answer 1

[评论太长]

这一行

*argv++ = line; /* with char ** argv */

指的是无效的内存，就像代码中的 *argv[n] (with char **argv[64]) 一样，它什么都不引用。

你使用的命名并没有让生活更轻松。

尝试以下命名：

void parse(char *line, char **cmd)
{
    while (*line != '\0') {     
        while (*line == ' ' || *line == '\t' || *line == '\n')
            *line++ = '\0'; 
        *cmd++ = line;     
        while (*line != '\0' && *line != ' ' && *line != '\t' && *line != '\n'){
            line++;
        }
    }
    *argv = '\0';  
}

void parsePipe(char *line, char ***cmdline)
{
    char *cmd = strtok(line, "|");
    int word_counter = 0;

    while (cmd != NULL) 
    {
       printf("Printing word -> %s\n", cmd);
       word_counter++; 
       parse(cmd, *cmdline++);

       cmd = strtok(NULL, "|");
    }

    printf("This string contains %d words separated with |\n",word_counter);
}


void  main(void)
{
    char  line[1024];          
    char  **cmdline[64]; 
    while (1) {    
        printf("lsh -> ");   
        gets(line);     
        printf("\n");
        parsePipe(line, cmdline);
    }
}

cmds 使用的内存都没有被分配。

所以

*cmd++ = line;

失败，因为cmd 没有指向任何地方，但是被取消引用并且代码尝试写入它指向的地方，这是无处可去的，这是无效的内存。

---

可以通过将char*** 传递给parse(char *** pcmd) 并计算找到的令牌来解决此问题

size_t nlines = 0;

...

++nlines.

和做一个

*pcmd = realloc(*pcmd, nlines + 1); /* Allocate one more as needed to later find the end of the array. */
(*pcmd)[nlines -1] = line;
(*pcmd)[nlines] = NULL; /* Initialise the stopper, marking the end of the array. */

对于找到的每个令牌。

显然你需要这样称呼它：

 parse(cmd, cmdline++);

为了完成所有这些工作，初始数组需要正确初始化（无论如何你都应该这样做）：

char  **cmdline[64] = {0};