C++ コピーコンストラクタの使用方法

Question

###前提・実現したいこと
showdataに値渡しが行われた際に、デストラクタが発生するのを防ぐために、コピーコンストラクタを実装しましたが、エラーが発生し解決できません。
エラー箇所を解決できないので、すいませんが教えてください。

###発生している問題・エラーメッセージ
エラー文は下記のものだけです。
Runtime error(Exit status:134(Abort signal from abort(3)))
###ソースコード

＜ヘッダー文＞
class Worker{
 public:
   int number;
   char* name;
   double salary;
   Worker();
  Worker(const Worker &obj);
   ~Worker();
};
＜メイン文＞
#include <iostream>
#include "Worker.h"
using namespace std;

Worker::Worker(){
 name = new char[80];
 cout<< " This is Constructor "  << "\n";
 strcpy(name, "undifined");
 number = 0;
 salary = 0;
}


Worker::Worker(const Worker &obj){
    cout<< " This is Copy Constructor "  << "\n";
    name = new char[80];
    name = obj.name;
    this->number = obj.number;
    this->salary = obj.salary;
};


Worker::~Worker(){
    delete[] name;
    cout << "デコンストラクタ" << "\n";
}


void ShowData(Worker w2){
   cout << "name = " << w2.name << "\n";
   strcpy(w2.name,"AVD");
   cout << "name = " << w2.name << "\n";
   cout << "number = " << w2.number << "\n";
   cout << "salary = " << w2.salary << "\n";
}

int main(){
    Worker w1;
    strcpy(w1.name,"Takayuki");
    w1.number =10;
    w1.salary = 100;
    cout << " Before shwodata" << "\n";
    ShowData(w1);
    cout << " After shwodata" << "\n";
    
    return 0;
}

###補足情報(言語/FW/ツール等のバージョンなど)
paiza.io

Answer 1

今回質問されているエラーについて何も記載されていませんので、怪しいところの指摘のみですが、コピーコンストラクタを書くとしたら

Worker::Worker(const Worker &obj){
    cout<< " This is Copy Constructor "  << "\n";
    name = new char[80];
    strncpy(name, obj.name, 80);
    this->number = obj.number;
    this->salary = obj.salary;
};

と書かないと、コピーされたオブジェクトを破棄する際に、もとのnameメンバも解放してしまいますよ。

あと、ちょっと苦言で申し訳ないですが、エラーの内容が判っているのは質問者さんのみで、ここを読んでいる人にはどういう操作をしたらエラーになった等の状況がさっぱりわかりません。エラーメッセージだけでなく、そういった情報もできるだけ書いてもらった方が的確な回答が集まりやすいです。

Comments

[saito.kaz]

今回、エラー文が表示されず、上記のエラーメッセージのところに記載してある文のみでした。お手数をおかけし、申し訳ありません。

[Chironian]

KoichiSugiyamaさんの回答でFinal Answerと思いますので、こちらで補足させて下さい。

TAKAYUKI_MIWAさん、先の質問からのトライですね。
下記はコンパイラが自動生成するものと事実上同じですので、コピーコンストラクタの生成をコンパイラに任せた場合と同じ結果になるのですよ。

> name = obj.name;

後、私のサンプル・コードでは初期化子を使ってました。まだC++初めて一週間の方に提示するのは適切でなかったかも。私が初期化子の存在を知ったのはC++を初めてたぶん10年後くらいだったような．．．ごめんなさい。

下記に初期化子を含めてC++のクラス定義について説明がありました。
http://homepage2.nifty.com/well/Class.html
中々良いサイトですので、時間がとれたら見てみることをお薦めします。

Answer 2

コピーコンストラクタの以下の実装は問題があります。

name = new char[80];
name = obj.name;

name = new char[80]
により、新しいメモリをヒープ領域から確保し、その先頭アドレスが name に入りました。

ところが、
name = obj.name

として、name で保持していたアドレス値が、コピー元のnameのアドレス値に上書きされてます。
※コピーコンストラクタで渡される obj は、コピー元のインスタンスです。

先ほど確保したメモリのアドレスが行方不明になってしまうのはわかりますか？
そのため、先ほど確保したメモリを解放（delete）する機会が無くなってしまいます。

このオブジェクトがメソッドに渡されるたびに、解放されることのないメモリが確保されていくので、
やがてメモリリークを起こすことになります。

さらに、
name = obj.name
としているため、この関数終了のときに呼ばれるデストラクタで、コピー元の name が指し示すメモリが解放されます。
mainロジック完了の際に、もともとのオブジェクトも、デストラクタにより同一の name が指し示すメモリを解放してしまうため、２重解放になってしまいます。

これを解決するために、KoichiSugiyamaさんがご提示されているようにしなければなりません。

name = new char[80];
strncpy(name, obj.name, 80);

name = new char[80]
によって、新しいメモリ領域を割り当てたあと、strcpy によって、コピー元のobjのname の内容を、新しいメモリ領域にコピーしています。
(strcpyはバッファ上限を指定しないため、セキュリティ上脆弱です。利用しないことが推奨されており、KoichiSugiyamaさんはstrncpyにされています）

name には、新しく割り当てたメモリのアドレス値が入っていることを確認してください。

このメソッドが完了した際に呼ばれるデストラクタでは、
delete name
によって、新しく割り当てたメモリ領域が解放されることになります。

ポインタまわりは、難しいかもしれませんが、必ず腑に落ちるまで学習しておいたほうが良いと思います。
私は、こちらの書籍で理解できましたので、ご参考までに紹介させていただきます。

C言語ポインタ完全制覇

Comments

[saito.kaz]

ありがとうございます。
とてもご参考になりました。