16,473
社区成员
发帖
与我相关
我的任务
分享这个函数是内存泄露了吗？如何写好些呢？
CPoint* CArcShpShowDoc::TRAN(ARCPOINT* Point,int k)
{
CPoint* Scr=(CPoint *)malloc(sizeof(CPoint)*k);
for (int i=0;i<k;i++){
Scr[i].x =(Point[i].dX -3)*lScaleX;
Scr[i].y =(stcBox.dYmax-5)*lScaleY;
}
DPtoLP((HDC)pDC,Scr,l);
return Scr;
free(Scr);
}
{
CPoint* Scr=(CPoint *)malloc(sizeof(CPoint)*k);
for (int i=0;i<k;i++){
Scr[i].x =(Point[i].dX -3)*lScaleX;
Scr[i].y =(stcBox.dYmax-5)*lScaleY;
}
DPtoLP((HDC)pDC,Scr,l);
return Scr;
free(Scr);
}
...全文
请发表友善的回复…
发表回复
IceWall 2002-12-24
- 打赏
- 举报
行吧，谢谢！，结贴！
lllhaier 2002-12-23
- 打赏
- 举报
是一本书，不过比较薄，100页左右吧。
你用个全局变量不就行了吗？搞的这么辛苦。
你用个全局变量不就行了吗？搞的这么辛苦。
IceWall 2002-12-23
- 打赏
- 举报
ralphchen(听海):
林锐的高质量C++是一本书吗？
林锐的高质量C++是一本书吗？
ralphchen 2002-12-23
- 打赏
- 举报
你返回Scr的目的何在，如果你的目的是在CArcShpShowDoc::TRAN外仍然使用这些点，那你把点的空间申请放到CArcShpShowDoc类里不是更好。关于内存的问题推荐看林锐的高质量C++代码。
wyljery 2002-12-23
- 打赏
- 举报
或者直接建个全局嘛，大数组，现在流行这个
wyljery 2002-12-23
- 打赏
- 举报
你应该仿照bstr（参见讲解com技术的bstr一章）的工作原理，在做了操作后在释放
IceWall 2002-12-23
- 打赏
- 举报
可是这样不行大哥阿，释放后再返回就是空值啦？
那究竟该到哪去释放Scr指针呢？
那究竟该到哪去释放Scr指针呢？
manbug 2002-12-23
- 打赏
- 举报
建议将Scr设置成全局变量
manbug 2002-12-23
- 打赏
- 举报
错了,错了,哈哈,释放了就不能返回了
manbug 2002-12-23
- 打赏
- 举报
晕死,当然有啦,
最后三行应该这样
DPtoLP((HDC)pDC,Scr,l);
free(Scr);//这才是释放了
return Scr;//程序返回,如果上面的一行面之行下面,就永远也释放不了内存
最后三行应该这样
DPtoLP((HDC)pDC,Scr,l);
free(Scr);//这才是释放了
return Scr;//程序返回,如果上面的一行面之行下面,就永远也释放不了内存
