user-0 提问: 给刚用分配好的结构体一赋值就出错

回复:
user: user-1
content: 仅供参考：//将c:\\tmp文件夹下的所有文件的内容全部放到用malloc分配的内存中 #include <stdio.h> #include <stdlib.h> #include <string.h> #include <io.h> struct FB {     char fn[256];     size_t fl;     char *b;     struct FB *next;     struct FB *prev; } *fh,*fb,*ft; char ln[256]; char fpn[256]; FILE *af; FILE *f; int L,n; int main() {     system("dir /b /a-d c:\\tmp\\*.* >c:\\allfn.txt");     af=fopen("c:\\allfn.txt","r");     if (NULL==af) {         printf("Can not open file c:\\allfn.txt!\n");         return 1;     }     fh=NULL;     fb=NULL;     n=0;     while (1) {         if (NULL==fgets(ln,256,af)) break;         L=strlen(ln);         if ('\n'==ln[L-1]) ln[L-1]=0;         printf("read %s\n",ln);         strcpy(fpn,"c:\\tmp\\");         strcat(fpn,ln);         ft=(struct FB *)malloc(sizeof(struct FB));         if (NULL==ft) {             printf("Can not malloc ft!\n");             fclose(af);             return 2;//之前的malloc在main退出后由操作系统自动free         }         printf("ft[%d]==%p\n",n,ft);         strcpy(ft->fn,fpn);         f=fopen(fpn,"rb");         if (NULL==f) {             printf("Can not open file %s!\n",fpn);             fclose(af);             return 3;//之前的malloc在main退出后由操作系统自动free         }         ft->fl=_filelength(fileno(f));         ft->b=malloc(ft->fl);         if (NULL==ft->b) {             printf("Can not malloc ft->b!\n");             fclose(f);             fclose(af);             return 4;//之前的malloc在main退出后由操作系统自动free         }         printf("ft[%d]->b==%p\n",n,ft->b);         if (ft->fl!=fread(ft->b,1,ft->fl,f)) {             printf("fread error!\n");             fclose(f);             fclose(af);             return 5;//之前的malloc在main退出后由操作系统自动free         }         fclose(f);         ft->next=NULL;          if (NULL==fh) {             ft->prev=NULL;             fh=ft;         } else {             fb->next=ft;             ft->prev=fb;         }         fb=ft;         n++;     }     fclose(af);     printf("-----list-----\n");     for (ft=fh;NULL!=ft;ft=ft->next) {         printf("%8d %s\n",ft->fl,ft->fn);         if (NULL!=ft) fb=ft;     }     printf("-----free-----\n");     n--;     if (NULL!=fh) {         for (ft=fb->prev;NULL!=ft;ft=ft->prev) {             if (NULL!=ft->next->b) {                 printf("ft[%d]->b==%p\n",n,ft->next->b);                 free(ft->next->b);             }             if (NULL!=ft->next) {                 printf("ft[%d]==%p\n",n,ft->next);                 free(ft->next);             }             n--;         }         if (NULL!=fh->b) {             printf("ft[0]->b==%p\n",fh->b);             free(fh->b);         }         printf("ft[0]==%p\n",fh);         free(fh);     }     return 0; } //C:\tmp\tmp\Debug>dir /a-d c:\tmp // 驱动器 C 中的卷是 C_HD5_1 // 卷的序列号是 1817-D526 // // c:\tmp 的目录 // //找不到文件 // //C:\tmp\tmp\Debug>tmp //找不到文件 //-----list----- //-----free----- // //C:\tmp\tmp\Debug>dir /a-d c:\tmp // 驱动器 C 中的卷是 C_HD5_1 // 卷的序列号是 1817-D526 // // c:\tmp 的目录 // //2011-06-30  18:04            44,840 my_c.rar //2011-06-30  17:18             1,036 err.frm //2011-06-30  14:32            14,243 出租.txt //2011-06-28  12:08            23,681 MSDN98书签.txt //             4 个文件         83,800 字节 //             0 个目录 17,041,870,848 可用字节 // //C:\tmp\tmp\Debug>tmp //read my_c.rar //ft[0]==00421800 //ft[0]->b==00520068 //read err.frm //ft[1]==00421670 //ft[1]->b==0052AFC0 //read 出租.txt //ft[2]==00421530 //ft[2]->b==00378F28 //read MSDN98书签.txt //ft[3]==004213F0 //ft[3]->b==0052B3F8 //-----list----- // 44840 c:\tmp\my_c.rar //  1036 c:\tmp\err.frm // 14243 c:\tmp\出租.txt // 23681 c:\tmp\MSDN98书签.txt //-----free----- //ft[3]->b==0052B3F8 //ft[3]==004213F0 //ft[2]->b==00378F28 //ft[2]==00421530 //ft[1]->b==0052AFC0 //ft[1]==00421670 //ft[0]->b==00520068 //ft[0]==00421800 // //C:\tmp\tmp\Debug>

user: user-2
content: 引用 6 楼 liehu232 的回复:[quote=引用 2 楼 under_tackle 的回复:] [quote=引用 1 楼 liehu232 的回复:]         p2=(struct stu *)malloc(LENS);     p1->next=p2; 改到循环前面，要不然p1->next一直==NULL 抱歉，我不太清楚。这两句话改到哪个循环前面？[/quote]     while(p1!=NULL)       p1=p1->next; 这里运行完后，p1=NULL 所以要新分配一个结构体。  p2=(struct stu *)malloc(LENS);     p1->next=p2; 把这个移动到这个循环后面[/quote] 应该是p1=(struct stu *)malloc(LENS) p1->next=NULL: 而且后面        p2=(struct stu *)malloc(LENS);     p1->next=p2;     p3=p1;     p1=p2;     p1->next=NULL; 没有意义

user: user-2
content: 引用 2 楼 under_tackle 的回复:[quote=引用 1 楼 liehu232 的回复:]         p2=(struct stu *)malloc(LENS);     p1->next=p2; 改到循环前面，要不然p1->next一直==NULL 抱歉，我不太清楚。这两句话改到哪个循环前面？[/quote]     while(p1!=NULL)       p1=p1->next; 这里运行完后，p1=NULL 所以要新分配一个结构体。  p2=(struct stu *)malloc(LENS);     p1->next=p2; 把这个移动到这个循环后面

user: user-3
content: 楼主这各种不加大括号的习惯最好改一改，要不根本不知道你到底是程序写错了，还是就是只取下面一条语句

user: user-3
content: while(p1!=NULL) p1=p1->next; printf("请输入员工的员工号:"); 这个while循环判断有点问题，估计你是想移动到最后一个，需要 while(p1->next!=NULL)这个样子。

user: user-4
content: 你的程序刚开始 p1=head;没有问题，然后head=malloc(...)后，head的地址已经变了，但是p1还是指导head刚才的位置，这样p1的位置还是空的，这样引用p1->XXX的任何内容会导致错误。C语言里用指针和地址是经常容易导致错误的地方。

user: user-0
content: 引用 1 楼 liehu232 的回复:p2=(struct stu *)malloc(LENS);     p1->next=p2; 改到循环前面，要不然p1->next一直==NULL 抱歉，我不太清楚。这两句话改到哪个循环前面？

user: user-2
content: p2=(struct stu *)malloc(LENS);     p1->next=p2; 改到循环前面，要不然p1->next一直==NULL