一段神奇的 C++代码，大家觉得有没有问题？

[Wangjl]

如下代码，我在 vs2015 中，for 循环 1000 遍，没有问题，10000 遍就报错

大家觉得是哪里的问题呢？

#include <stdio.h>

void f(char* p)
{
	delete[] p;
	p = new char[20000];
}

int main()
{
	
	for (int i = 0; i < 10000; i++)
	{
		char *p = new char[2];
		f(p);
		delete[] p;
	}
	getchar();
	return 0;
}

按理说以上代码只是在循环创建 20000 个字节的堆内存，创建了又销毁，不应该出问题才对啊。

大家有什么看法？

[hihibin]

@Wangjl 上条搞错了，习惯直接对地址操作，左值右值都搞错了。。你最后解释对的

[WANGJIEKE]

我寻思着这不是 double delete 吗。。。double delete 在我电脑上跑一次就炸的，不知道为什么你这跑 1000 次不出问题

[missdeer]

一楼正解

[zwh2698]

人生到处都是坑，且行且小心！前途未知概因不曾顿悟。

[huluhulu]

@iwong0exv2 你这个会影响。你和楼主的案例不一样。

[Wangjl]

这可能和 vs 有关，我现在又无法重现了，之前 for1000 次都没问题，可昨天晚上，我 for1 次就不行了。 但我做过测试，如果函数内的 p 的本身地址和函数外的 p 的地址一样的话，就不会出问题，我估计我之前那是偶然现象，可能是 vs 的问题。
只要函数内的指针地址和外面的不一样，那一定会挂，因为二次释放。

[Wangjl]

#include <stdio.h>

void f(char* p)
{
printf("f 函数中 p 指针本身的地址是: %x\n\n", &p);
printf("f 函数中 p 指针里面存放的地址是: %x\n\n", *p);
printf("释放 f 函数中 p 指针里面存放的地址 %x 指向的内存空间\n\n", *p);
delete[] p;
p = new char[20000];

}

int main()
{

for (int i = 0; i < 1000; i++)
{
char *p = new char[2];
printf("外部 p 指针本身的地址是: %x\n\n", &p);
printf("外部 p 指针里面存放的地址是: %x\n\n", *p);
f(p);
printf("释放外部 p 指针里面存放的地址 %x 指向的内存空间\n\n", *p);
delete[] p;

}
getchar();
return 0;
}

以上代码就可以看出原因。

[iwong0exv2]

@huluhulu 兄弟你也是花厂的？

[huluhulu]

@iwong0exv2 不是

[leido]

用 f 的参数 p 指针用引用才对

[iwong0exv2]

@huluhulu 哦。如果写成 p->next=p;的话，是会修改链表，但反过来写只是利用同一个指针来做遍历，所以并不会修改链表本身。其实我这个解释有点多余，代码本身已经很清楚了。

[huluhulu]

@iwong0exv2 你是对的

[darknoll]

char*& p

[Chenamy2017]

一楼正解，看来指针还需要再好好学习下了

[tkhmy]

传值，传引用，传指针好好复习一下，你这里是传值的