スマートポインタの実引数にthisポインターを使用する方法

Question

現在DxLibを用いてゲームのフレームワークを作っています。

#include "Actor.h"
#include <memory>

void Actor::draw() {
	DrawGraph(position_.x, position_.y, renderer.GetTextureHandle(texture_), TRUE);
}
void Actor::react(std::shared_ptr<Actor> other) {}

bool Actor::is_collide(std::shared_ptr<Actor> other)const {
	DXrect my_rect{ position_.x, position_.y, position_.x + size_.x, position_.y + size_.y };
	DXrect other_rect{ other->position_.x, other->position_.y,
					other->position_.x + other->size_.x, other->position_.y + other->size_.y };

	return CollisionRectAndRect(&my_rect, &other_rect) == DX_TRUE;
}

// 衝突処理
void Actor::collide(std::shared_ptr<Actor> other) {
	if (is_collide(other)) {
		react(other);


// ここが問題の場所===================================================================
		other->react(*this);
// =================================================================================
	}
}

Actorクラスというゲームに登場する全てのキャラクターが継承するベースクラスのcppファイルです。

ここのreactに自分自身のポインターを渡したいのですが、この方法ではエラーになってしまい困っています、、、。

何かいい方法はないでしょうか。

Answer 1

こんにちは。

reactは本当にstd::shared_ptrを受け取らないとダメなのでしょうか？
その使い方の場合、当たり前ですがcollideから戻ってくるまでthisが破棄されないことをcollideの呼び出し側で保証する必要があります。従って、このケースのreactはstd::shared_ptrで受け取って破棄されないことを保証する必要は無いのではないでしょうか？
であれば、生ポインタ（Action*）を受け取るprivateなreact_impl的な関数を作って、その中で実処理を行い、react_implの呼び出し側で生ポインタが破棄されないことを保証すればクリアできると思います。

ところで、メモリに関しては、誰がいつ開放するのかきちんと設計しないと確実にハマりますので、安易にstd::shared_ptrを使うのはリスキーと思います。（特にstd::shared_ptrは意外にハマりポイントが多いですよ。）
例えば、ツリー構造を実装する場合など、どうしてもthisからstd::shared_ptrを作りたいケースはありますので、そのような時はyumetodoさん回答のshared_from_thisが有用です。

Comments

[suittizihou]

具体的な解決方法、並びに参考サイトなどなど、本当にありがとうございます。

Answer 2

shrared_from_thisをググってみましょう。きっとあなたの求めているものです。

Comments

[suittizihou]

ありがとうございます。
この部分のエラーはなくなったのですが、別のところからエラーが出てしまったのでまた色々調べようと思います。