段错误，请坛友帮忙看看

iloveoovv

#include <iostream> #include <string> using namespace std; class Expr; class Expr_node { &nbsp;&nbsp;&nbsp;&nbsp;public: &nbsp;&nbsp;&nbsp;&nbsp;    virtual void print(ostream&) const = 0; &nbsp;&nbsp;&nbsp;&nbsp;protected: &nbsp;&nbsp;&nbsp;&nbsp;    friend ostream& operator<<(ostream& o, const Expr_node& e) &nbsp;&nbsp;&nbsp;&nbsp;    { &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;    e.print(o); &nbsp;&nbsp;&nbsp;&nbsp;        return o; &nbsp;&nbsp;&nbsp;&nbsp;    }&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;    friend class Expr; &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;int use; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;Expr_node(): use(1){ }; &nbsp;&nbsp;&nbsp;&nbsp;    virtual ~Expr_node(){ }; }; class Expr { &nbsp;&nbsp;&nbsp;&nbsp;private: &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;friend ostream& operator<<(ostream& o, const Expr& e) &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;{ &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;e.p->print(o); &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;return o; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;} &nbsp;&nbsp;&nbsp;&nbsp;protected: &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;Expr_node* p; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;friend class Expr_node; &nbsp;&nbsp;&nbsp;&nbsp;public: &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;Expr(int); &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;Expr(const string&, Expr); &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;Expr(const Expr&); &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;Expr& operator=(const Expr&); &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;~Expr(); }; class Int_node: public Expr_node { &nbsp;&nbsp;&nbsp;&nbsp;public: &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;friend class Expr; &nbsp;&nbsp;&nbsp;&nbsp;    int n; &nbsp;&nbsp;&nbsp;&nbsp;    Int_node(int k): n(k){ } &nbsp;&nbsp;&nbsp;&nbsp;    void print(ostream& o) const &nbsp;&nbsp;&nbsp;&nbsp;    { o << n; } }; class Unary_node: public Expr_node { &nbsp;&nbsp;&nbsp;&nbsp;public: &nbsp;&nbsp;&nbsp;&nbsp;    friend class Expr; &nbsp;&nbsp;&nbsp;&nbsp;    string op; &nbsp;&nbsp;&nbsp;&nbsp;    Expr uni; &nbsp;&nbsp;&nbsp;&nbsp;    Unary_node(string a, Expr b):op(a), uni(b){ }; &nbsp;&nbsp;&nbsp;&nbsp;    void print(ostream& o) const &nbsp;&nbsp;&nbsp;&nbsp;    { &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;    o << "(" << op << uni << ")"; &nbsp;&nbsp;&nbsp;&nbsp;    } }; Expr::Expr(int n) { &nbsp;&nbsp;&nbsp;&nbsp;p = new Int_node(n); }; Expr::Expr(const string& op, Expr t) { &nbsp;&nbsp;&nbsp;&nbsp;p = new Unary_node(op, t); }; Expr::Expr(const Expr& t) { p = t.p; ++p->use;} Expr& Expr::operator=(const Expr& rhs) { &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;rhs.p->use++; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;if (--p->use == 0) &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;delete p; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;p = rhs.p; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;return *this; }; Expr::~Expr() { delete p;} int main() { &nbsp;&nbsp;&nbsp;&nbsp;string plus = "+"; &nbsp;&nbsp;&nbsp;&nbsp;Expr *t; &nbsp;&nbsp;&nbsp;&nbsp;t = new Expr(5); &nbsp;&nbsp;&nbsp;&nbsp;cout << *t <<endl; &nbsp;&nbsp;&nbsp;&nbsp;Expr s(*t); &nbsp;&nbsp;&nbsp;&nbsp;cout << Expr(plus,s )<< endl; &nbsp;&nbsp;&nbsp;&nbsp;return 0; }

eveson

错误在
cout << Expr(plus,s )<< endl;

iloveoovv

原帖由 eveson 于 2009-1-8 19:50 发表
错误在
cout  

#include   <string>   #include   <iostream>   using   namespace   std;   class   Expr;   class   Expr_node       {   public:     virtual void print(ostream&) const = 0; protected:     friend class Expr;     friend ostream& operator << (ostream& o, const Expr_node& e)       {         e.print(o);           return   o;       };     Expr_node():use(1){};       virtual   ~Expr_node(){};       int use;   //for   count   the   number;   };   class   Expr   {    protected:     friend   class   Expr_node;       friend   ostream&   operator<<(ostream&   o,   const   Expr&   t)     {         t.p->print(o);         return   o;     }   public:     Expr(int n);     Expr(const string& op, Expr t);     Expr(const Expr& t);     Expr&   operator=(const   Expr&   rhs);     ~Expr(); protected:     Expr_node *p; };      class Int_node: public Expr_node {     public:         Int_node(int k):n(k){};     protected:         void print(ostream& o) const         {             o << n;         };     private:         friend class Expr;         int n; };       class Unary_node:public Expr_node {     protected:         friend class Expr;         string op;         Expr opnd;         void print(ostream& o) const         {             o << "(" << op << opnd << ")";         };     public:         Unary_node(const string& a, Expr b):op(a),opnd(b){} };   Expr::Expr(int n){p = new Int_node(n);} Expr::Expr(const string& op, Expr t) {     p= new Unary_node(op,t); }   Expr::Expr(const Expr& t) {     p = t.p;      ++p->use; };   Expr& Expr::operator=(const Expr& rhs) {     rhs.p->use++;     if(--p->use == 0)       delete p;     p = rhs.p;     return *this; };   Expr::~Expr() {     if(--p->use == 0)       delete p; }; int   main(void)   {     string temp = "+";     Expr *t;     t = new Expr(5);     cout << *t <<endl;     Expr s(*t);     cout << Expr("-",s )<< endl;     return 0; }

一楼的代码运行时有段错误，同样的main下的代码，用这一段可以通过，请大家帮忙看看一楼的问题出在什么地方，谢谢了！

isscc

大致看了一下，应该是Expr::~Expr()函数有问题，你既然是用Expr_node：：use来维持一个引用计数，当然在delete一个Expr_node类型的变量的时候要判断引用计数是否为0，只有引用计数为0的时候才能正真的释放空间
Expr::~Expr()
{
    p->use--;
    if(!p->use){
        delete p;
    }
}
这样写就不会出现这个问题了，

这样改了之后在Expr& Expr:perator=(const Expr& rhs)函数里面应该也有点问题，个人感觉在这个函数里面不应该减少引用计数

isscc

没看下面的帖子，楼主自己已经改了

yangsf5

大段大段的代码只有有空的同志才会帮助你的。
也有即使有空也懒得帮你。

要学会自己调试。。
在gdb里bt一下看看最近调用的函数，再认真看看。（这只是其中的一点）