プログラマ専用SNS ミクプラ

「クラスのコンストラクタで別クラスのポインタを渡し、コピーして使う」という使い方は、オブジェクト指向プログラミングとして正しいのでしょうか。

また、これが正しい場合コピーしたクラスのポインタはdeleteする必要はあるのでしょうか。

コード:

class A
{
public:
    void funcA(void)
    {
        // 省略
    }
};

class B
{
private:
    A *a;
public:
    B(A *a)
    {
        this->a = a;
    }
    ~B(void)
    {
        delete a;
    }
    void funcB(void)
    {
        a->funcA();
    }
};

すみませんでした。deleteの問題は自己解決しました。
ポインタのコピーはアドレスをコピーしているだけなので、
deleteすると他の場所で使えなくなるのでdeleteしてはいけませんね。

雨ガエルさんが書きました:「クラスのコンストラクタで別クラスのポインタを渡し、コピーして使う」という使い方は、オブジェクト指向プログラミングとして正しいのでしょうか。

「クラスのコンストラクタで別クラスのポインタを渡し、コピーして使う」こと自体は問題はありません。
たとえばJavaのBufferedReaderクラスのコンストラクタにはReader型のオブジェクトを渡します（Javaの場合は参照ですがポインタと同じようなものです）

オブジェクト指向な設計として適切かどうかは、クラスの詳細が分からないと議論できません。

雨ガエルさんが書きました:「クラスのコンストラクタで別クラスのポインタを渡し、コピーして使う」という使い方は、オブジェクト指向プログラミングとして正しいのでしょうか。

適切かどうかは場合によります。

雨ガエルさんが書きました: また、これが正しい場合コピーしたクラスのポインタはdeleteする必要はあるのでしょうか。

メモリ管理をポインタを持っているクラスに任せるという設計であれば、deleteする必要があります。
つまり、AクラスのインスタンスがBのポインタの独占的な所有権を持っている場合、ということになります。

ただし、Aのデストラクタでdeleteするとインスタンスをコピーしたときにコピー先のインスタンスがdeleteされたポインタを持ってしまうので、コピーコンストラクタでBのインスタンスを複製するか、参照カウント方式などで管理する必要があります。

シングルトンよりメンテナンス性が高く・可読性の高い良い使い方であるなら問題ないと思います。
実装でどうしようもない部分はあるので、理想のオブジェクト指向じゃないにしても後々困らなかればよいかなって感じで私は作っています。
※　オブジェクト指向を極めたわけでもなんでもないんですけどね。

インスタンスがnewで作られてない可能性も。

コード:

A a;
B b(&a);

beatleさん、h2so5さん、softya（ソフト屋)さん、ISLeさん、回答ありがとうございました。
クラスのポインタをコピーして使うのは問題ないのですね。
deleteすると思い通りの動作をしないことがわかりました。
ありがとうございました。解決しました。

プログラマ専用SNS ミクプラ

C++ クラスのコンストラクタで別クラスのポインタを渡して使う

C++ クラスのコンストラクタで別クラスのポインタを渡して使う

Re: C++ クラスのコンストラクタで別クラスのポインタを渡して使う

Re: C++ クラスのコンストラクタで別クラスのポインタを渡して使う

Re: C++ クラスのコンストラクタで別クラスのポインタを渡して使う

Re: C++ クラスのコンストラクタで別クラスのポインタを渡して使う

Re: C++ クラスのコンストラクタで別クラスのポインタを渡して使う

Re: C++ クラスのコンストラクタで別クラスのポインタを渡して使う