c++ - C++ 配列の初期化、グッドプラクティス

Question

これが間違っていないことを確認したいと思います:私は配列を初期化します

 double* lower = input->getLowerBox();

関数 getLowerBox() は double* を返します。それが正しいか？または、この方法で初期化する必要があります::

double* lower = new double[nbP];
for (int i=0;i<nbP;i++)
     lower[i]=input->getLowerBox()[i];

または getLowerBox への複数の呼び出しを避けるために、

double* lower = new double[nbP];
double* tmp = input->getLowerBox();
for (int i=0;i<nbP;i++)
     lower[i]=tmp[i];
delete[] tmp;

score 4 · Accepted Answer

2 ステップのレシピ:

score 0 · Accepted Answer

まあ、それはあなたが何をしたいかによります。新しいアレイが必要かどうか。

最初のスニペットは新しい配列を作成しないため、メモリ管理がより重要になります。

例えば：

double* lower = input->getLowerBox();
delete[] lower;

無効になる可能性がありますinput->getLowerBox()。または次のようなもの：

double* lower = NULL;
{
   Class input;
   lower = input->getLowerBox();
}
//...

デストラクタで配列の内容をクリアするlowerと、ダングリングポインタが作成されます。input

最後の 2 つのスニペットは、新しい配列を作成します。IMO の方が安全ですが、余分なメモリも使用します。

やりたいことに応じて、どちらも正しいです。どちらを選択する場合でも、必ず完全に文書化してください。

score 0 · Accepted Answer

複数の理由から、私は間違いなく最初のものを選びます。よりクリーンで、不要な呼び出しや変数の作成などが回避されます。「->」を使用している場合は、入力がポインターであることを確認してください。それ以外の場合は、「.」を使用してください。

score 0 · Accepted Answer

最初の方法は問題ありません。double へのポインターを返しますが、これは配列に関連していると想定しています (そうでない場合は、修正してください)。そうすることで、その配列の最初の要素を指しているので、インデックスを付けるlowerか、ポインター演算を使用して他の要素にアクセスできます。

編集: の定義を投稿して、getLowerBox()何をしようとしているのかがより明確になりますか?

score 0 · Accepted Answer

あなたは変化する能力を持っていますgetLowerBox()か？もしそうなら、ベクトルを返すように変更します。

実装によっては、所有できるポインターまたは内部静的 (悪いが可能性あり) へのポインターが返される可能性があるため、それが何をしているのかを知り、ポインターを保持するか、配列のコピーをそれぞれ取得することによって、それに応じて行動する必要があります。 .

制御できずgetLowerBox()、返される配列のサイズがわかっている場合は、それをベクターにコピーするのが合理的な考えです。

double* lower = input->getLowerBox(); 
vector<double> lowerV(lower, lower + N );

（N配列のサイズはどこですか-ところで、これはメモリからのものであり、コンパイルしていません。）

c++ - C++ 配列の初期化、グッド プラクティス