Question

I have been having trouble with my program trying to gray out ( and disable ) a sub menu item.

What I’m looking for is that the “run” item be disabled unless the required .ini entry is not empty.

My code

LRESULT CALLBACK WndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
{
HMENU hmenu = GetMenu(hWnd);
// Reading in ini
    if (0 == strcmp(webLocation, "")){
    EnableMenuItem(hmenu,ID_WEBSERVICES_RUN,MF_DISABLED | MF_GRAYED);
    WritePrivateProfileString(_T("WEBSERVICES"), _T("Location"), _T("Tool Not Found"), WpathStr);
}

I am unsure as to whether I am getting the HMENU correctly and why this code is not working for the desired effect.

Any help with this would be greatly appreciated.

Answer 1

You can’t just put this in the WndProc at the top level. WndProc process events, whether the window has been constructed or not. It’ll be called many times for many different reasons.

Your WndProc will almost certainly look like a big switch on message. The one you want here is WM_INITDIALOG:

LRESULT CALLBACK WndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
{
    switch(message)
    {
         case WM_INITDIALOG:
             // jump to a new function that reads the .ini
             // and disables the control etc.
             return OnInitDialog(hWnd, wParam, lParam);

         default:
             return DefWindowProc(hWnd, message, wParam, lParam);
    }
}