关于指针自增运算的问题

[water0313]

写了一个求阶乘的程序，使用了指针变量，但是在VS2008中运行时，指针的自增运算会产生中断错误，不知道哪里考虑欠缺，就把程序贴出来，让大家帮忙找找
#include <iostream>
using namespace std;
void main()
{
int x=1;
long * p = new long [16];
*p = 1;
for (x;x<16;x++)
{
p[x]=(p[x-1])* x;
}
x=0;
for (x;x<16;x++)
{
cout<<*p<<endl;
p++;
}
delete [] p;

}

[rocktyt]

你需要用一个指针保存一开始申请的内存空间，并用这个地址来delete p已经移掉了是没法delete成功的

[water0313]

恍然大悟啊

[漫步者、]

p[x]=(p[x-1])* x; 这句等价于*(p+x)=*(p+x-1)*x;其实每次在原来的基础上移动x个单元，这样导致 cout<<*p<<endl; p++; 越界了。

[water0313]

引用 1 楼 rocktyt2 的回复:

你需要用一个指针保存一开始申请的内存空间，并用这个地址来delete p已经移掉了是没法delete成功的

恍然大悟啊

[water0313]

引用 3 楼 ganpengjin1 的回复:

p[x]=(p[x-1])* x; 这句等价于*(p+x)=*(p+x-1)*x;其实每次在原来的基础上移动x个单元，这样导致 cout<<*p<<endl; p++; 越界了。

如果是这样的话，它为什么还能显示正确的结果，但是运行结束后才弹出错误框呢

[漫步者、]

所以越界了导致的，更别提后面的delete了

[water0313]

引用 6 楼 ganpengjin1 的回复:

所以越界了导致的，更别提后面的delete了

我认为p[x]=(p[x-1])* x表达式中，p没有发生变化，只是指向的值发生变化，越界是由第二个for循环中的p++导致的，而且我的猜测已经测试过了。 很感谢你能提到这一点，让我对指针的认识加深了

[starytx]

应该没有越界，只是因为p自增后指向改变，所以delete就错了，保存原始指针，delete原始指针即可

[lifeisshortyouneed]

#include <iostream>
using namespace std;
int main()
{
 	int x=1;
 	long * p = new long [16];
 	*p = 1;
 	for (x;x<16;x++)
 	{
		p[x]=p[x-1]* x;
 	}
 	x=0;
 	for (x;x<16;x++)
 	{
 		cout<<p[x]<<endl;//问题应该在这里，x=15的时候，P已经指向了申请的最一个地址了，
//就是说最后一次循环的p++ 使P越界,其实像我这样改一下输出就行了，没有必要让p自增了。
 	}
 	delete [] p;
 	return 0;
}