Handling KeyBoard messages/accelerators handling in MFC dialog based applications

ForNow

got a compile error when it I did that but I had semi colon after the open paren so the logic should work regardless I'll work to change it thanks

Lost User

ForNow wrote:

I had semi colon after the open paren

:omg:

ForNow

I am changing that so if I get a zero from from

TranslateAccelerator(

that means (and can you confirm this) it wasnt one the key strokes I defined right not necessarly an error I would get a zero from TranslateAccelerator maybe thats the problem in my logic

Dave Kreskowiak

Not to mention the semi-colons on the end of the if line, essentially making it entirely useless.

Asking questions is a skill CodeProject Forum Guidelines Google: C# How to debug code Seriously, go read these articles.
Dave Kreskowiak

Lost User

Yes, I have never seen that done deliberately before.

Graham Breach

Here's the Windows docs page: TranslateAcceleratorA function (winuser.h) - Win32 apps | Microsoft Docs[^] The function returns nonzero when it successfully translates the accelerator, which is when you should not pass the message through to the default handler. Your code is bypassing the default handler entirely when your h_accel and debug variables are set (without checking if the message was a translated keycode).

ForNow

thanks if so this code by Nish

BOOL CPreTransTestApp::ProcessMessageFilter(int code, LPMSG lpMsg)
{
if(m_haccel)
{
if (::TranslateAccelerator(m_pMainWnd->m_hWnd, m_haccel, lpMsg))
return(TRUE);
}

return CWinApp::ProcessMessageFilter(code, lpMsg);

}

is incorrect this code from Microsoft docs is correct

MSG msg;
BOOL bRet;

while ( (bRet = GetMessage(&msg, (HWND) NULL, 0, 0)) != 0)
{
if (bRet == -1)
{
// handle the error and possibly exit
}
else
{
// Check for accelerator keystrokes.

    if **(!**TranslateAccelerator( 
            hwndMain,      // handle to receiving window 
            haccel,        // handle to active accelerator table 
            &msg))         // message data 
    {
        TranslateMessage(&msg); 
        DispatchMessage(&msg); 
    } 
}

}

as it has not TranlateAccelterator wouldnt you agree

Graham Breach

They are both correct. The first one skips the default handler if the accelerator has been translated. The second one calls the default handler if the accelerator has not been translated.

ForNow

thank you so much for your patience me However the way mine is set is INCORRECT as I dont have an IF testing for the validity of the TranlateAccelarator and that is why my keyboard gets locked becasue I havr return TRUE for all

Graham Breach

Yes, that's right.