質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
C++11

C++11は2011年に容認されたC++のISO標準です。以前のC++03に代わるもので、中枢の言語の変更・修正、標準ライブラリの拡張・改善を加えたものです。

C++

C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

Q&A

解決済

3回答

5056閲覧

C++のstd::shared_ptr(or std::weak_ptr)とthisでのバグ

Paalon

総合スコア232

C++11

C++11は2011年に容認されたC++のISO標準です。以前のC++03に代わるもので、中枢の言語の変更・修正、標準ライブラリの拡張・改善を加えたものです。

C++

C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

0グッド

1クリップ

投稿2016/10/10 23:59

親子関係を持つようなクラスを作成したいと思い、次のようなクラスを書きました。

C++

1#include <vector> 2#include <memory> 3 4class Container { 5public: 6 std::vector<std::shared_ptr<Container>> children; 7 std::weak_ptr<Container> parent; 8 Container() { 9 } 10 ~Container() = default; 11 void addChild(std::shared_ptr<Container> child) { 12 children.push_back(child); 13 child->parent = this; // 怒られる 14 } 15};

しかし、std::shared_ptr<Container> child の指している Containerstd::weak_ptr<Container> parentaddChild を呼び出した std::shared_ptr<Container> を代入するつもりで書いた child->parent = this で怒られてしまいます。thisContainer *型だから怒られるのでしょうか?良く分かりません。助言をお願いします!!

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答3

0

ベストアンサー

thisポインタからstd::shared_ptr型(やstd::weak_ptr型)を取り出す場合は、std::enable_shared_from_thisクラステンプレートを利用してください。同クラスからCRTPの形でpublic継承し、shared_from_this()メンバ関数を呼び出します。

C++

1#include <cassert> 2#include <vector> 3#include <memory> 4 5class Container 6 : public std::enable_shared_from_this<Container> { 7public: 8 std::vector<std::shared_ptr<Container>> children; 9 std::weak_ptr<Container> parent; 10 Container() {} 11 ~Container() = default; 12 void addChild(std::shared_ptr<Container> child) { 13 children.push_back(child); 14 child->parent = shared_from_this(); 15 } 16}; 17 18int main() 19{ 20 auto p = std::make_shared<Container>(); 21 auto c1 = std::make_shared<Container>(); 22 auto c2 = std::make_shared<Container>(); 23 p->addChild(c1); 24 p->addChild(c2); 25 assert(p.use_count() == 1); 26}

なおshared_ptr<Container>(this);として強引にshared_ptr型を生成するのはNGです。型変換によりコンパイラはごまかせますが、このshared_ptrオブジェクトのスコープが切れたときに意図せずデストラクタ~Container()を呼び出し、thisオブジェクトが破壊されてしまいます。

投稿2016/10/11 02:53

編集2016/10/11 03:01
yohhoy

総合スコア6189

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

yohhoy

2016/10/11 03:11 編集

絶対的な指標ではありませんが、質問にあるような親子関係をもつデータ構造の場合、子から親を指すポインタは生ポインタでもよいかもしれません。というのも、親オブジェクトのライフサイクルが、子オブジェクトが親オブジェクトを指す区間を*必ず*包含すると保証でき、複雑な所有権管理を必要としないためです。(もちろん、"安全"のためにweak_ptrを使う実装がダメという訳ではありません。)
Chironian

2016/10/11 03:23

yohhoyさん。 なるほど、ちゃんと用意されているのですね。これは知りませんでした。ありがとうございます。
guest

0

こんにちは。

this は Container *型だから怒られるのでしょうか?

その通りです。std::weak_ptr<Container>型のparentにContainer*型のthisを代入しようとしています。
std::weak_ptr<T>の代入演算子(operator=)にはstd::weak_ptr<T>かstd::shared_ptr<T>を受け取るものしか定義さていません。
従って、「対応する関数無し」になります。

対処方法は意外に難しいと思います。
構造的にツリー構造と思いますので、thisもstd::shared_ptr<>で管理されているのではないでしょうか?
その場合、parentはthisを管理しているstd::shared_ptr<>から作ったstd::weak_ptrを渡す必要が有る筈です。それってthisの親が管理しているので受取りは難しそうです。
どこかにデザイン・パターンとしてありそうな気もしますが、私は見たことはないです。

しかし、parentをContainerへの生のポインタとして定義すれば簡単ですね。
通常のツリー構造であれば、親がdeleteされる時は先にchildの方がdeleteされますので、生ポインタでも問題ないように思いますが、ちょっと自信ありません。

投稿2016/10/11 03:17

編集2016/10/11 03:32
Chironian

総合スコア23272

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Chironian

2016/10/11 03:19

yohhoyさんの回答がついているのを確認せずに投稿してしまいました。ごめん。
guest

0

cpp

1child->parent = std::shared_ptr<Container>(this);

と書けばよいのではないでしょうか。
意味は以下と同じです。

cpp

1child->parent = std::weak_ptr(std::shared_ptr<Container>(this));

投稿2016/10/11 00:57

pebble8888

総合スコア390

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

pebble8888

2016/10/11 05:11

thisに対してはダメなんですねえ。 勉強になりました。 yohhoyさんの方法を使ってください。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問