コピーコンストラクタでエラーが生じて進まない

Question

やりたいこと

九九の計算をobj1の中の配列に格納して、obj2に深いコピー（コピーコンストラクタ）を用いてコピーする

困っていること

コピーコンストラクタのところでエラーが生じて進まない。
コピーコンストラクタをコメントアウトした場合はきちんと動作した。
コピーコンストラクタを呼び出すところ”kuku_array obj2 = obj1;”においてコピーコンストラクタの中身までは言っていることが確認できたが（yesの出力があった）for文に入ったところで止まってしまっている。
newで作成した2次元配列の開放の仕方が分からない。（今のやり方だと1次元のみな気がする）

エラーコード

exited segmentation fault

#include <iostream>	// coutのため
#include <iomanip>	// setwのため
using namespace std;


class kuku_array {
// write here...
public:
  int **n;
  int i;
  int j;
  kuku_array(){//九九を格納する2次元配列を作成するコンストラクタ
    n = new int*[20];
    for( int i = 0; i < 20; i++ ){
  		n[i] = new int[20];//この時点では空だから、mainの方で値を入れる
  }
}

  void kuku_in(int a,int b){
    n[a][b] = a*b;
  }

  kuku_array(const kuku_array &obj){//なぜ引数がポインタ型ののか？？？
cout << "yes" << endl;
    for(i = 1; i < 10; i++){
      for(j = 1; j < 10; j++){
        n[i][j] = obj.n[i][j]; //引数の配列をコピー
      }
      cout << i << endl;
    }
  }

  void kuku_show(){
    cout<<setw(2)<<"|"<<setw(3)<<1 <<setw(3)<<2 <<setw(3)<<3 <<setw(3)<<4 <<setw(3)<<5 <<setw(3)<<6 <<setw(3)<<7 <<setw(3)<<8 <<setw(3)<<9 <<endl;

      // 横線の表示
    cout<<"-------------------------------"<<endl;


      // 九九本体の表示

      for(i = 1; i < 10; i++){
        cout<<i<<"|";
        for(j = 1; j < 10; j++){
          cout<<setw(3)<<n[i][j];
        }
        cout <<endl;
      }
  }

  ~kuku_array(){
    cout << "Destructor is called." << endl;
    delete[] n;
    n = 0;
  }

};

int main() {

  // 九九オブジェクトobj1の生成
  // write here...
  kuku_array obj1;
  int i,j;
cout << "Hello!" << endl;


  // 九九を計算して、obj1の配列に入れる
  // write here...
  for( i = 0 ;i < 10 ; i++ ){
    for(j = 0 ; j < 10 ; j++ ){
      obj1.kuku_in(i,j);
    }
    cout << i << endl;
  }




  // 九九オブジェクトobj2にobj1の中身をコピーする
  // write here...
  obj1.kuku_show();

  kuku_array obj2 = obj1; //obj1の中身をobj2にコピーする



  // obj2に保持されている九九を表示する
  // write here...
  obj2.kuku_show();



  return 0;
}

実行結果

Hello!
0
1
2
3
4
5
6
7
8
9
| 1 2 3 4 5 6 7 8 9

1| 1 2 3 4 5 6 7 8 9
2| 2 4 6 8 10 12 14 16 18
3| 3 6 9 12 15 18 21 24 27
4| 4 8 12 16 20 24 28 32 36
5| 5 10 15 20 25 30 35 40 45
6| 6 12 18 24 30 36 42 48 54
7| 7 14 21 28 35 42 49 56 63
8| 8 16 24 32 40 48 56 64 72
9| 9 18 27 36 45 54 63 72 81
yes

よろしくお願いいたします。

Comments

[y_waiwai]

エラーが出るならエラーメッセージを提示しましょう

[naka]

exited segmentation fault　です

Answer 1

   kuku_array(const kuku_array &obj){//なぜ引数がポインタ型ののか？？？
 cout << "yes" << endl;
+    kuku_array a;
     for(i = 1; i < 10; i++){
       for(j = 1; j < 10; j++){
-        n[i][j] = obj.n[i][j]; //引数の配列をコピー
+        a.n[i][j] = obj.n[i][j]; //引数の配列をコピー
       }
       cout << i << endl;
     }
+    n = a.n;
+    a.n = nullptr;
   }

引数の obj はポインタではなく参照です。

デストラクタの delete[] も間違っていませんか？
new[] の回数と合いませんよ。

Comments

[naka]

ご指摘ありがとうございます。
delite[]のことなのですが。2次元配列を開放するためにはnewの時のようにfor文で開放するものなのでしょうか？

[kazuma-s]

new[] と同数の delete[] が必要です。
代入演算子も用意しないと、n だけのコピーとなって、代入先の元のデータはメモリリークし、
さらに代入元ののデータをデストラクタで二重に delete してしまいますよ。
メンバの i と j は必要ですか？ メンバ関数のローカル変数にしたほうが良いでしょう。

Answer 2

kuku_array(const kuku_array &obj)

コピーコンストラクタ↑で必要な領域を確保せぬままナカミをコピーしていますから。

Comments

[naka]

領域の確保はコピーコンストラクタの中で行うものなのですね。
ありがとうございました。
うまくいきました。