关于全局函数，句柄的问题、

Question

一个程序，是MFC的对话框程序，有N个对话框切来切去，每个对话框上的按钮需要用位图美化一下，
采用的是如下函数：
void CDlg1::SetBmpToButton(int nButtonID, CString str)
{
CButton *pButton = (CButton *)GetDlgItem(nButtonID); //1、定义 but类的指针。
pButton->ModifyStyle(0,WS_CHILD|BS_BITMAP,NULL); //2、设置风格  。 
//3、然后 SetBitmap，用大初始化中已经赋过值 的 hbitmap 类 的g_hIconUp
HBITMAP hIcon = (HBITMAP)LoadImage(AfxGetInstanceHandle(),_T(str),
IMAGE_BITMAP, 0, 0, LR_CREATEDIBSECTION|LR_LOADFROMFILE);

pButton->SetBitmap( hIcon ) ;

if ( hIcon != NULL )
{
hIcon = NULL;
delete(hIcon);
}
}我现在有很多Dlg ，每个Dlg里都写了这个函数，
我想把这个函数写成全局函数，放到公共的文档里，所有对话框都能使用此函数。
但是遇到了问题。我的全局函数是这样写的：
void g_SetBmpToButton(HWND hDlg ,int nButtonID, CString strBmpPath)
{
CButton *pButton = (CButton *)GetDlgItem(hDlg , nButtonID);
pButton->ModifyStyle(hDlg , 0,WS_CHILD|BS_BITMAP,NULL);
HBITMAP hIcon = (HBITMAP)LoadImage(AfxGetInstanceHandle(),_T(strBmpPath),
IMAGE_BITMAP, 0, 0, LR_CREATEDIBSECTION|LR_LOADFROMFILE);
pButton->SetBitmap(  hIcon);

if ( hIcon != NULL )
{
hIcon = NULL;
delete(hIcon);
}
pButton = NULL;
delete(pButton); 
}与单独对话框里的函数相比，只是多了一个句柄的参数。
然后在任意对话框里是这样调用的：
g_SetBmpToButton( m_hWnd  , IDC_BUT_1,"res\\美化效果1.bmp");
现在问题就出现在了第一个参数上 ,第一个参数无论传 m_hWnd   还是 GetSafeHwnd() ，都无法得到完整的wnd.所以在pButton->ModifyStyle(hDlg , 0,WS_CHILD|BS_BITMAP,NULL);这一句上就死掉了。请大侠帮忙看看，第一个句柄应该如何传下来啊、

Answer 1

void g_SetBmpToButton(CButton *pButton, CString strBmpPath)
{
pButton->ModifyStyle(0,WS_CHILD|BS_BITMAP,NULL);
HBITMAP hIcon = (HBITMAP)LoadImage(AfxGetInstanceHandle(),_T(strBmpPath),
IMAGE_BITMAP, 0, 0, LR_CREATEDIBSECTION|LR_LOADFROMFILE);
pButton->SetBitmap(  hIcon);

if ( hIcon != NULL )
{
hIcon = NULL;
delete(hIcon);
}
pButton = NULL;
delete(pButton); 
}然后这样调用
g_SetBmpToButton( (CButton *)GetDlgItem(IDC_BUT_1) ,"res\\**.bmp");

Answer 2

调用时的窗口句柄m_hWnd有效么？ID控件存在么？
void g_SetBmpToButton(HWND hDlg ,int nButtonID, CString strBmpPath)
{
  if(IsWindow(hDlg))
  {
    CButton *pButton = (CButton *)GetDlgItem(hDlg , nButtonID);
    if(pButton && pButton->m_hWnd)
    {
      ……
    }
  }
}

Answer 3

在全局函数里调用的GetDlgItem应该是直接调用了API，所以返回的是控件的句柄，自然不能通过类型强制转换变成CWnd*指针！应该用：
CButton *pButton = (CButton *)CWnd::FromHandle(GetDlgItem(hDlg , nButtonID));
同时还是要注意句柄、指针有效性的检查。