c - 排序篮球运动员姓名的功能

我需要编写一个程序来接收（来自用户）篮球队的球员数量，然后我需要以动态方式创建一个数组。程序将按字母顺序对数组进行排序，然后按此顺序打印。
我写了这段代码：

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

#define LENGTH 20

int main(void)
{
    int numplayers, i, j;
    char persname[LENGTH], tname[LENGTH], temp[LENGTH];

    printf("Please insert the amount of basketball players in the group\n");
    scanf("%d", &numplayers);

    char **players = (char **) malloc(numplayers * sizeof(char *));

    printf("Please insert the names of your %d basketball players\n", numplayers);
    for (i = 0; i < numplayers; i++) {
        gets(persname);
        strcpy(persname[i], tname[i]);
    }

    for (i = 0; i < numplayers-1; i++) {
        for (j = i+1; j < numplayers; j++) {
            if (strcmp(persname[i], persname[j])) {
                strcpy(temp[i], persname[i]);
                strcpy(persname[i], persname[j]);
                strcpy(persname[j], temp);
            }
        }
    }

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

    return 0;
}

但是当我运行代码时，在输入团队中的玩家数量之后，我会得到一个错误，Unhandled exception at 0x507340E3 (msvcr120d.dll) in Question4.exe: 0xC0000005: Access violation reading location 0xFFFFFFCC.
我做错了什么。

最佳答案

输入所有名称的循环不使用players。相反，它错误地使用了pername和tname。这一行：

strcpy(persname[i], tname[i]);

不应该编译，你用一种没有任何意义的方式来混合类型。您应该输入一行，然后动态地将新内存分配到players[i]并将输入复制到那里。如果你有strdup()，那就是它的好处。
基本上，输入循环应该类似于：

for (i = 0; i < numplayers; i++)
{
    char line[1024];
    if(fgets(line, sizeof line, stdin) != NULL)
    {
       const size_t len = strlen(line);
       players[i] = malloc(len + 1);
       if(players[i] == NULL)
       {
         fprintf(stderr, "**Out of memory!\n");
         exit(1);
       }
       memcpy(players[i], line, len + 1);
    }
    else
      fprintf(stderr, "**I/O error!\n");
}

这使用了fgets()这比可怕的从未使用过的gets()怪物要安全得多。
此外，您不会为单个名称分配任何空间，只为字符串指针数组分配空间。
这一行：

char** players = (char**)malloc(numplayers*sizeof(char*));

可以简化为更清晰：

char** players = malloc(numplayers * sizeof *players);

不需要重复类型名，并且no need to cast the return value of malloc()。

关于c - 排序篮球运动员姓名的功能，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/29699184/