クラスをprivateで継承したい

Question

前提・実現したいこと

基底クラスのデストラクタが仮想関数ではないため，クラスをprivateで継承したいです．
継承ではなくメンバ変数として持つことも考えましたが，基底クラスのメンバ関数をすべて再定義するのは避けたいです．
基底クラスは変更できませんのでご了承ください．
よろしくお願いします．

発生している問題・エラーメッセージ

error C2243: '型キャスト': 'B *' から 'const A &' の変換は存在しますが、アクセスできません。

該当のソースコード

class A {
private:
	int x;
public:
	A(const int &n) : x(n) {}
	int func() {
		return x;
	}
	int func(const A& a) {
		return x + a.x;
	}
};
class B : private A {
public:
	using A::A;
	using A::func;
};
int main() {
	B b(0);
	b.func();
	b.func(b); // error
	return 0;
}

Comments

[y_waiwai]

なぜ普通に継承するのではなくprivateの必要があるんでしょうか。

[CID8705]

基底クラスのポインタに派生クラスをnewするのを防ぐためです．
A *b = new B(0); // error

[y_waiwai]

なぜそれがダメなんでしょう
ムリなことをやろうとしてませんか

Answer 1

「BはAである」と言えない関係を作ったなら，A型の引数にBを渡そうというのが間違いではないでしょうか．
引数にBを受けたいなら，素直に引数の型がBであるメソッドを追加すれば良いのではないでしょうか．

class B : private A
{
  ...
public: //例えばこんなのを書く．
  int func( const B& b ){	return A::func( *this );	}
};

Answer 2

基底クラスにキャストしてはいけないのであれば、継承をやめて委譲を使ってみてはどうでしょうか。

委譲 - Wikipedia
【第5回】委譲 (1/4)：CodeZine（コードジン）

Comments

[CID8705]

オーバーライドして基底クラスのメンバ関数を呼び出すということでしょうか？

[Zuishin]

オーバーライドは継承の時に使う言葉なので、間違っていると思います。
リンクを二つ貼りましたから、とりあえず両方読んでみてください。

[dodox86]

Zuishinさんの「基底クラスにキャストしてはいけないのであれば」の記述が気になり、私自身が質問の意図を取り違えているかもしれないと思って本回答を避けました。すみませんがこちらに便乗させてください。

乱暴ですが、
> b.func((A&)b);
とか
> b.func(reinterpret_cast<A&>(b));

でコンパイルと実行は通りました。このようなコーディングを許さない、厳格なコードはどう書いたら良いか？と言う質問だったのでしょうか。＞s8079さん

Answer 3

やっぱり、私もコンポジション ( メンバ変数として保持すること ) するかなぁ。

継承ではなくメンバ変数として持つことも考えましたが，基底クラスの

それなら、Qtとかそういうライブラリによくある、

B obj;
A a = obj.a();
bool r = a.func1();

とかみたいにするとか、

B obj;
if( b.a()->func1() ) std::cout << "OK" << std::endl;

みたいに、B obj ( 質問者さんが作ろうとしているクラスのオブジェクト ) が持つ A a_; (質問でいう、基底クラス?) のアドレスを a() (いわゆるgetter) で返す。
すると、a_ の値を書き換えることができる。

そういう風に、コンポジションすることもできると思います。

基底クラスのポインタに派生クラスをnewするのを防ぐためです．

それなら単純に、AクラスとBクラスは別物として生成して、Aクラスのオブジェクトを生成させて、初期設定なんかは呼び出し元にさせて、Bクラスのオブジェクトを生成するときにコンストラクタで渡すとか。

[例]

A a;
a.setText( "C++ is cool!" );
a.setPen();
// ここでほかにも初期設定
B b( a ); // aオブジェクトを渡してB bのメンバ変数等にさせる

Answer 4

こんにちは。

BへのポインタをAへのポインタへキャストする（A* a = &b;）ことと、BのインスタンスをAの参照へキャストする（A& a = b;）は同等なので、b.func(b);はできません。

例えば、以下のようにすれば近いことはできます。

class A {
private:
    int x;
public:
    A(const int &n) : x(n) {}
    int func() {
        return x;
    }
    int func(const A& a) {
        return x + a.x;
    }
};
class B : private A {
public:
    using A::A;
    using A::func;
    A& base() { return *this; }
};
int main() {
    B b(0);
    b.func();
    b.func(b.base()); // error
    return 0;
}