События клавиш цифровой клавиатуры приводят к зависанию клавиш из GetKeyboardState

У меня есть приложение C ++ (MFC), которому нужно проверять состояние ключа по таймеру. Если пользователь удерживает клавишу, мы задерживаем обработку некоторого кода.

Вот чек на keydown:

if (!GetKeyboardState(keyState)) 
{
    s_executeDeferredResult = e_executeDeferredButtonCheckFailed;
    return;
}   
s_executeDeferredStuckKeys.clear();
for (int index=0; index<arrsize(keyState); ++index)
{
    if (keyState[index] & 0x80)
    {
        s_executeDeferredStuckKeys.insert(index);
    }
}
if (!s_executeDeferredStuckKeys.empty())
{
    s_executeDeferredResult = e_executeDeferredButtonsActive;
    return;
}

Но есть некоторые ключевые комбинации, которые застревают:

  1. Включите NUMLOCK
  2. Нажмите SHIFT
  3. Нажмите NumPad8
  4. Отпустите SHIFT
  5. Отпустите NumPad8
    (это один пример, есть и другие, в том числе дузи с CTRL - ALT - DEL )

GetKeyboardState теперь сообщит, что VK_UP нажата.

Происходящие события (соответствующие действиям выше).

  1. <None>
  2. WM_KEYDOWN, VK_SHIFT
  3. WM_KEYUP, VK_SHIFT
    WM_KEYDOWN, VK_UP
    WM_KEYDOWN, VK_SHIFT
  4. WM_KEYUP, VK_SHIFT
  5. WM_KEYUP, VK_NUMPAD8

Итак, Windows не распознает, что появилась клавиша «Вверх», и теперь GetKeyboardState не работает.

Есть ли хороший способ проверить реальное состояние ключа? GetAsyncKeyState и GetKeyState оба сообщают, что ключ тоже не работает.


windows c++ keyboard-events mfc
person Craig Steele    schedule 28.08.2013    source источник
comment
Вы пробовали использовать другую клавиатуру?   -  person Jonathan Potter    schedule 29.08.2013
comment
Да, это происходит на всех клавиатурах, которые мы пробовали.   -  person StilesCrisis    schedule 29.08.2013
comment
Поскольку это приложение MFC, обязательно проверьте свою PreTranslateMessage реализацию. Вот где со временем накапливается всякое плохое. Также обратите внимание на замечание к документации GetKeyboardState: < i> Статус меняется, когда поток удаляет сообщения клавиатуры из своей очереди сообщений. Другими словами: если ваше приложение делает что-то подозрительное, есть вероятность, что GetKeyboardState запутается.   -  person IInspectable    schedule 29.08.2013
comment
Как оказалось, такое поведение можно воспроизвести в небольшом автономном тестовом приложении. Но хорошая мысль.   -  person StilesCrisis    schedule 29.08.2013

Ответы (1)


arrow_upward
2
arrow_downward

Решил это.

Я подключился к событиям клавиатуры в InitInstance и отслеживаю взлеты и падения с помощью кода сканирования (карта с кодом сканирования в качестве ключа и виртуальными ключами в качестве значения).

m_keyboardHook = SetWindowsHookEx(WH_KEYBOARD, &KeyboardHook, NULL, GetCurrentThreadId());

static LRESULT CALLBACK KeyboardHook(
    __in int nCode,
    __in WPARAM wParam,
    __in LPARAM lParam
    )
{
    // According to the docs, we're not allowed to do any "further processing" if nCode is < 0.
    // According to the docs, we should process messages if we get code HC_ACTION. 
    // http://msdn.microsoft.com/en-us/library/windows/desktop/ms644984(v=vs.85).aspx
    // It doesn't specify what to do if another code comes in, so we will just ignore any other codes.
    if (nCode == HC_ACTION)
    {
        uint8 scanCode = (uint8)((lParam & 0x00FF0000) >> 16);
        uint8 virtKey = (uint8)wParam;
        if (lParam & 0x80000000) // key up
            KeyState::SetKeyUp(scanCode);
        else
            KeyState::SetKeyDown(scanCode, virtKey);
    }

    // We must call the next hook in the chain, according to http://msdn.microsoft.com/en-us/library/windows/desktop/ms644975%28v=vs.85%29.aspx
    // First param is ignored, according to http://msdn.microsoft.com/en-us/library/windows/desktop/ms644974%28v=vs.85%29.aspx )
    return CallNextHookEx(0, nCode, wParam, lParam);
}

Итак, моя отложенная проверка становится:

// Similarly, don't process deferred events if there are keys or mouse buttons which are currently down.
s_executeDeferredStuckKeys.clear();
if (KeyState::AnyKeysDown(s_excludeKeys, arrsize(s_excludeKeys)))
{
    s_executeDeferredResult = e_executeDeferredButtonsActive;
    KeyState::GetDownKeys(s_executeDeferredStuckKeys);
    return;
}
person Craig Steele    schedule 29.08.2013