动态内存分配 - C++

Question

我正在自学 C++，目前正在学习动态分配内存。这是我目前正在使用的代码：

#include <iostream>

using namespace std;

int *memAdd(int* dyn_Point, int *lenPoint){
   int *new_Dyn_Point = new int[*lenPoint * 2];
   *lenPoint = *lenPoint * 2;
   for(int i = 0; i < *lenPoint; i++){
       new_Dyn_Point[i] = dyn_Point[i];
   }

   delete lenPoint;
   delete[] dyn_Point;
   return new_Dyn_Point;

}

int main(){

   int len = 2;
   int *lenPoint = &len;
   int current = 0;
   int val;
   int *dyn_Point = new int[len];


   cout << "Input a value for point 1: ";
   cin >> val;
   dyn_Point[current] = val;


   while(val > 0){
      current++;

      cout << "Input a value for point " << current+1 <<" (0 to exit): ";
      cin >> val;

      if(current+1 == len){
         *dyn_Point = *memAdd(dyn_Point, lenPoint);
         cout << len;
      }

      dyn_Point[current] = val;


   }

   for(int i = 0; i < len; i++){
     cout << &dyn_Point[i] << "\n";
     cout << dyn_Point[i] << "\n\n";

 }
 delete[] dyn_Point;

}

我的问题：当添加更多内存时，它是否必须增加某个值？

每当我的“len”变量中的值不是 2 时，我的程序将在我尝试分配更多内存或在分配更多内存并且必须第二次添加更多内存之后崩溃。

这是应该的样子还是我在这里完全遗漏了什么？

Answer 1

您的 while 循环需要 break

while() {

    //do your steps
    break;
}

在函数memAdd 中需要进行以下更改：

// *lenPoint = *lenPoint * 2; 

// 上面一行需要注释，否则会导致溢出来影响循环条件：

    for(int i = 0; i < (*lenPoint-1); i++){

// 纠正for循环以解决溢出问题

下面的删除是不必要的，因为你没有分配内存 使用这个变量

// delete lenPoint;

对于您的问题：添加更多内存时是否必须增加某个值？

在这方面没有硬性规定。每当需要更多内存时，std::vector 将其大小（内存分配）加倍。它与您的方法略有不同。在达到分配的上限之前，您将内存翻倍。

**Edit**

根据 OP 请求编译完整代码

#include <iostream>

using namespace std;

int *memAdd(int* dyn_Point, int *lenPoint){
   int *new_Dyn_Point = new int[*lenPoint * 2];
  // *lenPoint = *lenPoint * 2;
   for(int i = 0; i < (*lenPoint-1); i++){
       new_Dyn_Point[i] = dyn_Point[i];
   }

   //delete lenPoint;
   delete[] dyn_Point;
   return new_Dyn_Point;

}

int main(){

   int len = 2;
   int *lenPoint = &len;
   int current = 0;
   int val;
   int *dyn_Point = new int[len];


   cout << "Input a value for point 1: ";
   cin >> val;
   dyn_Point[current] = val;


   while(val > 0){
      current++;

      cout << "Input a value for point " << current+1 <<" (0 to exit): ";
      cin >> val;

      if(current+1 == len){
         *dyn_Point = *memAdd(dyn_Point, lenPoint);
         cout << len<<"\n";
      }

      dyn_Point[current] = val;
     break;

   }

   for(int i = 0; i < len; i++){
     cout << dyn_Point[i] << "\n";
     cout << &dyn_Point[i] << "\n\n";

 }
 delete[] dyn_Point;

}

Answer 2

“我的问题：当添加更多内存时，它必须增加某个值吗？”

当然，您必须使用这样的设计来管理内存分配。

特别是您应该遵守Rule of Three (Five)，并在重新分配内存以增加到必要数量时复制所有现有元素。

---

比自己做所有事情（这可能容易出错）更好的选择是使用

std::vector<int> dyn_point;

和/或你班上的同类。

内存管理由container implementations负责，您无需为此烦恼。