标题:<自己找不到bug>手工编辑一个文本文件,文件中每行为一个整数.读取文件内容构 ...
只看楼主
feiyuanyong
Rank: 1
等 级:新手上路
帖 子:9
专家分:0
注 册:2013-11-24
结帖率:25%
已结贴  问题点数:15 回复次数:2 
<自己找不到bug>手工编辑一个文本文件,文件中每行为一个整数.读取文件内容构成链表.将链表按整数从小到大的顺序排序,按排序后的顺序将链表中的整数写
问题很奇怪, 主要问题是在这里p1->value=num;,可是不知道怎么回事
//=====================================================================
//文件名:
//功能说明:【文件综合操作】手工编辑一个文本文件,文件中每行为一个整数.
//读取文件内容构成链表.将链表按整数从小到大的顺序排序,按排序后的顺序将
//链表中的整数写入另一个文件保存.
//作者:
//时间:13.12.20
//备注:
//=====================================================================

#include<stdio.h>
#include<stdlib.h>
#define LEN sizeof(Node)

//=======创建结构体========
typedef struct Node
{
    int value;
    struct Node *next;
}Node;

//============================函数声明==================================
//===================================
//函数名:sort_list
//功能说明:对整数链表进行排序
//参数:TagNode *---头结点
//返回值:void
//===================================
void *sort_list(Node *head);

//========================================================
//函数名:free_node()
//功能说明:释放结点
//参数:Info*---头结点
//返回值:void
//=========================================================
void free_node(Node *head);

//===========================主函数测试=================================
int main()
{
    FILE *fp1,*fp2;
    int num;
    Node *head,*p1,*p2;

    //=====================打开文件==============================
    fp1=fopen("in.txt","r");
    if(fp1==NULL)
    {
        printf("文件无法打开!");
        exit(0);
    }
   
    //=====================操作文件==============================
    //创建链表

    //分配头结点空间
    head=p2=(Node*)malloc(LEN);
    p1=(Node*)malloc(LEN);

    p1=p2->next;

    fscanf(fp1,"%d",&num);
    while(!feof(fp1))
    {
        p1->value=num;
        p1->next=NULL;
        p2=p1;
        p1=(Node*)malloc(LEN);
        p1=p2->next;
        fscanf(fp1,"%d",&num);
    }
    free(p1);
    fclose(fp1);

    //排序
    sort_list(head);

    //新建文件
    fp2=fopen("out.txt","w");
    if(fp2==NULL)
    {
        printf("文件无法打开!");
        exit(0);
    }
    p2=head;
    p1=p2->next;
    num=p1->value;
    while(p1!=NULL)
    {
        fprintf(fp2,"%d",num);
        p2=p1;
        p1=p2->next;
        num=p1->value;
    }
    fclose(fp2);

    //=========================关闭文件==================================
    free_node(head);

    return 0;
}

//============================函数定义==================================
//===================================
//函数名:sort_list
//功能说明:对整数链表进行排序
//参数:TagNode *---头结点
//返回值:void
//===================================
void *sort_list(Node *head)
{
    Node *p1,*p2;
    int temp;
    p2=head->next;

    //检查链表是否为空
    if(p2==NULL)
    {
        puts("链表为空!");
        return 0;
    }

    //选择排序法
    while(p2->next!=NULL)
    {
        p1=p2->next;
        while(p1!=NULL)
        {
            if(p2->value > p1->value)
            {
                temp=p2->value;
                p2->value=p1->value;
                p1->value=temp;
            }
            p1=p1->next;
        }
        p2=p2->next;
    }
}

//========================================================
//函数名:free_node()
//功能说明:释放结点
//参数:Info*---头结点
//返回值:void
//=========================================================
void free_node(Node *head)
{
    Node *p1,*p2;
    p1=head;

    while(p1!=NULL)
    {
        p2=p1;
        p1=p1->next;
        free(p2);
    }
}
搜索更多相关主题的帖子: include 文本文件 手工 
2013-12-22 13:43
神机军师
Rank: 7Rank: 7Rank: 7
来 自:游鱼潜水
等 级:黑侠
威 望:2
帖 子:202
专家分:542
注 册:2013-12-21
得分:8 
1.malloc函数应该是包含在 malloc.h 文件里面。  上面应该加上 #include<malloc.h>
2.用typedef定义构造体,后面的名字最好换成别的如:NODE,不然容易混淆。
3.文件你使用了fp1=fopen("in.txt","r"); 所以前提 是你得有个文件,而且符合你所要求的字符类型的存储。不然,结果可能出错。
因为你用fopen("in.txt","r") 读取ASCII码 回车换行符算不算一个,具体忘了,这个你要考虑一下。
4.简单画了一下,你链表建立好像错误:
head=p2=(Node*)malloc(LEN);
    p1=(Node*)malloc(LEN);

    p1=p2->next;

    fscanf(fp1,"%d",&num);
    while(!feof(fp1))
    {
        p1->value=num;
        p1->next=NULL;
        p2=p1;
        p1=(Node*)malloc(LEN);
        p1=p2->next;
        fscanf(fp1,"%d",&num);
    }

    在p1=p2->next 这里   上面说了p1->next=NULL;p2=p1; 所以p2->next=NULL;于是p1=p2->next里面p1=NULL。
5.排序后面输入文件的循环语句里面:
p2=head;
p1=p2->next;
num=p1->value;
while(p1!=NULL)
    {
        fprintf(fp2,"%d",num);
        p2=p1;
        p1=p2->next;
        num=p1->value;
    }
这个第三行 num=p1->value  你丢失了头结点的num值。当然,可能是你建立链表的时候,头结点没有参与数据存储,
不过链表建立好像错误了,所以你改正后,这里再推敲一下。
另外,再循环语句里面:num=p1->应该放在p1=p2->next之前。因为,当开始p1->next=NULL时,经过循环以后,p1=p2->next
这时候p1=NULL,(&NULL)->value没有意义(这时候num的值不确定),当然,这个不影响结果,就是感觉怪怪的。


呃,我没有运行程序,要改的太多,啊。
我也是新手,希望互相讨论,共同进步啊~

未知令人期待!
2013-12-22 15:04
yuccn
Rank: 16Rank: 16Rank: 16Rank: 16
来 自:何方
等 级:版主
威 望:167
帖 子:6809
专家分:42393
注 册:2010-12-16
得分:8 
    head=p2=(Node*)malloc(LEN);
     p1=(Node*)malloc(LEN);
 
    p1=p2->next;

这个地方已经泄漏了一个内存了,因为p1 申请了一个内存,只和p1 有指向了p2->next;
而且,这个地方p1 实际上是指向了一个野指针。

之后 的fscanf(fp1,"%d",&num);
     while(!feof(fp1))
     {
         p1->value=num;

循环开始时候,p1->value 这样访问就直接崩溃了,因为p1 野指针。

我行我乐
我的博客:
http://blog.yuccn. net
2013-12-22 15:06



参与讨论请移步原网站贴子:https://bbs.bccn.net/thread-425539-1-1.html




关于我们 | 广告合作 | 编程中国 | 清除Cookies | TOP | 手机版

编程中国 版权所有,并保留所有权利。
Powered by Discuz, Processed in 0.606143 second(s), 8 queries.
Copyright©2004-2024, BCCN.NET, All Rights Reserved