2

您好，我正在做一个struct基于项目的项目。我在保存结构的第一个地址时遇到问题。问题不在于我的 N 函数，它工作正常。我的问题是，在我的 V 函数中，它只打印我的最后一个结构列表。我希望它打印我的结构中的所有信息，而不仅仅是最后一个。我希望我让自己可以理解

代码：

typedef struct stuff {
    char signatura[12];
    char isbn[15];
    char kniha[100];
    char autori[100];
    int datum;
    int preukaz;
    struct stuff *p_dalsi;

} STUFF;

STUFF *alokuj(void){
    STUFF *p_pom;
    p_pom=(STUFF *) malloc (sizeof(STUFF));
    return p_pom;
}

void nacitaj(STUFF *p_akt){
    FILE *fr;
    int pocet_zaznam=0, pocet_enter=0, i;
    char c, s[100];

    if ((fr = fopen("KNIZNICA.TXT","r")) == NULL){
        printf("Zaznamy neboli nacitane\n");
    }
    while((c=getc(fr))!= EOF) { 
        if(c=='\n') pocet_enter++;
    }
    pocet_zaznam=(pocet_enter+1)/7;jeden zaznam ma 7 casti
    rewind(fr);i


    for (i=1;i<=pocet_zaznam;i++){

        fgets(s,100,fr);    //vynechanie prveho riadku v subore
        fgets(p_akt->signatura,12,fr);
        fgets(p_akt->isbn,15,fr);
        fgets(p_akt->kniha,100,fr);
        fgets(p_akt->autori,100,fr);
        fscanf(fr,"%d\n",&p_akt->datum);
        fscanf(fr,"%d\n",&p_akt->preukaz);

        p_akt->p_dalsi=NULL;
    }


    printf("Nacitalo sa %d zaznamov\n",pocet_zaznam);
    fclose(fr);

}

void vypis(STUFF *p_akt) {
    int zaznam_poradie=1;

    while(p_akt!=NULL) {
        printf("%d.\n",zaznam_poradie);
        printf("signatura: %s",p_akt->signatura);
        printf("isbn: %s",p_akt->isbn);
        printf("kniha: %s",p_akt->kniha);
        printf("autori: %s",p_akt->autori);
        printf("datum: %d\n",p_akt->datum);
        printf("datum: %d\n",p_akt->preukaz);
        zaznam_poradie++;

    p_akt=p_akt->p_dalsi;
    }

}

int main() {
    char c;
    STUFF *p_prv = NULL;
    STUFF *p_akt = NULL;


    p_akt = p_prv;
    p_prv = (STUFF *)malloc(sizeof(STUFF));

    while(c!='K') {
        c = getchar();
        if(c=='N') {
            p_akt = p_prv;
            nacitaj(p_akt);
        }
        if(c=='V') {
            p_akt = p_prv;
            vypis(p_akt);
        }

        if(c=='P');
        if(c=='Z');
        if(c=='H');
        if(c=='A');
    }


return 0;
}
4

1 回答 1

2

问题出在你的“N 函数”上；它不会创建列表，调用它的代码也不会。

提供的代码不会编译：

pocet_zaznam=(pocet_enter+1)/7;jeden zaznam ma 7 casti
rewind(fr);i

该jeden ... casti部分可能是没有评论标记的评论。i似乎只是无关紧要。分析时可以忽略这些问题。

但是，该函数似乎读取文件，计算其中的换行数，然后重新读取文件以加载数据。但是，它会继续在同一个结构上进行写入，并且不会尝试分配新结构或将当前结构与以前的结构联系起来。请注意，alokuj()您显示的代码中未调用您的分配函数 。

当最终调用“V 函数”时，它必须使用的唯一数据是读取的最后一个条目，因为先前条目的信息已被覆盖。

您需要重新编写代码，为要读取的每个项目分配一个新结构，将它们全部收集到一个列表中，并且（可能）将列表的头部返回给调用代码。

您还应该使用可能失败的错误检查功能。您确实进行fopen()了检查，但即使返回值是 NULL，您也会继续使用它——您不应该这样做，并且您应该将错误报告给标准错误，而不是标准输出。您应该检查内存分配；你应该检查一下fgets()。当然，当文件打开时，它包含一定数量的行；但是，当您重新读取文件时，它可能会被截断，因此您仍然需要检查。您可能还需要从字符串中删除换行符（fgets()包括字符串中的换行符）。

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

typedef struct stuff
{
    char signatura[12];
    char isbn[15];
    char kniha[100];
    char autori[100];
    int datum;
    int preukaz;
    struct stuff *p_dalsi;
} STUFF;

static STUFF *alokuj(void)
{
    STUFF *p_pom = (STUFF *) malloc (sizeof(STUFF));
    if (p_pom == 0)
    {
        fprintf(stderr, "Out of memory\n");
        exit(1);
    }
    return p_pom;
}

static void vypis(STUFF *p_akt)
{
    int zaznam_poradie=1;

    while (p_akt != NULL)
    {
        printf("%d.\n", zaznam_poradie);
        printf("signatura: %s", p_akt->signatura);
        printf("isbn: %s", p_akt->isbn);
        printf("kniha: %s", p_akt->kniha);
        printf("autori: %s", p_akt->autori);
        printf("datum: %d\n", p_akt->datum);
        printf("datum: %d\n", p_akt->preukaz);
        p_akt=p_akt->p_dalsi;
        zaznam_poradie++;
    }
}

static STUFF *nacitaj(void)
{
    FILE *fr;
    int   pocet_zaznam = 0;
    int   pocet_enter = 0;
    int   i;
    char  s[100];
    int   c;

    if ((fr = fopen("KNIZNICA.TXT", "r")) == NULL)
    {
        fprintf(stderr, "Zaznamy neboli nacitane\n");
        exit(1);
    }
    while ((c = getc(fr)) != EOF)
    { 
        if (c == '\n')
            pocet_enter++;
    }

    pocet_zaznam = (pocet_enter+1)/7;
    rewind(fr);
    STUFF *p_head = 0;
    STUFF *p_last = 0;

    for (i = 1; i <= pocet_zaznam; i++)
    {
        STUFF *p_akt = alokuj();
        if (fgets(s, 100, fr)                   == 0 || // 100 should be sizeof(s) ... etc
            fgets(p_akt->signatura, 12, fr)     == 0 ||
            fgets(p_akt->isbn, 15, fr)          == 0 ||
            fgets(p_akt->kniha, 100, fr)        == 0 ||
            fgets(p_akt->autori, 100, fr)       == 0 ||
            fscanf(fr, "%d\n", &p_akt->datum)   != 1 ||
            fscanf(fr, "%d\n", &p_akt->preukaz) != 1)
        // "%d\n" would be bad for interactive I/O; OK for file I/O
        {
            fprintf(stderr, "Data format error\n");
            exit(1);
        }
        p_akt->p_dalsi = NULL;

        //printf("Read entry %d:\n", i);
        //vypis(p_akt);

        if (p_head == 0)
            p_head = p_akt;
        else
            p_last->p_dalsi = p_akt;
        p_last = p_akt;
    }

    printf("Nacitalo sa %d zaznamov\n", pocet_zaznam);
    fclose(fr);
    return p_head;
}

int main(void)
{
    int c;
    STUFF *p_akt = NULL;

    while (c!='K')
    {
        c = getchar();
        if (c == EOF)
            break;
        else if (c == 'N')
            p_akt = nacitaj();
        else if (c == 'V')
            vypis(p_akt);
        else if (c != '\n')
            fprintf(stderr, "Unrecognized commmand: %c\n", c);
    }

    return 0;
}

数据文件 KNIZNICA.TXT

Rubbish1
signature1
0123456789212
Writing C Programs on Stack Overflow
Jonathan Leffler
20120505
1234
Rubbish2
signature2
9876543210212
Rewriting C Programs on Stack Overflow
Leffler, Jonathan
20130505
2234

示例运行（程序名称ak）：

$ ./ak
N
Nacitalo sa 2 zaznamov
V
1.
signatura: signature1
isbn: 0123456789212
kniha: Writing C Programs on Stack Overflow
autori: Jonathan Leffler
datum: 20120505
datum: 1234
2.
signatura: signature2
isbn: 9876543210212
kniha: Rewriting C Programs on Stack Overflow
autori: Leffler, Jonathan
datum: 20130505
datum: 2234
$

我通常会for (i = 0; i < N; i++)在for循环中使用——它比 C 更惯用for (i = 1; i <= N; i++)。这些函数是静态的，因为我使用了坚持在定义或使用函数之前声明函数的编译标志：

gcc -O3 -g -std=c99 -Wall -Wextra -Wmissing-prototypes -Wstrict-prototypes -Wold-style-definition ak.c -o ak

因为我是这样编译的，所以我不认为演员阵容malloc()是个问题。那些抱怨演员表的malloc()人主要关心“如果malloc()没有声明会发生什么”，但这已经是我编译时的警告（我实际上将其视为错误；我会添加-Werror以确保它正式成为错误)，所以这对我来说不是问题。（显示的代码也可以编译为有效的 C++ 程序，因为存在转换；如果缺少转换，它就不是有效的 C++ 程序。它不是一个好的 C++ 程序；它是一个 C 程序。）我用V-function调试N-function，我把V-function移到了N-function上面。

于 2013-05-05T15:11:37.777 回答