エラーコード[ 'Bullet &Bullet::operator =(const Bullet &)': 削除された関数を参照しようとしています。 ] というエラーの解決法が知りたい。

Question

提示コードの /////// コメント部の内部のコピーコンストラクタですがなぜ削除された関数を参照しようとしています。と表示されるのでしょうか？消していません。存在しているのですがどうすればいいんどえしょうか？

#include "../../Header/Game/Bullet.hpp"
#include "../../Header/Sprite.hpp"



// コンストラクタ
Bullet::Bullet(class Game* g,glm::vec2 pos,glm::vec2 vec) : Actor_2D()
{
	Owner = g;	//Game クラス

	mPosition = pos;	//座標
	mVector = vec;		//方向
	speed = 10;			//速度

//	mSprite = new Sprite(g, "Assets/Bullet.png");	// スプライトクラス
	mSprite = std::make_unique<Sprite>(g,"Assets/Bullet.png");	// スプライトクラス

	//mSprite = nullptr;
}

// 計算
void Bullet::Update()
{
	mPosition.x += mVector.x * speed;
	mPosition.y += mVector.y * speed;

//	printf("mPosition.x: %f\n", mPosition.x);
//	printf("mPosition.y: %f\n",mPosition.y);

}

// 描画
void Bullet::Draw()
{
	glm::vec2 size = mSprite->getSize();
	mSprite->DrawGraph(mPosition,glm::vec2(0,1),glm::vec2(size.x,size.y));
}


// 座標を取得
glm::vec2 Bullet::getPosition()
{
	return mPosition;
}

/////////////////////////////////////////////////////////////////////////////////////////////////////
// コピーコンストラクタ
Bullet::Bullet(const Bullet &b)
{
	//printf("Bullet コピーコンストラクタ\n");
	
	mPosition = b.mPosition;	//座標
	mVector = b.mVector;		//方向
	speed = b.speed;			//速度
	Owner = b.Owner;			//Game クラス
	
	//mSprite = new Sprite(b.Owner,"Assets/Bullet.png");	//スプライト
	mSprite = std::make_unique<Sprite>(Owner, "Assets/Bullet.png");	// スプライトクラス

	
}
//////////////////////////////////////////////////////////////////////////////////////////////////

// デストラクタ
Bullet::~Bullet()
{
	//printf("Bullet デストラクタ\n");
	//printf("アドレス: %x\n",mSprite);

    //delete mSprite;
    //mSprite = nullptr;
}

#ifndef ___BULLET_HPP_
#define ___BULLET_HPP_

#include "../Actor_2D.hpp"
//#include "../Sprite.hpp"
#include <iostream>


class Bullet : public Actor_2D
{
public:

	Bullet(class Game* g, glm::vec2 pos, glm::vec2 vec);	//コンストラクタ
	~Bullet();												//デストラクタ
	Bullet(const Bullet& b);									//コピーコンストラクタ

	void Update() override;	//計算
	void Draw() override;	//描画


	glm::vec2 getPosition();	

private:

//	class Sprite* mSprite;	//スプライト
	std::unique_ptr<class Sprite> mSprite;	//スプライト
	float speed;			
};


#endif

Answer 1

以前の質問から察すると、Bullet::mSprite を std::unique_ptr<Sprite> に変更したらエラーが出るようになったのですね?

今までコピー代入演算子がコンパイラによって自動生成されていましたが、上記の変更によりmSpriteがコピー不可になったので、自動生成がされなくなりました。
(ここで気付きましたが、以前の質問のヒープ破壊はコンパイラが自動生成したコピー代入演算子がmSpriteをコピーしたことによる多重開放が原因でしょうね)

対処は、コピー代入演算子を上手いこと実装するか、Bullet自体もコピー不可にすることです。

Comments

[退会済みユーザー]

ヒープ破壊なんて想像もしていませんでした。これからはスマートポインタを使うことにします。
ではnew delete はなんのためにあるのでしょうか？

[int32_t]

C++標準にスマートポインタが無かった時代もあったので、仕方がないのです。new/deleteはスマートポインタ自体を実装するためにも必要ですし。

業務でC++を書くときは可能な限りnew/deleteを使うのを避けますね。

Answer 2

コメント部の内部のコピーコンストラクタですがなぜ削除された関数を参照しようとしています。と表示されるのでしょうか？消していません。存在しているのですがどうすればいいんどえしょうか？

プログラムを見ているとないですね。
Bullet &Bullet::operator =(const Bullet &)という関数を追加して下さい。
それとコピーコンストラクタには必ずexplicitを付けるようにしましょう。

Comments

[退会済みユーザー]

std::unique_ptrをstd::shared_ptrにしたら動きました。！

[stdio]

そっちのエラーでしたか？
気が付きませんでした。やはりスマートポインタは悪い文明
早くデフォルトのポインタをスマートポインタにして欲しいですね。

[退会済みユーザー]

質問ですがexlipcit はこの場合　代入operatorとの区別を図るためでしょうか？

[stdio]

そういうことです。
引数が一つだけのコンストラクタ（コピーコンストラクタを含む）には必ずつけるように心がけましょう。
詳しくはこちらをどうぞ。
http://www7b.biglobe.ne.jp/~robe/cpphtml/html02/cpp02027.html