C++ポインタを介してネストされたオブジェクトのメンバーにアクセス

Question

スピードの理由から、私はポインタを使ってオブジェクトを渡してきました。しかし、オブジェクト内のオブジェクトのメンバ変数にアクセスしようとすると、プログラムがクラッシュしてしまいます。私が得意でないものがありますか？

/* 
ClassA 
- ObjectB 
    --Member X 
*/ 

class A 
{ 
    int x; 
} 

class B 
{ 
    A* obj; // initialized in constructor 
    B(); 
    func(); 
} 

B::B(std::vector<A> vectorA) 
{ 
    this->obj = &(vectorA.at(0)) 
} 

B::func() 
{ 
    A* pointerA = this->obj 
    std::cout << pointerA->x << std::endl; // causes crash 
} 

Answer 1

以前のポスターでは、コメントを正しく削除したため、コンストラクタに渡されたベクターが参照ではなく、したがってポインタがぶら下がっている状態で破棄されたことが指摘されました。

B::B(std::vector vectorA)

は

B::B(std::vector &vectorA)

する必要がありますあなたのようなもの見知らぬ人に感謝！

Answer 2

あなたはあまりにもこの1でセミコロンが欠落している

class A 
{ 
    int x; 
}; 

ことをセミコロン

  A* pointerA = this->obj; 

修正が欠落しています。私が得 エラー：

11：9：エラー：1：ISO C++のないタイプ[-fpermissive]

8と 'FUNC' の宣言を禁止エラー：新しいタイプのリターンに定義されなくてもよいですタイプ

8：1：注意：（おそらくセミコロンが 'B' の定義の後に欠落している）

14：1：エラー：のプロトタイプ 'BB :: funcが（）' 内の任意の一致しませんクラスB：

11：4：エラー：候補は：int B :: func（）

最初に修正してください。

Answer 3

Like @ user4581301は指摘しました：this->obj = &(vectorA.at(0))は非常に危険です。関数が終了するとすぐにポインタがぶら下がるように。

関数シグネチャB::B(std::vector<A> vectorA)を使用すると、関数が渡すベクトルのコピーが作成され、関数が終了すると破棄されます。これは、あなたが作成したポインタがぶら下がっていることを意味します。あなたが関数内で使用しているベクトルは、実際にあなたが引数として渡したものと同じである

B::B(std::vector<A> &vectorA) 

この方法：

は、私はあなたのような何かをするためのものだと思います。この方法では、渡すベクトルが破棄されない限り（または指摘されたようにサイズが変更されている限り）、ポインタは有効です。