c - 复制到二进制文件时出现段错误

Question

我有作业。从使用格式

的文本文件进行复制

name    birthday      street
NAME0;DAY/MONTH/YEAR;STREET0;
NAME1;DAY/MONTH/YEAR;STREET1;
NAME2;DAY/MONTH/YEAR;STREET2;

以上信息并按年份排序。排序后，所有内容都必须复制到二进制文件中。这一直给我带来段错误。我认为问题在于用于读取每一行的所有信息的 token 。

编辑: 使用调试器，我发现当 i = 4 时第 167 行导致了错误。 strcpy(stud[i].day,token);

编辑: 我认为问题出在输入文件中。我重新创建了，现在可以了。

void students_list(FILE *ifp , FILE *b_ifp)
{

    char line[150] ;
    char *token;
    int i = 0 , n , t = 0 ;

    memset(line,0,sizeof(line));

    ifp = fopen("students.txt","r");
    b_ifp = fopen("sorted.bin","wb");

    struct students
    {
        char name[20] , street[50] , year[5] , month[3] , day[3];

    };
    typedef struct students students;

    students stud[32], aux;

    while(fgets(linie,sizeof(line), ifp) != NULL)
    {

        token = strtok(line,";");
        strcpy(stud[i].name,token);

        token = strtok(NULL,"/");
        strcpy(stud[i].day,token);

        token = strtok(NULL,"/");
        strcpy(stud[i].month,token);

        token = strtok(NULL,";");
        strcpy(stud[i].year,token);

        token = strtok(NULL,";");
        strcpy(stud[i].street,token);

        i++;
    }

    n = i;
    while(t == 0)
    {
        t = 1;
        for(i = 0 ; i < n ; i++)
        {
            if(strcmp(stud[i].year,stud[i + 1].year) > 0)
            {
                aux = stud[i];
                stud[i] = stud[i + 1];
                stud[i + 1] = aux;
                t = 0;

            }
        }
    }

    for(i = 0; i < n ; i++)
    {
        fwrite(&stud[i], sizeof(stud[i]), 1, b_ifp);
    }

    fclose(ifp);
    fclose(b_ifp);
}

Answer 1

基本上，问题可能有几个。您可以拥有一个包含超过 32 条目的文件。学生。您不检查 strtok 的返回码(如果您没有提供正确的格式并且没有更多字段，则可以是 NULL )。您可以执行所有操作，但拒绝添加记录，如果 strtok返回值为NULL在输入上。并检查数组大小(使用类似 while(n < 32 && fgets(... 的内容，就像对该问题的一条评论所暗示的那样)

类似于:

char *name, *street, *year, *month, *day;

name   = strtok(line, ";");  if (!name) continue;
year   = strtok(NULL, "/");  if (!year) continue;
month  = strtok(NULL, "/");  if (!month) continue;
day    = strtok(NULL, ";");  if (!day) continue;
/* a line has a \n at the end, so we try to eliminate it if last ; not present */
street = strtok(NULL, ";\n"); if (!street) continue;

然后，您需要遵守可能的长字段，如 strcpy不会并且可能会导致您覆盖它们的末尾。如:

strncpy(stud[i].name, sizeof stud[i].name, name);
strncpy(stud[i].day, sizeof stud[i].day, day);
...

这还将检查是否正确地用空字符完成每个字符串，以防由于空格而必须截断字段(我认为您没有在 stud 结构的字符串字段中为其提供空间)

最后，我建议，当您读取并排序核心内存中的所有数据时，只执行一个 fwrite打电话，例如:

res = fwrite(stud, sizeof stud[0], n, b_ifp);

而不是循环遍历所有记录。

另一个需要争论的问题是，您是否希望按数字或字典顺序对年份进行排序，因为年份肯定不会跨越范围 1950..2050没有问题，但假设您必须订购 1..1000 范围内的数字按字典顺序，先出现 1 ，然后10 ，然后100 ，然后1000 ，然后101 ，然后102 ，...，然后110 , 111 , 112 , ... 199 , 2 , 20 , 200 , 201 ，...，我想这不是你想要的。最好将数字转换为 int s 并使用关系运算符进行排序。