TrackPopupMenu运行程序崩溃

[wshcy512]

void CMyListView::OnRButtonDown(UINT nFlags, CPoint point)
{
	CListCtrl &listCtrl=(CListCtrl&)GetListCtrl();
	CMenu menu;
	menu.CreatePopupMenu();
	if (menu.GetSafeHmenu() == NULL)
		return;

	menu.AppendMenu(MF_STRING, 1, "新建");
	if (listCtrl.GetSelectedCount() > 0)
	{
		menu.AppendMenu(MF_STRING, 2, "修改");
		menu.AppendMenu(MF_STRING, 3, "删除");
	}
	SetForegroundWindow();
	
	long lr = menu.TrackPopupMenu(TPM_LEFTBUTTON |TPM_LEFTALIGN ,point.x,point.y,this);
	menu.DestroyMenu();
	//CListView::OnRButtonDown(nFlags, point);
}

第一次运行到TrackPopupMenu没有错，在点右键运行到TrackPopupMenu就程序崩溃了
另外，很奇怪，调试的时候有时候会蹦，有时候不会蹦

[stjay]

不用menu.DestroyMenu();

MSDN
If the menu is assigned to a window, it is automatically destroyed when the window is destroyed. If the menu is added to an existing menu, it is automatically destroyed when that menu is destroyed.

Before exiting, an application must free system resources associated with a pop-up menu if the menu is not assigned to a window. An application frees a menu by calling the DestroyMenu member function.

大概意思是 菜单关联到窗口时会自动销毁

[stjay]

还有最好将 CMenu menu 设为成员变量

[Conry]

除了弹出位置不对外没发现其他问题
添加红色代码位置就没问题了
SetForegroundWindow();

ClientToScreen(&point);
long lr = menu.TrackPopupMenu(TPM_LEFTBUTTON |TPM_LEFTALIGN ,point.x,point.y,this);
menu.DestroyMenu();

[用户 昵称]

[Quote=引用 1 楼 stjay 的回复:]
不用menu.DestroyMenu();

MSDN
If the menu is assigned to a window, it is automatically destroyed when the window is destroyed. If the menu is added to an existing menu, it is automatically destroyed when that menu is destroyed.

Before exiting, an application must free system resources associated with a pop-up menu if the menu is not assigned to a window. An application frees a menu by calling the DestroyMenu member function.

大概意思是 菜单关联到窗口时会自动销毁
[/Quote]
看不懂，按你贴的注释，应该调用DestroyMenu。

[wshcy512]

我新建了一个单文档，用上面代码测试没问题，可能是我程序其他地方报错吧，谢谢各位