ポインタを戻り値とするメンバ関数

Question

クラスのメンバ変数としてポインタを定義します。
それを出力するメンバ関数を作りたいのですが、以下であっているのでしょうか。

class A{
private:
    double **ptr;
public:
   double **Output_ptr();
}

double A::**Output_ptr(){
   return **ptr;
}

Answer 1

return **ptr;

doubleを返してますね

Answer 2

これまでの回答と重なりますが、クラス内のポインタを、constもつけずに直接返してしまうと、取得した側で書き換え放題となってしまい、C++のオブジェクト指向の1要素であるカプセル化が完全に崩壊します。

まずは、「書き換え可能なポインタを返すこと」が本当に必要なのか、コードの構造レベルで吟味するほうがいいです。

（もちろん、正当な理由があるのなら致し方ない部分もあるとは思いますが、これまでの質問を見る限り、回避したほうが良いパターンだと判断します）

Comments

[the_hoots_under]

いつも回答ありがとうございます。
そう言われてしまうと、設計が間違っているのだろうなと思います。
大規模計算のために複数配列を用意せざるおえないために、いくつかのクラスで複数の配列を共有せざるおえない状況となっております。

コード自体は、関数の定義された分割ファイルを全て含めてもたかだか数千行と非常に小さなプログラムです。

初心者にとって、C++の大規模多次元配列の確保とポインタの扱いとクラス設計は難しいです。

[maisumakun]

> 初心者にとって、C++の大規模多次元配列の確保とポインタの扱いとクラス設計は難しいです。

だからこそ、ポインタを扱う部分はできるだけ局所的に封じ込めて、直接ポインタを扱う箇所を可能な限り少なくしていくほうがいいのです。

参照を使えば、C++のオブジェクトもポインタを使わずに共有できます。

Answer 3

合っているかどうかはコンパイルしてみたらわかるはずです。

#include <iostream>

class A {
private:
    double **ptr;
public:
    double **Output_ptr();
    A(double **p) : ptr(p) { }
};

double **A::Output_ptr() {  // ★
    return ptr;             // ★
}

int main()
{
    double d = 3.14159, *p = &d;
    A a(&p);
    std::cout << **a.Output_ptr() << std::endl;
}