std::vector配列でpush_backするとデストラクタが実行されてしまうがeraseして時にデストラクタを実行させたい

Question

提示コードのbulletクラスですがmain関数でpush_backするとデストラクタが動いてしまいeraseして時にデストラクタして欲しいのですが上手くs処理を実装出来ません。どうすればいいのでしょうか？

testクラスの変数aをがeraseを置いていないにも関わらずアクセスエラーを起こします。
自分のやりたいことはeraseするとtestクラスのa変数をdeleteするということをやりたいです。

#include "stdio.h"
#include "glm/glm.hpp"
#include <array>
#include <vector>
#include "conio.h"

class test
{
public:

	test() {

	};

	~test() {

	};

};

class Bullet {
public:

	Bullet(int r) {
		a = new test();
		printf("コンストラクタ\n");
	}

	~Bullet() {
		delete a;
		a = nullptr;
		printf("--- デストラクタ\n");
	}

private:

	test* a;

};


int main()
{
	Bullet b(1);

	std::vector<Bullet> bullet;
	bullet.push_back(Bullet(1));

//	bullet.erase(bullet.begin());






	int a = _getch();
	return 0;
}

Comments

[maisumakun]

何のためにそうしたいのですか？

[退会済みユーザー]

文章を修正しました。

Answer 1

vectorに突っ込むだの何だのを行うよりも前に，
まずはその class をちゃんと完成させましょう．

Bullet A(1);
Bullet B=A;

とか

Bullet A(1);
Bullet B(2);
B = A;

とかが，まともに（このBullet型にとっての「まとも」が何なのかは，あなたしか知らない）動くような実装にしましょう．
あるいは，このような操作を行えないような実装にしても良いでしょう．

Answer 2

バグですね。

class Bullet {
public:

    Bullet(int r) {
//        a = new test();
        printf("コンストラクタ\n");
    }

    ~Bullet() {
//        delete a;
        a = nullptr;
        printf("--- デストラクタ\n");
    }

private:

    test* a;

};

ポインタ操作をコメントアウトすれば正しくデストラクトされます。
ポインタのコピーにより多重deleteが発生しているように思われます。

[別解] push_back 改め emplace_back

int main()
{

    std::vector<Bullet> bullet;
    bullet.emplace_back(1);
    return 0;
}

Comments

[退会済みユーザー]

どうすればいいのでしょうか？

[episteme]

ポインタのコピーを避ける、もしくは複製する でしょうね。
解決法はいくつかあります。unique_ptr/shared_ptr を使うのもひとつの手。

Answer 3

testクラスの変数aをがeraseを置いていないにも関わらずアクセスエラーを起こします。

新しいインスタンスに対してaを確保するようなコピーコンストラクタを実装してください。

デフォルトのコピーコンストラクタは、ポインタもそのままコピーしますので、同じポインタが増えてしまうことになります。

Comments

[maisumakun]

push_back(一時オブジェクト)は、ムーブ代入演算子があればそれが使われますが、なければコピーコンストラクタが呼ばれます。

[退会済みユーザー]

新しいインスタンスに対してaを確保するようなコピーコンストラクタを実装してください。
とはどのような意味でしょうか？

[maisumakun]

コピーコンストラクタを実装して、その中でa = new test();を行ってください。

[maisumakun]

コピーにしろムーブにしろ、一時オブジェクトがデストラクタ送りになるのは避けられません。

[退会済みユーザー]

Bullet(const Bullet &obj)
{
printf("コピーコンストラクタ\n");
a = new test();
}
としましたが以前としてアクセスエラーが発生してしまいます。どうすればいいのでしょうか？

[maisumakun]

そもそも論ですが、Testをポインタで持たせる意味はどの程度あるのでしょうか？

[退会済みユーザー]

実際はSprite クラスで画像とかを持ったデータです。
上手く実装出来ないので最小環境にてテストしています。この場合意味はないですよね。

[maisumakun]

> 実際はSprite クラスで画像とかを持ったデータです。

参照メンバにする手もあります。

[退会済みユーザー]

なるほど。現在の場合はどうすればいいのでしょうか？

[maisumakun]

別な問題ではありませんか？コピーコンストラクタの追加で動くようになりました。

https://paiza.io/projects/JFv9A_Bg_QhofsN5KIYByw?language=cpp

[退会済みユーザー]

質問ですがなぜmain関数をこうするとデストラクタが二回呼ばれるのでしょうか？

int main()
{

std::vector<Bullet> bullet;
bullet.push_back(Bullet(4));

return 0;
}

[maisumakun]

Bullet(4)で生成した一時オブジェクトのデストラクタと、vectorに入った方のオブジェクトのデストラクタで2回です。

[退会済みユーザー]

vecotrに入った方とはどういう意味でしょうか？

[maisumakun]

書いてのとおりです。push_backはオブジェクトをコピー（あるいはムーブ）します。

[退会済みユーザー]

質問ですがもう少し順序を説明してもらってもいいでしょうか？

[maisumakun]

・Bullet(4)で生成される一時オブジェクト→この行内で寿命を迎えて破棄される
・push_backでコピーされてbullet[0]に入ったオブジェクト→bullet自体の寿命とともに破棄される

Answer 4

std::vector<>に入れるクラスは、コピー対応にするかムーブ対応にする必要があります。
Bulletをコピー・ムーブ対応にするコードは(私が書くのが)面倒なので、ムーブ対応のスマートポインタを使うのも手です。

class Bullet {
    Bullet(const Bullet&) = delete; // 意図せずコピーするのを防ぐ
    ...
};

    std::vector<std::unique_ptr<Bullet>> bullet;
    bullet.push_back(std::make_unique<Bullet>(2));