我使用 int *p = new int[size]分配了一个动态内存;现在,当我尝试使用删除 [] p 删除它时;我在执行代码时遇到分段错误(核心转储)。
最初,我能够动态输入数组元素并且它正常工作。但是,在执行一定次数后,现在它说 分段错误。我有一个函数,在该函数范围的末尾使用 new 稍后分配内存,我包含了 delete [] p.我应该在主函数中包含 delete 吗?
#include<iostream>
using namespace std;
void input(){
int n,d, i= 0, count;
cout<< "number of variables: "<<" ";
cin>>n;
cout<<"enter number of minterms: "<<" ";
cin>>count;
int x = pow(2, n);
int *p = new int[count] ; //dynamic allocation
for(i = 0; i<count; i++)
{
cout<< "enter the minterms in decimal: ";
cin>>d;
p[i] = d;
}
for( i =0; i< count; i++){
cout<<p[i]<<" ";
}
delete [] p; //(Do I need to write delete over here or in the main
//func, Right now I've used it here(i.e,the user defined function.)
cout<<"successfully deallocated";
}
//Main function:
int main(){
int *p = NULL; //Is this required ?
input();
//delete[] p; (Do I need to mention delete over here?)
return 0;
}
number of variables: 4
enter number of minterms: 8
enter the minterms in decimal: 1
enter the minterms in decimal: 2
enter the minterms in decimal: 3
enter the minterms in decimal: 4
enter the minterms in decimal: 5
enter the minterms in decimal: 6
enter the minterms in decimal: 7
enter the minterms in decimal: 8
1 2 3 4 5 6 7 8 successfully deallocated00000000119614428832765154679997521907-10100852163265911961440643276540008000800080004000-1005...<a lot of numbers>...07370419492536907748609097595Segmentation fault (core dumped)
这段代码在 clang、g++ 和 vc++ 下工作正常。 示例:https://rextester.com/PBN39654
这让我认为构建代码的环境要么有问题,要么在 main 成功返回后触发了核心转储。您如何构建此演示?
但是,有些事情您可以改进。 例如:
- 不要手动使用 DMA...尝试偏袒
std::vector
- 始终初始化变量。未初始化的非静态局部(作用域)变量(如
n,d
和count
)将收到垃圾值。 x
不在任何地方使用。删除它。int *p = NULL
有几个缺陷,其中之一是NULL
.NULL
是要0
的宏。如果确实要创建不指向任何内容的指针,请使用nullptr
。其次,int *p
与函数中的指针无关,因此毫无用处。删除它。
下面是更新示例的演示:https://rextester.com/BUTV13117
我已经测试了您的代码,对我来说它运行没有问题。
如果你在main()中分配空间,那么你通常也应该在main()中释放它。
你在输入中创建的指针 p 与你在 main 中创建的指针无关。因此,当您回到主服务器时,您将无法访问在输入中制作的数组。
如果这是你想要做的,你"不能"不删除输入中的p,将其返回到main,在main中使用它,然后在main中删除它。但是,像这样拆分新内容和删除内容并不是最佳编码做法。
如果你不想在main中使用数组,你应该删除main函数中对p的任何引用,不需要将其设置为null,当然也不要删除它。