帮偶改改这个C++程序!!!!

马鑫伟

#include<iostream.h>
float *fcin()
{
        static float *p;
        float a[9];
        for(int i=1;i<10;i++)
        {
                cin>>a;
        }
        return p=a;
}
float *fpaixu(float *pl)
{
        float a[9],max;
        float *pp;
        for(int i=0;i<9;i++)
        {
                a=*pl;
                pl++;
        }
        for(i=1;i<9;i++)
        {
                for(int j;j<i;j++)
                {
                        if(a[j]>a)
                        {max=a[j];a[j]=a;a=max;}
                }
        }
        return pp=a;
}
void fcout(float *pk)
{
        for(;*pk!='\0';pk++)cout<<*pk;
}
void main()
{
        float *pb,*pl;
        pb=fcin();
        pl=fpaixu(pb);
        fcout(pl);
} [em17] [em17] [em17] [em17] [em17]

xgreenclover

自己改，实在不行看看类似的例子，很好的锻炼机会……

xgreenclover

发现严重问题，你的数组a都是局部变量，在离开它所在的作用域时相应的内存空间会被释放。也就是a里的内容已经不受程序控制了，随时会被改写，函数所返回的地址p其实是一个非法地址。

应该定义全局的数组a或者在函数中使用new来为a分配空间。
一个函数是绝对不可以返回一个局部变量的地址的。

mousesure

楼上真热心呀，我一看没说明没注释，我就吃饭去了

马鑫伟

mousesure: Re:帮偶改改这个C++程序!!!!

楼上真热心呀，我一看没说明没注释，我就吃饭去了

以后一定加注释:那个问题我解决了,谢谢大家的支持!

fw8899

绝不要返回pointer或reference指向一个local stack对象，或返回reference指向一个heap-allocated对象，或返回pointer或reference指向一个local static对象而有可能同时需要多个这样的对象。

     ----effectiv C++ 3e 中文版第 21条