指针算术分段问题

Question

 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>

 int string_cmp(const void *p, const void *q);

 int main(int argc, char **argv)
 {
    int i;   // variable
    char **words_array = malloc(sizeof(char*)*(argc+1)); // sets new array to hold the words
    char *p; // another char pointer array
    p = *words_array;       // set both equal to eachother

    for(; *p < (argc - 1); p++) // for loop
    {
            p = malloc(strlen(*argv) + 1); // determines size based on user input
            argv++; // increments
            strcpy(p++, *argv); // copies words to the new array

    }
    p = NULL; // resets p

    qsort(words_array, argc-1, sizeof(char *), string_cmp); // sorts the array
    for(i = 0; i < argc - 1; i++){ // for loop to print properly
            printf("%s ", words_array[i]);
    }
    printf("\n");
    return 0;
  }

 int string_cmp (const void *p, const void *q) // compares the two different          strings and returns a value
{
    const char *value = *(const char**)p;
    const char *value_two = *(const char**)q;

    return strcmp(value, value_two);
}

所以我的程序应该接受命令行参数并返回它们使用 Qsort 排序。示例是“./a.out 你好黑暗我的老朋友应该作为黑暗朋友你好我的老朋友返回。我没有收到任何编译器错误但是我得到一个分段错误，我不知道如何解决这个问题我的指针算术。

Answer 1

你正在增加双指针（argv），即；

for(; *p < (argc - 1); p++) // for loop
{
        p = malloc(strlen(*argv) + 1); // determines size based on user input
        argv++; // increments
        strcpy(p++, *argv); // copies words to the new array

}

所以把它改成 (*argv)++

Answer 2

问题出在你的 for 循环中。您将*p 与argc 进行比较，这毫无意义。将循环替换为标准计数器i。

for (i = 1; i < argc; i++)

请注意，应该使用argc，而不是argc - 1，并且循环应该从1开始，而不是从0开始。 另外，在循环中你可以使用argv[i]而不是*argv

Answer 3

这些是一些建议：

将char** argv 更改为char* argv[]

argc 返回比实际传递的参数数量多一。额外的计数是针对可执行文件名称本身的。

所以最好做一些错误检查，做：

argc--;
if(argc > 0) // We have some arguments
{
/* Do something 
 * char **words_array = malloc(sizeof(char*)*(argc+1)
 * may be changed to 
 */
  char **words_array;
  words_array=malloc(argc*sizeof(char*));
/* Coming down 
 * You could change that for-loop to something like this.
 */
 for(int i=0;i<argc;i++)
     words_array[i]=argv[i]; //  You have all the arguments stored in words_array
/* Now go on sort words_array
 * and so and so forth
 */    

}