标题:C++中在用new分配类成员的类在运行阶段程序终止问题,请大家帮忙找找错误原因 ...
只看楼主
沿途有鬼
Rank: 1
等 级:新手上路
帖 子:68
专家分:0
注 册:2008-7-20
 问题点数:0 回复次数:5 
C++中在用new分配类成员的类在运行阶段程序终止问题,请大家帮忙找找错误原因,谢谢!
//头文件
#ifndef STOCK_H_H_
#define STOCK_H_H_
#include<iostream>
using std::ostream;
using std::istream;
class Stock
{
private:
    char * str;
    int shares;
    int len;
    double share_val;
    double total_val;
    void set_tot(){total_val=shares * share_val;}
public:
    Stock();
    Stock(const char * co,int n=0,double pr=0.0);
    ~Stock();
    void buy(int num,double price);
    void sell(int num,double price);
    void update(double price);

    Stock & operator=(const Stock & s);
    
    Stock & operator=(const char * s);
    
    const Stock & topval(const Stock & s)const;

friend ostream & operator<<(ostream & os,const Stock t);

};


#endif

//源文件
#include<iostream>
#include<cstring>
#include"stock.h"
//using std::ostream;
//using std::istream;
    
Stock::Stock()
{
    len=4;
    str=new char[1];
    str[0]='\0';
    shares=0;
    share_val=0.0;
    total_val=0.0;
    
}
Stock::Stock(const char * co,int n,double pr)
{
    len=strlen(co);
    str=new char[len+1];
    strcpy(str,co);
    shares=n;
    share_val=pr;
    set_tot();
}

Stock & Stock::operator=(const Stock & s)
{
    if(this==&s)
        return *this;
    if(str!=0)
    delete [] str;
    else if(s.len>0)
    {
        len=s.len;
        str=new char[len+1];}
    else
    {std::cerr<<"len < 0 ";
    return *this;
    }
    if(str==0)
    {
        std::cerr<<"memor error!\n";
        return *this;
    }

     
    strcpy(str,s.str);
    shares=s.shares;
    share_val=s.share_val;
    set_tot();
    return *this;
}
    
Stock & Stock::operator=(const char * s)
{
    if(str!=0)
    delete [] str;
    len=strlen(s);
    if(len>0)
    str=new char[len+1];
    else
        return *this;
    if(str==0)
        return *this;

    strcpy(str,s);
    
    return *this;

}

Stock::~Stock()
{
    delete [] str;
}

void Stock::buy(int num,double price)
{
    if(num<0)
    {
        std::cerr<<"Number of shares purchased can't be negative. "
        <<"Transaction is aborted.\n";
    }
    else
    {
    
    shares=shares+num;
    share_val=price;
    set_tot();
    }
}
void Stock::sell(int num,double price)
{
    using std::cerr;
    if(num<0)
    {
        cerr<<"Number of shares purchased can't be negative. "
        <<"Transaction is aborted.\n";
    }
    else if(num>shares)
    {
        cerr<<"You can't sell more than you have! "
            <<"Transaction is aborted.\n";
    }
    else
    {
        shares=shares-num;
        share_val=price;
        set_tot();
    }
}

void Stock::update(double price)
{
    share_val=price;
    set_tot();
}
    
const Stock & Stock::topval(const Stock & s)const
{
    if(s.total_val>total_val)
        return s;
    else
        return *this;
}
ostream & operator<<(ostream & os,const Stock t)
{
    using std::endl;
    os<<"Conpany: "<<t.str
        <<" Shares: "<<t.shares<<endl
        <<" Share Price: $"<<t.share_val
        <<" Total Worth: $"<<t.total_val<<endl;

        return os;
}

//main()
#include<iostream>
#include"stock.h"

const int STKS=4;
int main()
{
    using std::cout;
    using std::ios_base;

    Stock stocks[STKS]=
    {
        Stock("NanoSmart",12,20.0),
        Stock("Boffo objects",200,2.0),
        Stock("Monolithic Objects",130,3.25),
        Stock("Fleep Enterprises",60,6.5)
    };

    cout.precision(2);
    cout.setf(ios_base::fixed,ios_base::floatfield);
    cout.setf(ios_base::showpoint);
    int st=0;
    for(st=0;st<STKS;st++)
        cout<<stocks[st];
    Stock top(stocks[0]);
    for (st=1;st<STKS;st++)
        top=top.topval(stocks[st]);//死在这个地方
    cout<<"\n Most valuable holding:\n";
    cout<<top;//这里没显示进过比较价格最高的股票
    return 0;//过了这里还有个终止报错
}

程序的运行结果是比较不出来最后的那个求出那个股票价格最高,但是算法应该没问题,在就是在显示完成之后出现终止运行问题,我想我关键是错在构造函数和operator=()深度赋值上,下面的关于判断内存申请是否成功之类的我也不知道写对了没有,麻烦各位帮我看看错在哪里,我自己看不出来了~
Stock & Stock::operator=(const Stock & s)
{
    if(this==&s)
        return *this;
    if(str!=0)//这个 str 判断它是不是空的不知道写对了没有?
    delete [] str;
    else if(s.len>0)
    {
        len=s.len;
        str=new char[len+1];}
    else
    {std::cerr<<"len < 0 ";
    return *this;//如果len<0应该返回什么呢?
    }
    if(str==0)//申请完内存,判断是否申请成功不知道写得是不是对的?
    {
        std::cerr<<"memor error!\n";
        return *this;//内存申请什么应该返回什么呢?
    }

strcpy(str,s.str);
    shares=s.shares;
    share_val=s.share_val;
    set_tot();
    return *this;
}


下面是我创好的VC工程,大家直接下载点击123Stock.dsw帮我看看错误,感激不尽啊~
123Stock.rar (867.09 KB)


[[it] 本帖最后由 沿途有鬼 于 2008-8-13 21:33 编辑 [/it]]
搜索更多相关主题的帖子: new 成员 运行 
2008-08-13 19:09
xlh5225
Rank: 2
等 级:论坛游民
威 望:2
帖 子:188
专家分:25
注 册:2007-8-14
得分:0 
你的程序错误太多了,由于时间关系,我在这里只给你指出一个,余下的还请你自己更正.
其实你的程序早在如下语句就已经错了:
  for(st=0;st<STKS;st++)
      cout<<stocks[st];
请你看看自己的输出符重载是怎么回事,申明如下:
  friend ostream & operator<<(ostream & os,const Stock t)
定义如下:
ostream & operator<<(ostream & os,const Stock t){
  using std::endl;
  os<<"Conpany: "<<t.str<<" Shares: "<<t.shares<<endl
  <<" Share Price:  $"<<t.share_val<<" Total Worth: $"<<t.total_val<<endl;
  return os;
}
请注意,你又是按值传递的,也就是说又会产生一个Stock的临时对象.现在让我们在看看发生了什么.你没有提供一个拷贝构造函数,所以编译器会为你生成一个拷贝构造函数,但是!这构用吗?会不会出错呢?
请注意,编译器为你生成的拷贝构造函数,只所谓的bitwise copy,也就是浅拷贝! 马上你就要大祸临头了!Stock类有一个成员是char *str,同时使用了动态内存.也就是在传递参数时,生成的临时对象会拥有"同样"一个指针str,它和被拷贝的str指针是指向同一个内存区域!好了.....不知道你想到问题出在那里没有???
呵呵,让我说完吧,不要说我烦哈~~~
在这个函数中,临时对象当然是会程序退出函数而消亡!也就是会自动地调用Stock的析构函数!
Stock::~Stock(){
  delete [] str;
}
啊!你居然把它释放了!
很明显,经过你的输出循环,数组stocks中的所有Stock对象的str指针都是"野的",出错啦!!!
修改方法,和我上次解答一样,请按引用传递!其实就算这样,也是不治本的啊!!


再次强调,这只是你程序的一处错误,你的错误还多!改了之后
for (st=1;st<STKS;st++)
  top=top.topval(stocks[st]);
第一次循环不会有问题了,但第二次还有问题,一样会死掉!
请自己解决吧~~~~
2008-08-13 22:10
xlh5225
Rank: 2
等 级:论坛游民
威 望:2
帖 子:188
专家分:25
注 册:2007-8-14
得分:0 
如若对上述错误有问题,请将Stock类的所有数据成员改为public,然后在main中:
 for(st=0;st<STKS;st++)
   cout<<stocks[st];
之后加上如下语句,看看是不是真的出问题了...而且是致命的问题!
 for(st=0;st<STKS;st++)
   cout<<stocks[st].str<<std::endl;

哎,我也累了,就说这么多,自己慢慢看吧,顺便一句,你的程序相当相当的不规范!
2008-08-13 22:14
zerocn
Rank: 1
等 级:新手上路
帖 子:126
专家分:0
注 册:2006-4-11
得分:0 
再次强调,这只是你程序的一处错误,你的错误还多!改了之后
for (st=1;st<STKS;st++)
  top=top.topval(stocks[st]);
第一次循环不会有问题了,但第二次还有问题,一样会死掉!


请问第二次的问题是不是自身赋值的问题,自身赋值会就什么后果?
2008-08-13 22:33
xlh5225
Rank: 2
等 级:论坛游民
威 望:2
帖 子:188
专家分:25
注 册:2007-8-14
得分:0 
建议你自己搞定它,只有在找问题过程中才参学到真东西...
另外,我个人感觉你还是基础东西不明白,很多C++核心的内容,你不了解,就比如说临时对象的产生问题及以构造函数的调用问题,如果没记错,你上次的问题实质上是一样的,你很多都不明白...要治本,还得你自己啊...我是帮不了你的.
再推荐举你一本书:effective C++看了它,在对C++的认识上,你应该会有很大的提高,不会再范同样的错误了...
2008-08-13 22:43
沿途有鬼
Rank: 1
等 级:新手上路
帖 子:68
专家分:0
注 册:2008-7-20
得分:0 
xlh5225 非常感谢你的解答啊,你解答的这么详细我一点都不介意,要是论坛能多有几个你这样的热心人就好了,再次谢过~!

[[it] 本帖最后由 沿途有鬼 于 2008-8-14 11:01 编辑 [/it]]
2008-08-14 10:54



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




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

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