您好我正在开发基于struct的项目。保存我的结构的第一个地址时遇到问题。问题不在我的N功能它工作正常。我的问题是，在我的V函数中，它只打印我的最后一个结构列表。我希望它从我的结构中打印出我的所有信息，而不仅仅是最后一个。我希望自己能够理解
CODE：
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;
}
答案 0 :(得分：2)
问题在于你的'N功能';它不会创建一个列表，代码也不会调用它。
提供的代码无法编译：
pocet_zaznam=(pocet_enter+1)/7;jeden zaznam ma 7 casti
rewind(fr);i
jeden ... casti部分可能是没有评论标记的评论。 i似乎只是无关紧要。分析可以忽略这些问题。
但是，该函数似乎读取文件，计算其中的换行符数，然后重新读取该文件以加载数据。然而，它继续在相同的单个结构上进行写入，并且不会尝试分配新的结构或将当前的结构与先前的结构相链接。请注意，您的分配函数alokuj()未在您显示的代码中调用。
当最终调用'V-function'时，它必须处理的唯一数据是读取的最后一个条目，因为之前的条目已被覆盖。
您需要重新编写代码，为每个要读取的项目分配新结构，将它们全部收集到列表中，并且（可能）将列表的头部返回到调用代码。
您还应该检查可能失败的错误功能。你检查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 - 它比for (i = 1; i <= N; i++)更惯用于C。这些函数是静态的，因为我使用的编译标志坚持在定义或使用之前声明函数：
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函数来调试N函数，我将V函数移到N函数之上。