[C++] Pimplイディオム [コードデザイン]

Question

こんばんは，みなさんはAPIやその他配布物を作成されるとき，Pimplイディオムを使用されていますか？
また使用時は，Implに隠ぺいする情報には何を選択していますか？

例）
1．プライベートメンバすべて
2．プライベートメンバ変数のみ
3．プライベートメンバメソッドのみ

別の質問になりますが，まず次のコードを見ていただきたいです．
※ 疑似コードよりなので名前や引数，処理は適当です．

// A.h
class A{
private:
class Impl;
Impl mImpl;

public:
// method
};

// A.cpp
class A::Impl{
public:
char *mData;
int mSize;

Impl()
: mData(nullptr),
mSize(0)
{}
};

A::A( /* args... */ )
: mImpl( new A::Impl() )
{
// process...
mImpl->mSize = input.getDataSize();
mImpl->mData = new char[ mImpl->mSize ];
// process...
}

A::~A()
{
delete mImpl;
mImpl = nullptr;
}

このとき，mImpl->mDataのdeleteはclass Implのデストラクタで行うべきなのか，class Aのデストラクタで行うべきなのかどちらなのでしょう？
いまはclass Implでdelete[] mImplと記述してますが，heapメモリの確保はclass Aで行っているのでなんだか気持ち悪いというのがあります．

Answer 1

Pimplイディオム自体は廃れていないと思います。書籍「C++のためのAPIデザイン」でも紹介されています。

ちなみにPimplイディオムを採用するならば、A::Implクラスのデータメンバ(mData,mSize)管理も、外側のAクラスではなくA::Implクラス自身に行わせた方が良いのではないでしょうか？
公開クラスA自身はデータを持たず、公開API呼び出しをうけて実装クラスA::Implに委譲するだけ、という構造が基本パターンだと思います。（それ以外はダメという訳ではないですが、教科書的にはこのような実装パターンです）

このとき，mImpl->mDataのdeleteはclass Implのデストラクタで行うべきなのか，class Aのデストラクタで行うべきなのかどちらなのでしょう？

上記の設計を前提として、「Implのデストラクタで行うべき」だと思います。AとA::Implのように密接に関連したクラスであっても、あるクラス（A::Impl）が確保した動的メモリの解放処理を、他クラス（A）に任せる設計は適切ではありません。

Answer 2

APIやその他配布物を作成されるとき，Pimplイディオムを使用されていますか？
Implに隠ぺいする情報には何を選択していますか？

ライブラリを作成する際に、OS依存コードや、他ライブラリのヘッダを隠蔽するために使用していました。

mImpl->mDataのdeleteはclass Implのデストラクタで行うべきなのか，
class Aのデストラクタで行うべきなのかどちらなのでしょう？

カプセル化の観点から、newを実行したクラスが、deleteすると思います。
上記コードであれば、Aのコンストラクタで、 mImpl->mData のnewを行っているため、
Aでdeleteを行うと思います。

ただ、mImpl->mData のnew/deleteは、Implで行うほうが良いとは思います。