在函数内new的，应该在哪个地方delete

2014-02-25 04:44:50 711 阅读 AI 写同款 · GEO 优化AI 选题数据监测一键分发 ›

new和delete的底层实现原理 - new/new[]和delete/delete[]是什么？ new/new[]和delete/delete[]是操作符；是C++用来实现动态内存管理的操作符； new/new[] 操作符是用来申请空间的； delete/delete[]操作数是用来释放动态申请出来的空间； - new/delete的实现原理 new和delete是用户进行动态内存申请和释放的操作符，operator new... 阅读详情

我在一个按钮内，new出了一个对话框，这个delete dlg应该放在什么地方呢?
void CPrjDlg::OnBtn1()
{
// TODO: Add your control notification handler code here
DlgTest* dlg = new DlgTest;
dlg->Create(IDD_DLG_TEST,this);
dlg->ShowWindow(TRUE);
}

...全文

new delete 详解常规的new 和 delete 当需要内存时，可以使用new申请内存，但在用完内存时应当使用delete 去归还内存，以防止内存泄漏，具体使用方案见下面代码： int* ps = new int ; // 使用new 申明变量空间 ... ... delete ps ; '// delete 删除所申明的变量空间上述代码将删除ps 指针指向的内存空间，但不会删除指针本身，即仍然可以使用该指针去指向其他int 类型的变量。new 和 delete 一定需要成对的使用否则极易发生内存泄漏。注意：不阅读详情

cbzjzsb123

C++ 语言 65,214 成员 250,514 内容

+ 关注 + 加入社区

切换为时间正序

漫步者、 2014-02-26

打赏
举报

dlg->ShowWindow(TRUE); 这行代码后面释放掉。

worldy 2014-02-26

打赏
举报

引用楼主 cbzjzsb123 的回复:

我在一个按钮内，new出了一个对话框，这个delete dlg应该放在什么地方呢? void CPrjDlg::OnBtn1() { // TODO: Add your control notification handler code here DlgTest* dlg = new DlgTest; dlg->Create(IDD_DLG_TEST,this); dlg->ShowWindow(TRUE); }

可以在destroy函数里，delete this;

wb_rock 2014-02-26

打赏
举报

非要那么麻烦吗？？ DlgTest指针定义成CPrjDlg类的成员变量就行了

FrankHB1989 2014-02-26

打赏
举报

没搞清楚生存期的，老老实实用智能指针，不要在应用层代码里出现delete。能make_unique/make_shared就不出现new。搞清楚的，一般也没兴趣用内建指针了。

cbzjzsb123 2014-02-26

打赏
举报

引用 15 楼 jianwen0529 的回复:

[quote=引用 13 楼 Automation_dmu 的回复:] [quote=引用 10 楼 cbzjzsb123 的回复:] [quote=引用 4 楼 buyong 的回复:] CPrjDlg { ... DlgTest* m_dlg } CPrjDlg::CPrjDlg:m_dlg(NULL) { .... } CPrjDlg::~CPrjDlg { if(m_dlg) { m_dlg->Close(); //?? close the dlg, you may use SendMessage(WM_QUIT) delete m_dlg; m_dlg = NULL; } } void CPrjDlg::OnBtn1() { // TODO: Add your control notification handler code here m_dlg = new DlgTest; m_dlg->Create(IDD_DLG_TEST,this); m_dlg->ShowWindow(TRUE); }

你这个方法比较好！[/quote]倘若这个OnBtn1被多次点击可如何是好？[/quote] 多次点击有什么关系么？也就是对你创建的对话框进行显示隐藏而已，不需要重新new一个进行显示！如果内容不同，你进对其内容进行重新设置就可以了[/quote] 我也想到了这个问题，多次点击就多次new，最后只有一个delete。那这样说的，我这样写的方式不合理了，不能点一次new一次，可以点他就显示和影藏就行了。

还差得远呢 2014-02-26

打赏
举报

对话框也是一个类，所以应该在析构函数里释放

Johnblx 2014-02-26

打赏
举报

一般小的设计原则是是谁申请谁释放，也就是在一个函数内申请，在函数结尾的时候要释放。

引用楼主 cbzjzsb123 的回复:

我在一个按钮内，new出了一个对话框，这个delete dlg应该放在什么地方呢? void CPrjDlg::OnBtn1() { // TODO: Add your control notification handler code here DlgTest* dlg = new DlgTest; dlg->Create(IDD_DLG_TEST,this); dlg->ShowWindow(TRUE); }

幻夢之葉 2014-02-26

打赏
举报

引用 13 楼 Automation_dmu 的回复:

[quote=引用 10 楼 cbzjzsb123 的回复:] [quote=引用 4 楼 buyong 的回复:] CPrjDlg { ... DlgTest* m_dlg } CPrjDlg::CPrjDlg:m_dlg(NULL) { .... } CPrjDlg::~CPrjDlg { if(m_dlg) { m_dlg->Close(); //?? close the dlg, you may use SendMessage(WM_QUIT) delete m_dlg; m_dlg = NULL; } } void CPrjDlg::OnBtn1() { // TODO: Add your control notification handler code here m_dlg = new DlgTest; m_dlg->Create(IDD_DLG_TEST,this); m_dlg->ShowWindow(TRUE); }

你这个方法比较好！[/quote]倘若这个OnBtn1被多次点击可如何是好？[/quote] 多次点击有什么关系么？也就是对你创建的对话框进行显示隐藏而已，不需要重新new一个进行显示！如果内容不同，你进对其内容进行重新设置就可以了

幻夢之葉 2014-02-26

打赏
举报

引用 9 楼 cbzjzsb123 的回复:

[quote=引用 3 楼 jianwen0529 的回复:] 不需要用，或者销毁CPrjDlg（析构）的时候你这是动态创建么？你这样子的，应该用个成员来保存你的指针，否则你函数退出，局部变量就被销毁了！

为什么不需要用?不会内存泄漏吗？我看以前别人写的代码，也没有delete。[/quote] 不好意思，表达失误！是不需要的用new出来的东西的时候把它delete掉，或者在你类对象析构的时候delete！也就是常说的哪里new哪里delete，有时也是相对的说法！ new必须delete，否则内存泄露！

AndyStevens 2014-02-26

打赏
举报

引用 10 楼 cbzjzsb123 的回复:

[quote=引用 4 楼 buyong 的回复:] CPrjDlg { ... DlgTest* m_dlg } CPrjDlg::CPrjDlg:m_dlg(NULL) { .... } CPrjDlg::~CPrjDlg { if(m_dlg) { m_dlg->Close(); //?? close the dlg, you may use SendMessage(WM_QUIT) delete m_dlg; m_dlg = NULL; } } void CPrjDlg::OnBtn1() { // TODO: Add your control notification handler code here m_dlg = new DlgTest; m_dlg->Create(IDD_DLG_TEST,this); m_dlg->ShowWindow(TRUE); }

你这个方法比较好！[/quote]倘若这个OnBtn1被多次点击可如何是好？

「已注销」 2014-02-26

打赏
举报

你这个地方原则上是在该函数体最后释放，但是你的意思好像是需要一直使用，你可以将dlg的声明放在外面，然后在dlg生命周期结束的地方释放掉就OK，也可以在DlgTest析构函数中释放。

chinacoding 2014-02-26

打赏
举报

这段代码写的就有问题, 直接CDlg dlg; dlg.showwindow就可以了，根本就不用考虑指针的释放问题。用new 不是好方法。

赵4老师 2014-02-26

打赏
举报

在家里吃的，应该在哪个地方拉？

cbzjzsb123 2014-02-25

打赏
举报

引用 5 楼 xihu1364 的回复:

[quote=引用楼主 cbzjzsb123 的回复:] 我在一个按钮内，new出了一个对话框，这个delete dlg应该放在什么地方呢? void CPrjDlg::OnBtn1() { // TODO: Add your control notification handler code here DlgTest* dlg = new DlgTest; dlg->Create(IDD_DLG_TEST,this); dlg->ShowWindow(TRUE); }

楼上正解！加上：你代码使用的非模态对话框，你可以使用模态对话框，domodel后直接delete就好了[/quote] 我这个对话框要嵌在别的对话框里，用不了模态对话框。

cbzjzsb123 2014-02-25

打赏
举报

引用 4 楼 buyong 的回复:

CPrjDlg { ... DlgTest* m_dlg } CPrjDlg::CPrjDlg:m_dlg(NULL) { .... } CPrjDlg::~CPrjDlg { if(m_dlg) { m_dlg->Close(); //?? close the dlg, you may use SendMessage(WM_QUIT) delete m_dlg; m_dlg = NULL; } } void CPrjDlg::OnBtn1() { // TODO: Add your control notification handler code here m_dlg = new DlgTest; m_dlg->Create(IDD_DLG_TEST,this); m_dlg->ShowWindow(TRUE); }

你这个方法比较好！

cbzjzsb123 2014-02-25

打赏
举报

引用 3 楼 jianwen0529 的回复:

不需要用，或者销毁CPrjDlg（析构）的时候你这是动态创建么？你这样子的，应该用个成员来保存你的指针，否则你函数退出，局部变量就被销毁了！

为什么不需要用?不会内存泄漏吗？我看以前别人写的代码，也没有delete。

SKATE11 2014-02-25

打赏
举报

什么地方不需要了就释放

lpcads 2014-02-25

打赏
举报

引用 4 楼 buyong 的回复:

CPrjDlg { ... DlgTest* m_dlg } CPrjDlg::CPrjDlg:m_dlg(NULL) { .... } CPrjDlg::~CPrjDlg { if(m_dlg) { m_dlg->Close(); //?? close the dlg, you may use SendMessage(WM_QUIT) delete m_dlg; m_dlg = NULL; } } void CPrjDlg::OnBtn1() { // TODO: Add your control notification handler code here m_dlg = new DlgTest; m_dlg->Create(IDD_DLG_TEST,this); m_dlg->ShowWindow(TRUE); }

我在想 m_dlg = NULL; 是否有必要。因为逻辑上，一个对象已析构后，是不会（不应该）再访问它的成员的。

「已注销」 2014-02-25

打赏
举报

用智能指针

boost::shared_ptr<DlgTest> dlg(new DlgTest());

版主大哥 2014-02-25

打赏
举报

引用楼主 cbzjzsb123 的回复:

我在一个按钮内，new出了一个对话框，这个delete dlg应该放在什么地方呢? void CPrjDlg::OnBtn1() { // TODO: Add your control notification handler code here DlgTest* dlg = new DlgTest; dlg->Create(IDD_DLG_TEST,this); dlg->ShowWindow(TRUE); }

楼上正解！加上：你代码使用的非模态对话框，你可以使用模态对话框，domodel后直接delete就好了

加载更多回复（4）

在函数内new的，应该在哪个地方delete

相关推荐