Using global keyboard hook (WH_KEYBOARD_LL) in WPF / C#

51,616

Solution 1

You're creating your callback delegate inline in the SetHook method call. That delegate will eventually get garbage collected, since you're not keeping a reference to it anywhere. And once the delegate is garbage collected, you will not get any more callbacks.

To prevent that, you need to keep a reference to the delegate alive as long as the hook is in place (until you call UnhookWindowsHookEx).

Solution 2

IIRC, when using global hooks, if your DLL isn't returning from the callback quick enough, you're removed from the chain of call-backs.

So if you're saying that its working for a bit but if you type too quickly it stops working, I might suggest just storing the keys to some spot in memory and the dumping the keys later. For an example, you might check the source for some keyloggers since they use this same technique.

While this may not solve your problem directly, it should at least rule out one possibility.

Have you thought about using GetAsyncKeyState instead of a global hook to log keystrokes? For your application, it might be sufficient, there's lots of fully implemented examples, and was personally easier to implement.

Solution 3

The winner is: Capture Keyboard Input in WPF, which suggests doing :

TextCompositionManager.AddTextInputHandler(this,
    new TextCompositionEventHandler(OnTextComposition));

...and then simply use the event handler argument’s Text property:

private void OnTextComposition(object sender, TextCompositionEventArgs e)
{
    string key = e.Text;
    ...
}

Solution 4

I have used the Dylan's method to hook global keyword in WPF application and refresh hook after each key press to prevent events stop firing after few clicks . IDK, if it is good or bad practice but gets the job done.

      _listener.UnHookKeyboard();
      _listener.HookKeyboard();

Implementation details here

Share:
51,616
Ciantic
Author by

Ciantic

Updated on July 08, 2022

Comments

  • Ciantic
    Ciantic over 1 year

    I stitched together from code I found in internet myself WH_KEYBOARD_LL helper class:

    Put the following code to some of your utils libs, let it be YourUtils.cs:

    using System;
    using System.Diagnostics;
    using System.Runtime.InteropServices;
    using System.Runtime.CompilerServices;
    using System.Windows.Input;
    
    namespace MYCOMPANYHERE.WPF.KeyboardHelper
    {
        public class KeyboardListener : IDisposable
        {
            private static IntPtr hookId = IntPtr.Zero;
    
            [MethodImpl(MethodImplOptions.NoInlining)]
            private IntPtr HookCallback(
                int nCode, IntPtr wParam, IntPtr lParam)
            {
                try
                {
                    return HookCallbackInner(nCode, wParam, lParam);
                }
                catch
                {
                    Console.WriteLine("There was some error somewhere...");
                }
                return InterceptKeys.CallNextHookEx(hookId, nCode, wParam, lParam);
            }
    
            private IntPtr HookCallbackInner(int nCode, IntPtr wParam, IntPtr lParam)
            {
                if (nCode >= 0)
                {
                    if (wParam == (IntPtr)InterceptKeys.WM_KEYDOWN)
                    {
                        int vkCode = Marshal.ReadInt32(lParam);
    
                        if (KeyDown != null)
                            KeyDown(this, new RawKeyEventArgs(vkCode, false));
                    }
                    else if (wParam == (IntPtr)InterceptKeys.WM_KEYUP)
                    {
                        int vkCode = Marshal.ReadInt32(lParam);
    
                        if (KeyUp != null)
                            KeyUp(this, new RawKeyEventArgs(vkCode, false));
                    }
                }
                return InterceptKeys.CallNextHookEx(hookId, nCode, wParam, lParam);
            }
    
            public event RawKeyEventHandler KeyDown;
            public event RawKeyEventHandler KeyUp;
    
            public KeyboardListener()
            {
                hookId = InterceptKeys.SetHook((InterceptKeys.LowLevelKeyboardProc)HookCallback);
            }
    
            ~KeyboardListener()
            {
                Dispose();
            }
    
            #region IDisposable Members
    
            public void Dispose()
            {
                InterceptKeys.UnhookWindowsHookEx(hookId);
            }
    
            #endregion
        }
    
        internal static class InterceptKeys
        {
            public delegate IntPtr LowLevelKeyboardProc(
                int nCode, IntPtr wParam, IntPtr lParam);
    
            public static int WH_KEYBOARD_LL = 13;
            public static int WM_KEYDOWN = 0x0100;
            public static int WM_KEYUP = 0x0101;
    
            public static IntPtr SetHook(LowLevelKeyboardProc proc)
            {
                using (Process curProcess = Process.GetCurrentProcess())
                using (ProcessModule curModule = curProcess.MainModule)
                {
                    return SetWindowsHookEx(WH_KEYBOARD_LL, proc,
                        GetModuleHandle(curModule.ModuleName), 0);
                }
            }
    
            [DllImport("user32.dll", CharSet = CharSet.Auto, SetLastError = true)]
            public static extern IntPtr SetWindowsHookEx(int idHook,
                LowLevelKeyboardProc lpfn, IntPtr hMod, uint dwThreadId);
    
            [DllImport("user32.dll", CharSet = CharSet.Auto, SetLastError = true)]
            [return: MarshalAs(UnmanagedType.Bool)]
            public static extern bool UnhookWindowsHookEx(IntPtr hhk);
    
            [DllImport("user32.dll", CharSet = CharSet.Auto, SetLastError = true)]
            public static extern IntPtr CallNextHookEx(IntPtr hhk, int nCode,
                IntPtr wParam, IntPtr lParam);
    
            [DllImport("kernel32.dll", CharSet = CharSet.Auto, SetLastError = true)]
            public static extern IntPtr GetModuleHandle(string lpModuleName);
        }
    
        public class RawKeyEventArgs : EventArgs
        {
            public int VKCode;
            public Key Key;
            public bool IsSysKey;
    
            public RawKeyEventArgs(int VKCode, bool isSysKey)
            {
                this.VKCode = VKCode;
                this.IsSysKey = isSysKey;
                this.Key = System.Windows.Input.KeyInterop.KeyFromVirtualKey(VKCode);
            }
        }
    
        public delegate void RawKeyEventHandler(object sender, RawKeyEventArgs args);
    }
    

    Which I use like this:

    App.xaml:

    <Application ...
        Startup="Application_Startup"
        Exit="Application_Exit">
        ...
    

    App.xaml.cs:

    public partial class App : Application
    {
        KeyboardListener KListener = new KeyboardListener();
    
        private void Application_Startup(object sender, StartupEventArgs e)
        {
            KListener.KeyDown += new RawKeyEventHandler(KListener_KeyDown);
        }
    
        void KListener_KeyDown(object sender, RawKeyEventArgs args)
        {
            Console.WriteLine(args.Key.ToString());
            // I tried writing the data in file here also, to make sure the problem is not in Console.WriteLine
        }
    
        private void Application_Exit(object sender, ExitEventArgs e)
        {
            KListener.Dispose();
        }
    }
    

    The problem is that it stops working after hitting keys a while. No error is raised what so ever, I just don't get anything to output after a while. I can't find a solid pattern when it stops working.

    Reproducing this problem is quiet simple, hit some keys like a mad man, usually outside the window.

    I suspect there is some evil threading problem behind, anyone got idea how to keep this working?


    What I tried already:

    1. Replacing return HookCallbackInner(nCode, wParam, lParam); with something simple.
    2. Replacing it with asynchronous call, trying to put Sleep 5000ms (etc).

    Asynchronous call didn't make it work any better, it seems stop always when user keeps single letter down for a while.