C++ 分配指针时内存泄漏
温度=p_长;显示内存泄漏。我不知道这是怎么回事内存泄漏C++ 分配指针时内存泄漏,c++,visual-c++,memory-leaks,C++,Visual C++,Memory Leaks,温度=p_长;显示内存泄漏。我不知道这是怎么回事内存泄漏 long *temp = NULL; for (int i = 1; i < 10; i++) { if (i < 3) { long *p_long = new long; if ( p_long ) { if ( 0 == *p_long ) { flag = true; } else if ( 1 == *p_long ) {
long *temp = NULL;
for (int i = 1; i < 10; i++) {
if (i < 3) {
long *p_long = new long;
if ( p_long ) {
if ( 0 == *p_long ) {
flag = true;
} else if ( 1 == *p_long ) {
temp = p_long; -----> showing memory leak here
continue;
}
}
}
}
if (temp)
delete temp;
long*temp=NULL;
对于(int i=1;i<10;i++){
如果(i<3){
长*长=新长;
如果(p_long){
如果(0==*p_长){
flag=true;
}else if(1==*p_long){
temp=p_long;------>此处显示内存泄漏
继续;
}
}
}
}
如果(临时)
删除临时文件;
谢谢。您没有释放堆分配
long*p_long=new long
考虑到这是一个for
循环,您将孤立一些内存块(new
ed,但不再有指针为delete
-ion引用它们。)
如果未将p_long
分配给temp
,则必须在循环的适当部分释放p_long
。您的continue
语句是多余的(可能您的意思是break
?)。另外,NULL
的现代版本是nullptr
另一点是,如果当前针对该类型的新的堆分配器(long
)存在分配问题(例如内存不足),那么if(p\long)
检查基本上只会是NULL
,并且以下if(0==*p\long)
正在检查新分配的long
是否自动初始化为0
。事实并非如此,因为在C++中,你只需要为你需要的东西付费。该值未定义(实际上,它可能是该内存地址处的现有未阻尼值)。long*temp=NULL;
对于(int i=1;i<10;i++){
如果(i<3){
长*长=新长;
如果(p_long){
如果(0==*p_长){
flag=true;
}else if(1==*p_long){
如果(临时)
delete temp;------>此处释放泄漏的内存
温度=p_长;
继续;
}
}
}
}
第一次输入for
循环temp
时为NULL
,但在以后的迭代中,您会为指针分配动态内存
long *p_long = new long;
在某些迭代中,您只需覆盖指针中包含的地址。您无法跟踪使用new
分配的内存。这并不意味着它会自动释放。内存仍然被分配,只是您丢失了指向它的指针
temp = p_long;
你需要这样的东西
else if ( 1 == *p_long )
{
delete temp; -----> leaked memory freed here.
temp = p_long;
continue;
}
这将在您将新分配的内存地址分配给temp
之前删除分配的内存
注意:也您正在调用new
而没有调用delete
。但是temp已经为空,没有指向任何内容。为什么可能是内存泄漏?删除已检查空值。你不需要显式地检查它。@A指向内存的指针有点像你离开迪斯科舞厅时挑选衣服的票。如果你的票丢了,你就不能取你的外套。如果你把你的票擦掉,使它不再显示你外套的号码,这并不意味着你没有丢失它。为了不丢失它,你需要一直指向那个数字,这个数字可能会被用来访问你的外套。这个数字是您分配内存的地址的值,是new返回给您的。灵感来自@DanielDaranas如果您用其他数字覆盖票证,在使用票证拾取您的外套之前,您的外套不会在稀薄的空气中消失。您的记录单可能会被覆盖以指向其他涂层,但您以前的涂层是孤立的,没有任何记录单来识别它。@Anand 1)它仍然泄漏,2)在调用delete之前,您不必检查NULL
。您甚至不必这样做。您只需调用delete temp代码>没有任何检查。谢谢juanchopanza,还有一件事,我最后也在删除,这会导致任何问题吗?我在“for”循环结束时删除temp。我不认为如果(1=*p_long)
是他想要做的。也许你的意思是如果(p_long)
。取消对未检查指针的引用可能是危险的(在这种情况下,它可能是nullptr
)。非null与==1
@PreetKukreti不同,实际上我想在else if block
中指示使用delete temp
。OP已经在检查if(p_long)
else if ( 1 == *p_long )
{
delete temp; -----> leaked memory freed here.
temp = p_long;
continue;
}