Vectorクラスのoperator =を作る方法は？

Question

私は分かりませんが、コピーコンストラクタや演算子=が正しくありません。 2つの "墓"でテストしたところ、プリンタは動作していましたが、プログラムの終わりにコンパイラは "デバッグアサーションが失敗しました"と言っていました。コメントを1として

#pragma once 
#include <cstdio> 
#include <cmath> 
#include <iostream> 
#include <cstdlib> 

class tomb { 
private: 
    double *adat; 
    int szam; 
public: 
    tomb(){ 
     adat = NULL; 
     szam = 0; 
    } 
    tomb(const tomb &u) { 
     adat = u.adat; 
     szam = u.szam; 
    }; 
    int meret()const { 
     return szam; 
    } 
    ~tomb() { 
     delete[] adat; 
    } 
    double & operator[](int n) { 
     return adat[n]; 
    } 
    const double & operator[](int n)const { 
     return adat[n]; 
    } 
    const tomb &operator=(const tomb &a) { 
     adat = a.adat; 
     szam = a.szam; 
     return *this; 
    } 
    tomb elso_valahany(int n) { 

    } 

    void push_back(const double &a) { 
     double *tmp; 
     tmp = new double[szam+1]; 

     for (int i = 0; i < szam; i++) 
     { 
      tmp[i] = adat[i]; 
     } 

     tmp[szam] = a; 
     delete[] adat; 
     adat = tmp; 
     ++szam; 
    } 
    void Kiir()const { 
     for (int i = 0; i < szam; i++) 
     { 
      std::cout << adat[i] << "\n"; 
     } 
    } 
}; 

Answer 1

、私は深いコピーを実行する方法を紹介します：あなたは、クラスをコピーするたびに、あなたの代わりに、単にデータへのポインタが、全体のベクトルをコピーすることはありません。

また、簡略化のために、私はSTDを使用します::ベクトル：

#pragma once 
#include <cstdio> 
#include <cmath> 
#include <iostream> 
#include <cstdlib> 
#include <vector> 

class tomb { 
private: 
    std::vector<double> adat; 
    int szam; 
public: 
    tomb(){ 
     szam = 0; 
    } 
    tomb(const tomb &u) : adat(u.adat), szam(u.szam) 
    { 
     adat = u.adat; 
     szam = u.szam; 
    }; 
    int meret() const { 
     return szam; 
    } 
    ~tomb() { 
    } 
    double & operator[](int n) const { 
     return adat[n]; 
    } 
    const double & operator[](int n) const { 
     return adat[n]; 
    } 
    tomb& operator=(const tomb &a) { 
     adat = a.adat; 
     szam = a.szam; 
     return *this; 
    } 


tomb elso_valahany(int n) { 

    } 

    void push_back(const double &a) { 
     adat.push_back(a); 
     ++szam; 
    } 
    void Kiir()const { 
     for (int i = 0; i < szam; i++) 
     { 
      std::cout << adat[i] << "\n"; 
     } 
    } 

私がテスト/コンパイルしていませんが、メモリ管理がで行われているように、それは、今も元気でなければなりません標準コピーコンストラクタ！