图标问题！请大虾捧场

[killjapan05]

一个基于对话框的MFC程序，想让左上角的图标动起来，下列代码有何问题（动不起来）：
void CClientDlg::OnTimer(UINT nIDEvent)
{
// TODO: Add your message handler code here and/or call default
static int roll=1;
HICON hIcon;
int rand=roll%4;
if(rand==0)
hIcon=AfxGetApp()->LoadIcon(IDI_UPICON);
else if(rand==1)
hIcon=AfxGetApp()->LoadIcon(IDI_RIGHTICON);
else if(rand==2)
hIcon=AfxGetApp()->LoadIcon(IDI_DOWNICON);
else if(rand==3)
hIcon=AfxGetApp()->LoadIcon(IDI_LEFTICON);
ASSERT(hIcon);
SetClassLong(this->GetSafeHwnd(),GCL_HICON,(LPARAM)hIcon);
//用WM_SETICON也不行
//this->SendMessage(WM_SETICON,TRUE,(LPARAM)hIcon);
roll++;
CDialog::OnTimer(nIDEvent);
}

[hadrons]

roll放在函数外面试试？

[sungang]

不能在OnTimer函数内置roll的初值

[killjapan05]

但它是个静态变量！这不应该出问题！

[killjapan05]

分太少？问题太难？

[Holly]

你的方法需要改进！我没有细看问题，但觉得你应当使用ImageList来实现，ImageList支持图标！

[Holly]

原因！看到你的动画每一次都有那么多的LoadIcon，太浪费！

[Holly]

给自己掌嘴！
没看清楚！
不过我已经实现了动画，请看：
resource.h:
#define IDI_ICON1

[Holly]

给自己掌嘴！
Sorry!没看清楚！
不过我已经实现了动画，请看：
-----------------------
resource.h:
...
#define IDI_ICONANI_FIRST 130
#define IDI_ICON1 130
#define IDI_ICON2 131
#define IDI_ICON3 132
#define IDI_ICON4 133
#define IDI_ICONANI_LAST 133
...
-----------------------
MyTest.cpp
class CAboutDlg : public CDialog
{
...
public:
UINT m_uIDTimer;
UINT m_uCurrICON;
...
};

int CAboutDlg::OnCreate(LPCREATESTRUCT lpCreateStruct)
{
...
m_uIDTimer = 1000;
m_uIDTimer = SetTimer(m_uIDTimer, 100, NULL);
m_uICON = IDI_ICONANI_FIRST;
SetIcon(AfxGetApp()->LoadIcon(m_uICON), FALSE);
...
}

void CAboutDlg::OnDestroy()
{
...
KillTimer(m_uIDTimer);
}

void CAboutDlg::OnTimer(UINT nIDEvent)
{
if(m_uIDTimer == nIDEvent)
{
m_uICON ++;
if( m_uICON > IDI_ICONANI_LAST)
m_uICON = IDI_ICONANI_FIRST;
SetIcon(AfxGetApp()->LoadIcon(m_uICON), FALSE);
}
}

[killjapan05]

TO HOLLY:
你将得到应有的报酬！问题被你解决！分很少，但感激之情很真！
好人有好报！

[longx]

AfxGetMainWnd()->SetIcon(hIcon,FALSE);

[longx]

如果还不行,将hIcon声明为类成员变量,
if(hIcon!=NULL) ::DeleteObject(hIcon);
.....
AfxGetMainWnd()->SetIcon(hIcon,FALSE);