関数に渡された配列は、ナノ要素を与える（C++）

Question

私は配列を取り、元の配列のサブセットからなる新しい配列を定義する関数を持っています。

#include<iostream> 
#include<cmath> 
using namespace std; 
double * subarray(double *array){ 
     double *sub= new double[100]; 
     for (int i=0; i<10; i++){ 
     sub[i]=array[i]; 
     } 
    return sub; 
} 
int main(){ 
    double *x=new double[100]; 
    double *y=new double[10]; 

    for(int j=0; j<100; j++){ 
    x[j]=sin(j*3.14/2.0); 
    } 
    y=subarray(x); 
    for(int k=0; k<10; k++){ 
    cout<<y[k]<<endl; 
    } 
return 0; 
} 

このコードを実行すると、サブアレイの要素の一部がnanとして出力されます。

Answer 1

呼び出し側のサブアレイが新しいdouble [100]を作成し、それを戻しています。しかし、それはy = new double [10]とは全く別の配列です。あなたが「新」と呼ぶときはいつでも、それは新しいものを作り、他のすべての「新しい」ものに分かれています。 yは "new double [100]"のアドレスを取得しますが、メインで作成した "new double [10]"のアドレスのトラックを失います。これはメモリリークです。新たに割り当てられたアイテムへのポインタを返す場合は、nullptrとして格納するポインタを開始し、終了したらそれを呼び出すかどうかによって「delete」または「delete []」を呼び出します新しい[]というものか、まったく新しいものでしたか？

#include<iostream> 
#include<cmath> 
using namespace std; 
double * subarray(double *array) { 
    double *sub = new double[10]; // don't allocated more than you want to use 
    for (int i = 0; i < 10; i++) { 
     sub[i] = array[i]; 
    } 
    return sub; 
} 
int main() { 
    double *x = new double[100]; 
    double *y = nullptr; // don't set this if you're going to overwrite it 

    for (int j = 0; j < 100; j++) { 
     x[j] = sin(j*3.14/2.0); 
    } 
    y = subarray(x); 
    for (int k = 0; k < 10; k++) { 
     cout << y[k] << endl; 
    } 

    delete [] x; // if you make new things, delete them afterwards 
    delete [] y; 

    std::cin.get(); 
    return 0; 
} 

私はこれを実行すると、私はすべてのNaNを得ることはありませんが、あなたは関数内で、設定れることは決してありません多くのメモリを割り当てています。 100スペースを割り当てていますが、10に書き込むだけです。これは必ずしもエラーではありませんが、効率的ではありません。