C++配列の初期化、良い練習

Question

私は、これは間違っていないことを確認してくださいしたいと思います：私は

double* lower = input->getLowerBox(); 

どこ機能getLowerBox（で配列を初期化すること）*一部のdoubleを返します。それが正しいか？それとも、私はこの方法を初期化SHOLD ::

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; 

Answer 1

2つのステップのレシピ：

1. 変更Input::getLowerBox()

Answer 2

まあ、それはあなたが何をしたいかに依存します。新しい配列が必要かどうか

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

：

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

はおそらく無効input->getLowerBox()をレンダリングします。か何かのように：inputはデストラクタでの配列の内容をクリアした場合

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

は、lowerダングリングポインタになるだろう。

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

あなたのやりたいことに応じてどちらも正しいです。どちらを選んでも、完全に文書化してください。

Answer 3

最初の方法は問題ありません。私は配列に関連していると仮定しているダブルへのポインタを返すつもりです（そうでなければ、私を修正してください）。これを行うことで、その配列の最初の要素を指しているだけで、lowerをインデックスするか、ポインタ演算を使用して他の要素にアクセスすることができます。

編集：getLowerBox()の定義を投稿して、それであなたがしようとしていることがより明確になりますか？

Answer 4

代わりにあなたが戻り値を変更したい場合はコピーを使用し、const参照std::vector<double> const&

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

実装によっては、自分が所有できるポインタや内部の静的なポインタ（悪いですが可能なもの）が返される可能性があります。そのため、ポインタを保持したり、コピーを取ったりして配列のそれぞれの配列。

あなたはgetLowerBox()を制御することはできません、あなたはそれを返し、配列の大きさを知っていれば、それは合理的な考えではNはの大きさである

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

（ベクトルにコピーします配列 - これはメモリからのものですが、私はそれをコンパイルしていません）。

Answer 5

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