C while循环只读文件的第一行

欢迎光临，我的while循环有问题。它只读取file.txt中的第一行。在添加函数以按strtok|r拆分我的行之前，我的应用程序工作不正常。此while循环的目的是从文件中读取所有行，并将每个行拆分为“|”，这些部分添加到相应的选项卡中。 我该怎么办？ 这是我的代码：

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <mem.h>
#include <assert.h>

struct movie {
    char title[100];
    int year;
    char genre[100];
};

int i = 0;
struct movie *movies[50];



int porownaj(struct movie* first, struct movie* second)
{
    if(first->year==second->year){
            if(first->title>second->title){
                return 1;
            }
            if(first->title<second->title){
                return -1;
            }
            if(first->title==second->title){
                    if(first->genre>second->genre){
                        return 1;
                    }
                    if(first->genre<second->genre){
                        return -1;
                    }
                    if(first->genre==second->genre){
                        return 0;
                    }
            }
    }
    if(first->year>second->year){
        return 1;
    }
    if(first->year<second->year){
        return -1;
    }

    if(first->title==second->title){
        if(first->year>second->year){
                return 1;
            }
            if(first->year<second->year){
                return -1;
            }
            if(first->year==second->year){
                    if(first->genre>second->genre){
                        return 1;
                    }
                    if(first->genre<second->genre){
                        return -1;
                    }
                    if(first->genre==second->genre){
                        return 0;
                    }
            }
    }
    if(first->title>second->title){
        return 1;
    }
    if(first->title<second->title){
        return -1;
    }

    if(first->genre==second->genre){
        if(first->year>second->year){
                return 1;
            }
            if(first->year<second->year){
                return -1;
            }
            if(first->year==second->year){
                    if(first->title>second->title){
                        return 1;
                    }
                    if(first->title<second->title){
                        return -1;
                    }
                    if(first->title==second->title){
                        return 0;
                    }
            }
    }
    if(first->genre>second->genre){
        return 1;
    }
    if(first->genre<second->genre){
        return -1;
    }

}


void zamien(struct movie* movies[50], int x, int y)
{
    struct movie* tmpMovie = movies[x];
    movies[x] = movies[y];
    movies[y] = tmpMovie;
}

int main()
{

    struct movie *movies[50];
    do
    {
        printf("\n");
        printf("Projekt nr 6 \n");
        printf("\n");
        printf("\b   MENU GLOWNE \n");
        printf("|-----------------| \n");
        printf("[1]Baza filmow \n");
        printf("[2]Dodaj film \n");
        printf("[3]Zakoncz Program \n");
        printf("|-----------------| \n");

        char znak;

        printf("Wybierz opcje z menu: ");
        scanf("%s", &znak);

        system("cls");

        switch (znak)
        {
        case '1':
            system("cls");
            printf("Wybrales opcje BAZA FILMOW \n");
            printf("TYTUL | ROK |  GATUNEK\n");
            printf("----------------------\n");

            FILE *baza;
            if ((baza = fopen("movies.txt", "r")) == NULL)
            {
                system("cls");
                printf("Blad otwarcia pliku\n");
                exit(1);
            }

            char tekst[512];

            char *title    = NULL;
            char *year     = NULL;
            char *genre    = NULL;

            char *buffer  = NULL;
            int x         = 0;

            while (fgets(tekst, 512, baza) != NULL)
            {
                printf("%s", tekst);

                title   = strtok_r(tekst, "|", &buffer);
                assert(title != NULL);
                strcpy(movies[x]->title, title);

                year     = strtok_r(NULL, "|", &buffer);
                assert(year != NULL);
                movies[x]->year = atoi(year);

                genre = strtok_r(NULL, "|", &buffer);
                assert(genre != NULL);
                strcpy(movies[x]->genre, genre);

                x++;


            }fclose(baza);


            do
            {
                printf("\n[1]Sortuj wg. tytulu");
                printf("\n[2]Sortuj wg. roku produkcji");
                printf("\n[3]Sortuj wg. gatunku filmu");
                printf("\n[4]Wroc do glownego menu\n");
                printf("|-----------------|\n");

                char click;

                printf("\nWybierz opcje: ");

                scanf("%s", &click);

                switch (click)
                {
                case '1':
                    system("cls");
                    printf("Sorotowanie wg. tytulu");
/*
                    for(int i=0;i<10-1;i++){
                                for(int j=0;j<10-i-1;j++){
                                    if(porownaj(&title[j],&title[j+1])>0){
                                        zamien(&title[j],&title[j+1]);
                                    }
                                }
                            }
                                for(int i=0;i<10;i++){
                                    printf("(%d, %d)", title[i].x, );
                                }
*/
                    break;
                case '2':
                    system("cls");
                    printf("Sorotowanie wg. roku produkcji");
/*
                    for(int i=0;i<10-1;i++){
                                for(int j=0;j<10-i-1;j++){
                                    if(porownaj(&year[j],&year[j+1])>0){
                                        zamien(&year[j],&year[j+1]);
                                    }
                                }
                            }
                                for(int i=0;i<10;i++){
                                    printf("(%d, %d)", year[i].x, );
                                }
*/
                    break;
                case '3':
                    system("cls");
                    printf("Sorotowanie wg. gatunku filmu");

                    break;
                case '4':
                    system("cls");
                    return main();
                default:
                    system("cls");
                    printf("Nie ma takiej opcji. Program sie wylaczy!");
                    return 0;
                }
            } while (1);

        case '2':
            system("cls");
            printf("Wybrales opcje DODAJ FILM \n");

            movies[i] = malloc(sizeof(movie));

            printf("Podaj nazwe filmu: ");
            while ((getchar()) != '\n');
            gets(movies[i]->title);

            printf("Podaj rok produkcji: ");
            scanf("%d", &(movies[i]->year));

            printf("Podaj gatunek filmu: ");
            while ((getchar()) != '\n');

            gets(movies[i]->genre);

            fflush(stdin);

            FILE *plik = fopen("movies.txt", "a+");
            fprintf(plik, "%s | %d | %s \n", movies[i]->title, movies[i]->year, movies[i]->genre);
            fclose(plik);
            i++;
            printf("Zapisane..");
            continue;
        case '3':
            system("cls");
            printf("ZAKONCZ PROGRAM \n");
            return 0;
        default:
            printf("Nie ma takiej opcji! \n");
            return main();
        }
        }while (1);

return (0);
}

---

在任何其他可能的问题中，这些行都是有问题的：

    title=movies[x]->title;
    year=movies[x]->year;
    genre=movies[x]->genre;

• 正如在备注中所说的，分配必须颠倒（

  movies[x]->foo=foo；

  ）
• 您需要复制字符串，否则将保存tekst的子地址，该子地址在每个循环中都会被修改，并且在使用它们时可能会消失

所以

或者，如果

movies[x]->title

和

movies[x]->genre

是

char[]

而不是

char*

则代码必须复制而不是复制：

    strcpy(movies[x]->title, title);
    movies[x]->year = year;
    strcpy(movies[x]->genre, genre);

---

如果需要，用strncpy替换strcpy以避免溢出

我打赌每一行都是用

printf（“%s”，tekst）打印的

但是您忘记了strtok

不会复制字符串。因此（如果您的分配以另一种方式固定），您仍然将POINTER分配到所有电影的相同

tekst

变量中。而是分配内存并复制字符串：

    title   = strtok_r(tekst, "|", &buffer);
    movies[x]->title= malloc(strlen(title)+1);
    strcpy(movies[x]->title, title);

---

其他字符串变量也是如此。

程序中有许多错误

首先：

        char title    = NULL;
        char year     = NULL;
        char genre    = NULL;

        title   = strtok_r(tekst, "|", &buffer);
        year     = strtok_r(NULL, "|", &buffer);
        genre = strtok_r(NULL, "|", &buffer);
        int year = atoi(year);

除此之外：

        char title    = NULL;
        char year     = NULL;
        char genre    = NULL;

        title   = strtok_r(tekst, "|", &buffer);
        year     = strtok_r(NULL, "|", &buffer);
        genre = strtok_r(NULL, "|", &buffer);
        int year = atoi(year);

您将

title

定义为

char

，但

strtok\u r

返回

char*

。因此，您可以通过在“char”变量中存储“char指针”来覆盖内存。 这可能会破坏文件指针。您得到的错误0xC0000005（正如您在评论中提到的）可能表明：

应用程序错误0xc0000005（访问冲突）错误通常是由于您的计算机无法正确处理文件

将定义更改为：

        char *title    = NULL;
        char *year     = NULL;
        char *genre    = NULL;

进一步：

        year     = strtok_r(NULL, "|", &buffer);  // A variable named year
        genre = strtok_r(NULL, "|", &buffer);
        int year = atoi(year);                    // A NEW variable with same name!!

将其更改为：

        year     = strtok_r(NULL, "|", &buffer);
        genre = strtok_r(NULL, "|", &buffer);
        int yearN = atoi(year);                // Use another name

此外，在调用

atoi

所以我的建议是简化调试代码。尝试：

        FILE *baza;
        if ((baza = fopen("movies.txt", "r")) == NULL)
        {
            system("cls");
            printf("Blad otwarcia pliku\n");
            exit(1);
        }

        char tekst[512];
        char* title    = NULL;
        char* year     = NULL;
        char* genre    = NULL;
        char* buffer  = NULL;
        int x         = 0;
    printf("while-loop start\n");
    while (fgets(tekst, 512, baza) != NULL)
    {
        printf("%s", tekst);
        //x++;

        //title   = strtok_r(tekst, "|", &buffer);
        //year     = strtok_r(NULL, "|", &buffer);
        //genre = strtok_r(NULL, "|", &buffer);
        //int year = atoi(year);

        //title=movies[x]->title;
        //year=movies[x]->year;
        //genre=movies[x]->genre;

    }
    printf("while-loop complete\n");

现在检查从这段代码中得到的输出。如果按照预期工作，您可以开始逐个取消注释行

最后，我想你的函数应该是这样的：

        FILE *baza;
        if ((baza = fopen("movies.txt", "r")) == NULL)
        {
            system("cls");
            printf("Blad otwarcia pliku\n");
            exit(1);
        }

        char tekst[512];
        char* title    = NULL;
        char* year     = NULL;
        char* genre    = NULL;
        char* buffer  = NULL;
        int x         = 0;

    while (fgets(tekst, 512, baza) != NULL)
    {
        printf("%s", tekst);

        title   = strtok_r(tekst, "|", &buffer);
        assert(title != NULL);
        strcpy(movies[x]->title, title);            

        year     = strtok_r(NULL, "|", &buffer);
        assert(year != NULL);
        movies[x]->year = atoi(year);

        genre = strtok_r(NULL, "|", &buffer);
        assert(genre != NULL);
        strcpy(movies[x]->genre, genre);            

        x++;
    }

以上假设您已经为

电影

中的所有指针分配了内存。比如：

struct movie { char title[100]; int year; char genre[100]; }; 

int main() {
    int i = 0; 
    struct movie *movies[50];
    for(i=0; i<50; ++i) movies[i] = malloc(sizeof *movies);
    . . .

struct电影{char title[100]；int year；char流派[100]；}；
int main（）{
int i=0；
struct movie*电影[50]；
对于（i=0；我将遇到堆栈溢出。请阅读、获取、阅读以及。最后，请尝试创建一个来向我们展示。
title=movies[x]->title；
您的意思是
movies[x]->title=title；
？但即使这样也会错…请处理一些人提供的问题检查表。我试图找到解决此问题的任何方法，但我无法：（首先向我们显示输入文件的内容。@PabloKwiatkowski您是说电影[x]->title和电影[x]->类型是char[]而不是char*？我必须检查它们是char[]@PabloKwiatkowski好的，看看strcpy，我刚刚编辑了我的answer@PabloKwiatkowski还有什么附加功能？崩溃？或者简单地使用
strdup（）
（如果可用）（几乎在任何地方）是的，就像我在几分钟前的回答中所做的那样^^^查看对您的问题的评论以了解更多错误。但是在每次新打印之前，使用
fgets
读取
tekst
，因此我无法理解这如何解释编译器的loopTurn警告中只打印了一行。存在一些问题（错误）这可能会扰乱程序。例如，将指针指定给字符。注释不用于扩展讨论；此对话已被删除。