将字符串从结构传递到字符串数组-C

Question

typedef struct mensagem
{
    int sender ;
    int receiver ;
    char *text ;
} *Item ;

typedef struct node
{
    Item item ;
    struct node *next ;
} *link ;

typedef struct queue
{
    link head, tail ;
    int size ;
} *Queue ;

     void listsorted(Queue list)
    {
        Queue temp = list ;
        int s=temp->size ;
        char *sorted ;
        int i=0;

        sorted = (char*) malloc(sizeof(char));

        while ( i < s )
        {
            strcpy( sorted[i] , list->head->item->text ) ;
            list->head = list->head->next ;
            i++ ;
        }

        i=0;

        for ( i=0 ; i<s ; i++ )
        {
            printf("%s\n", sorted[i] ) ;
        }
    }

我想按字母顺序对队列进行排序，因此我想将字符串复制到数组并对该数组进行qsort。 但是我什至没有用字符串来制作数组。 我在strcpy上遇到错误。 我究竟做错了什么？

Answer 1

您不是在malloc（）修改数组的两个维度。 查看以下代码（未经测试）：

 void listsorted(Queue list)
{
    Queue temp = list ;
    int s=temp->size ;
    char **sorted ; /* Changed here */
    int i=0;

    sorted = malloc((s + 1) * sizeof(char *)); /* Changed here */

    while ( i < s )
    {
        sorted[i] = malloc(strlen(list->head->item->text) + 1); /* Changed here */
        strcpy( sorted[i] , list->head->item->text ) ;
        list->head = list->head->next ;
        i++ ;
    }
    sorted[i] = NULL; /* Changed here. NULL terminate array */

    i=0;

    for ( i=0 ; i<s ; i++ )
    {
        printf("%s\n", sorted[i] ) ;
    }
}

Answer 2

sorted[i]是char类型，而不是char *类型。 那是因为您只分配一个一维数组（即字符串），而不是二维数组（即字符串数组）。 此外，你malloc荷兰国际集团字节，然后尝试的内容复制list->head->item->text到它。 即使您可以编译此代码，这也将是个坏消息。

确保始终分配正确的内存量，并确保始终使用strncpy而不是strcpy 。