Qsort 不对指向字符串的指针数组进行排序

Question

在这个 C 程序中，我将用键盘输入的单词读入一个 char 指针。指针存储在指针数组中。然后我想通过函数比较用 qsort 对数组进行排序。我给它指向我的指针数组的指针。 它根本不对数组进行排序。我不知道我是否在这里获得了 UB，或者我因分配错误而错过了内存。

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

bool read_word(char ***a, int *length);
int comparison(const void *p, const void *q);


int main()
{
    int *length = malloc(sizeof(int));
    *length = 0;
    char **array = malloc(sizeof(char *));
    bool go = false;

    while(go == false)
    {
        printf("Enter word: ");

        go = read_word(&array,length);
    }

    qsort(array, *length - 1,sizeof(char *), comparison);

    printf("\n");


    for(int i = 0; i < *length; i++)
        printf("%s\n", array[i]);

    return 0;
}

bool read_word(char ***a, int *length)
{
    char ch;

    ++*length;

    char *word = malloc(20 * sizeof(char) + 1);
    char *keep_word;

    char **temp = realloc(*a,*length * sizeof(*a));

    if(!temp)
        exit(EXIT_FAILURE);

    *a = temp;

    keep_word = word;

    while((ch = getchar()) != '\n')
        *keep_word++ = ch;

    *keep_word = '\0';

    if(word == keep_word)
    {
        free(word);
        --*length;
        return true;
    }



    (*a)[*length - 1] = word;

    printf("%s", (*a)[*length -1]);
    printf("\nh\n");
    return false;
}

int comparison(const void *p, const void *q)
{
    const char *p1 = p;
    const char *q1 = q;


    return strcmp(p1,q1);
}

Answer 1

改成

int length = 0;//no need malloc
char **array = NULL;//first value is NULL

qsort(array, length, sizeof(char *), comparison);//not length-1

int ch;//type of return value of getchar is int

for(int i = 0; i < 20 &&  (ch = getchar()) != '\n'; ++i)//i : guard

const char *p1 = *(const char **)p;//qsort pass pointer to object to comparison function  
const char *q1 = *(const char **)q;//if char *, char **

Answer 2

你会的

keep_word = word;

以后

if(word == keep_word)

括号中的条件始终为真。

附带说明，您的程序容易出错且难以理解，因为您使用了太多指针。在 main() 中，length 应该是 int，而不是 int*，数组应该是 char*，而不是 char**。在read_word 中，a 应该是char**，而不是char***。除非必要，否则不要使用指针！