C++でvoidからキャストされたポインタを削除しても安全ですか？

Question

は、私は公共のヘッダにそのような何かを持っていると言う：

class MyClass 
{ 
public: 
    MyClass(); 

protected: 
    virtual ~MyClass(); 

private: 
    void *pSomeInternalClass; 
} 

そして、ここでは私の実装です：

MyClass::MyClass() 
{ 
    pSomeInteralClass = (void *) new SomeInternalClass(); 
} 

MyClass::~MyClass() 
{ 
    SomeInternalClass *pTemp = (SomeInternalClass *) pSomeInternalClass; 
    delete pTemp; 
    pSomeInternalClass = NULL; 
} 

ボイドポインタの背後にある考え方は、私がdont'tということです含める必要がありますパブリックヘッダ内の内部クラスの定義。

私の質問は、私は安全な方法でvoidポインタを処理していますか？削除に問題はありますか？

ありがとうございます！

Answer 1

元のタイプにキャストするのは安全です。

しかし、これを行う必要はありません。あなたのパブリックヘッダにあなたの内部の型を前向きに宣言すれば、この厄介なキャストをする必要はありません。

class YourClass { 
    ... 
    private: 
    class Internal; 
    Internal *pInternal; 
}; 

Answer 2

パブリックヘッダーにクラスの定義を含める必要はありません。すなわち

class SomeInternalClass ; 
class MyClass 
{ 
public: 
    MyClass(); 

protected: 
    virtual ~MyClass(); 

private: 
    SomeInternalClass *pSomeInternalClass; 
}