紙上得來終覺淺,絕知此事要躬行 ---
今天對此話有了實際的領悟。之前學習C++的時候,自以為已經把深復制和復制構造函數等這些知識已經掌握了,但真正寫起項目來的時候,還是不能提前考慮這些問題,直到問題出現,然后慢慢找出原因才開始根據以前的知識修改代碼。
項目中使用到了這個弧形類
/*圓弧類*/ class Curve:public Graph {int pointCount;Point *points; public:Curve(Point *points,int pointCount);Curve(const Curve &_curve);~Curve();void draw(Graphics* graphics,Pen* pen); };
明顯points指針需要深復制,因此需要重寫copy constructor。但由于Curve類之前一直都是使用一次就析構,因此沒有重寫復制構造函數也沒問題。
/*元器件類*/ class Component {vector<Line> lines;vector<Curve> curves;vector<Circle> circles;vector<Text> texts; public:Component();//~Component();void pushLine(Line _line);//每次放入一條線void pushCurve(Curve _curve);//每次放入一條弧線void pushCircle(Circle _circle);//每次放入一個圓形void pushText(Text _text);//每次放入一段文本void draw(Drawer _drawer);//繪制這個元器件 };
后來,新增了一個元器件類封裝了這些圖形。然后就出錯了,思考后發現出錯的原因是:放入component之前的curve實例析構了一次,然后component析構是vector里面的curve又析構了一次,就是第二次析構的時候delete了一個已經釋放的內存空間,因此出現了運行時錯誤。
既然出現了問題,解決的方案也很明顯,重寫copy constructor進行深復制。