正しく

Question

ポインタメンバ変数を設定し、私は次のように簡略化されたクラスL持っている：

Cが別のクラスである

class L 
{ 
    private: 
     C * _pc ; 
    public: 
     C * getc() const ; // getter 
     void setc(const C * ipc); //setter 
     ~L() 
     { 
      delete _pc ; 
     } 
}; 

を。

私はまた、次のように簡略化ヘルパークラスCHelperを持っている：

class CHelper 
{ 
    C _c ; 
    CHelper(L & ic) 
    { 
     // 1st constructs _c (code omitted); 
     // then sets ic's _pc pointer member variable : 
     ic.setc(&_c); 
    } 
}; 

私は確信してされることなく、何とか_pcの削除に問題があるだろうと感じています。それはどうですか？

このような承認の瑕疵は何ですか？正しいアプローチで同じ「機能」（ポインタメンバー変数を正しく設定する）にはどうすれば到達できますか？

Answer 1

あなたのコードは現在コンパイルされません。 CHelperのコンストラクタはCオブジェクトを要求します。 Cクラス（おそらく）にはsetcメソッドがありません。おそらくCHelper(L& ic)を意味するでしょう。

とにかく、あなたのCHelperオブジェクトが範囲外になり、破壊された場合、メンバー変数_cはそれと共に破壊されます。あなたのLオブジェクトにぶら下がっているポインタを残す。それだけでなく、Lも現在3/5のルールに違反しています。

あなたがC++ 11へのアクセス権を持っている場合は、私は非常にCHelperクラスが必要な場合std::shared_ptrと生のポインタ_pcと_cを交換することをお勧め（したがってそのデストラクタでdeleteの除去）し、そうでない場合はstd::unique_ptrでしょう。

Answer 2

アレイにdelete [] _pcを使用する必要があるため、ipcパラメータが配列のように割り当てるかどうかを確認してください。