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，然后是{{ 1}}，...，然后102，110，111，... 112，199，2，{{ 1}}，20，...，我想这不是你想要的。最好将数字转换为200并使用关系运算符进行排序。

我复制到二进制文件时出现分段错误

1 个答案: