C++のポインタ配列

Question

コード

#include "stdafx.h" 
#include <iostream> 

void someFunc(double* pDoubleArray, int length) 
{ 
    double* pNewDoubleArray = new double[length]; 

    for(int i = 0; i < length; i++) 
    { 
     pNewDoubleArray[i] = i * 3 + 2; 
    } 

    pDoubleArray = pNewDoubleArray; 
} 
int main() 
{ 
    double dbls[] = { 1, 2, 3, 4, 5 }; 

    int length = sizeof dbls/sizeof dbls[0]; 

    std::cout << "Before..." << std::endl; 

    for(int i = 0; i < length; i++) 
    { 
     std::cout << dbls[i] << ", "; 
    } 

    std::cout << std::endl; 

    someFunc(dbls, length); 

    std::cout << "After..." << std::endl; 

    for(int i = 0; i < length; i++) 
    { 
     std::cout << dbls[i] << ", "; 
    } 

    std::cout << std::endl; 

    while(true){ } 

    return 0; 
} 

出力

Before... 
1, 2, 3, 4, 5, 
After... 
1, 2, 3, 4, 5, 

は、ここで私が何をしようとしていますものです： 1.配列を作成し、いくつかの値 2パスでそれを埋めます新しい配列を作成し、渡された配列を新しく作成された配列に再割り当てする関数へのポインタとしての配列 3.変更を印刷しますs

私はまだ変更が表示されていない、と私はなぜわからない。その結果、メモリリークの問題を無視

Answer 1

関数someFuncのインターフェイスが間違っています。新しい配列のアドレスを返すためには、ポインタのアドレス（またはポインタへのポインタ）の参照が必要です。それ以外の場合は、単にローカル値を変更するだけです。

int main() 
{ 
    double dbls[] = { 1, 2, 3, 4, 5 }; 
    double* pArray = dbls; 
    // ... 

    someFunc(pArray, length); 
    // ... 

    for(int i = 0; i < length; i++) 
    { 
    std::cout << pArray[i] << ", "; 
    } 
    // ... 
} 

Answer 2

：

Answer 3

void someFunc(double* & pDoubleArray, int length) 
// pass by reference ^^^ the pointer 

線pDoubleArray = pNewDoubleArray。ポインタ

どちらかのローカルコピーは、参照によってポインタを渡すことへのポインタを渡し、またはスタイルの問題だ新しい価値私の好みは、新しい値を返すようになり

、ビットを返し割り当てます。

Answer 4

古い配列をが使用しない関数に渡す理由は不明です。

個々の値を変更している場合は、新しいアレイインスタンスを作成する必要はありません。そうでない場合は、新しい配列を作成して返します。だから、

、いずれかの元の配列に変更：

void someFunc(double* pDoubleArray, int length) 
{ 
    for(int i = 0; i < length; i++) 
    { 
     pDoubleArray[i] = i * 3 + 2; 
    } 
} 

または関数から新しい配列を返す：

// this indicates that the returned value is 
// actually a new instance 
double* getNewArray(double* pDoubleArray, int length) 
{ 
    double* pNewDoubleArray = new double[length]; 

    for(int i = 0; i < length; i++) 
    { 
     pNewDoubleArray[i] = i * 3 + 2; 
    } 

    return pNewDoubleArray; 
} 

を代替は、参照することによって、入力配列を渡すことですが、それは解放する複雑に未使用のインスタンス。

[編集]

この最後の場合明確にする：私は前にコメントしてきたよう

void someFunc(double** pDoubleArray, int length) 
{ 
    double* pNewDoubleArray = new double[length]; 

    for(int i = 0; i < length; i++) 
    { 
     pNewDoubleArray[i] = i * 3 + 2; 
    } 

    *pDoubleArray = pNewDoubleArray; 
} 

void main() 
{ 
    double dbls[] = { 1, 2, 3, 4, 5 }; 
    double* pArray = dbls; 

    // this will change what pArray 
    // points to 
    someFunc(&pArray, 5); 

    return 0; 
} 

を、ヒープへpArray点が前の配列を割り当てた場合、後者のアプローチは、メモリリークにつながりますsomeFuncを呼び出してください。