Question

按照你说的做了，而且在析构函数里面 KillTimer(1) ,但是程序一运行后就崩溃了，是不是不要KillTimer(1)

BOOL CDialogInfo::OnInitDialog()
　　{
　　CDialog::OnInitDialog();
　　SetTimer(1,2000,NULL);
　　return TRUE;
　　}
　　void CDialogInfo::OnTimer(UINT nIDEvent)
　　{

　　if (nIDEvent == 1)
　　{
　　CDialog::OnOK();
　　}
　　CDialog::OnTimer(nIDEvent);
　　}

Answer 1

拿代码说话吧。

运行立刻崩溃，和OnTimer代码应该没什么关系，和OnInitDialog里面的代码可能有关系。

BOOL CDlgTestDlg::OnInitDialog()
{
	CDialog::OnInitDialog();

	// Set the icon for this dialog.  The framework does this automatically
	//  when the application's main window is not a dialog
	SetIcon(m_hIcon, TRUE);			// Set big icon
	SetIcon(m_hIcon, FALSE);		// Set small icon
	
	// TODO: Add extra initialization here
	SetTimer(1,5000,NULL);
	
	return TRUE;  // return TRUE  unless you set the focus to a control
}
void CDlgTestDlg::OnTimer(UINT nIDEvent) 
{
	// TODO: Add your message handler code here and/or call default
	if(nIDEvent==1)
	{
		KillTimer(1);
		CDialog::OnOK();
	}
	
	CDialog::OnTimer(nIDEvent);
}

经测试，代码没有问题，VC6运行无误。

追问

我想问的是 KillTimer(1) 放在 OnOK 前面的原因？

追答

这个地方最方便。其实不加问题也不大（定时器会随着窗口销毁而销毁），OnOK会关闭窗口，关闭之前KillTimer最安全。