动态内存删除功能

Question

这是我的第一篇文章，很抱歉，如果没有问好。

基本上我有动态内存的麻烦，我想知道这是否是我误解的概念，或功能中的至少一个。好的，所以我使用的是C++，我需要管理一个在主程序循环内改变大小的数组，但是当我尝试删除内存时，我总是收到一个堆错误。 （下面是我想要做的简化版）。

void main(void) 
{ 
    //Initialization 

    //main loop 
    while(true) 
    { 
    int* array; 
    function(&array); 

    printf("test %d",array[0]); //basically use the data 

    delete [] array; 
    } 
} 

//in separate file 

void function(**int val) 
{ 
    *val=new int[size of array] // i pass the size of the array... 
           //to the function as well 
    //fill the array with data 
    return; 
} 

好了，所以在此之后我就可以读取数据，因此必须连接到指针“数组”，但那么为什么它不会让我删除这些数据，就好像它已经被删除了？

任何意见将不胜感激thanx。

Answer 1

您的代码，在其当前状态，应该工作。然而，这是非常不好的做法，用new和delete这样，尤其是新/在不同的地方删除。

您应该使用std::vector代替：

// main function 
std::vector<int> array = function(); 

printf("test %d",array[0]); //basically use the data 

和你（）函数将是：

std::vector<int> function() 
{ 
    std::vector<int> val(size); 
    //fill the array with data 
    return val; 
} 

Answer 2

不是主要问题，但你确实有语法错在这里，

void function(**int val); 

应该是：

void function(int **val); 

但你并不需要一个双指针，你可以简单地传递指针按引用：

void function(int *&val); 

由此可见，你的程序应该是这样的：

int main() // main should return int 
{ 
    int *array; 
    function(array, 5); 

    printf("test %d", array[0]); 

    delete [] array; 
} 

void function(int *&val, int size) 
{ 
    val = new int[size]; 
} 

你也不需要while (true)循环。

Answer 3

你说的“任何建议”受到欢迎，所以这里是我的建议：

不要使用C风格数组摆在首位，这将不会是一个问题。使用vector代替：

#include <vector> 
#include <algorithm> 

void main(void) 
{ 
    //Initialization 

    //main loop 
    while(true) 
    { 
     std::vector <int> array; 
     function (array); 
     printf ("test %d", array[0]); 
    } 
} 

//在单独的文件

void function(std::vector <int>& vec) 
{ 
    vec.push_back (1); 
    vec.push_back (2); 
    // ...etc... 
} 

以上是使用vector而不是C风格的数组和动态内存管理基本的和幼稚的实现。有很多改进的机会，但你明白了。

Answer 4

另一个想法。通过引用传递事物的价值是大型对象的复制成本。用指针你不需要担心这一点。这个代码也可以像这样清理。

int main() // main should return int 
{ 
    int *array = function(SOME_SIZE); 

    printf("test %d", array[0]); 

    delete [] array; 
} 

int * function(int size) //Just return the pointer 
{ 
    int *temp = new int[size]; 
    return temp; 
} 

因为我们正在处理的指针...返回他们是不是什么大不了的，而且是典型的实践，当你需要动态分配对象的“转移”的所有权。这就是说，引用矢量或其他标准容器的帖子是最好的方法。避免使用新的和删除使代码更安全。