Развертывание консольного приложения C # для Keyboard Listener

В вашем коде большое количество проблем с niggly, все из которых сговариваются, чтобы не иметь понятия, почему он не запускается на другой машине. Пройдя через источник:

    static bool debug = false;

У вас нет возможности диагностировать что-либо с этой переменной, равной false. Вы перехватываете исключения и не показываете сообщение об исключении и трассировку стека. Ты слепой, как летучая мышь. Это должен быть элемент конфигурации.

    static string _com = "COM3";

Никогда не серийный номер жесткого кода. Шансы, что он все еще COM3 на другой машине с другим эмулятором USB и другим драйвером, очень низки. Вы должны сделать его элементом конфигурации.

    setConsoleWindowVisibility(false);

Вы потеряли единственный шанс увидеть диагностику. Это нужно, если (! Debug) перед ним.

    _hookID = SetHook(_proc);

Проверка ошибок вообще отсутствует. Если SetWindowsHookEx() терпит неудачу, вы никогда не узнаете. Принудительные функции winapi всегда требуют проверки ошибок, у вас больше нет дружественных исключений.NET, чтобы не допустить неприятностей. Бросьте исключение Win32Exception, когда вызов winapi завершится с ошибкой.

    scaleSerialPort.Handshake = Handshake.None;

Устройства последовательного порта всегда используют квитирование. Они обращают внимание на ваши сигналы DTR и RTS и ничего не посылают, когда они выключены. Легко не видеть это, когда вы отлаживаете свой код, вы будете использовать программу эмулятора терминала, которая включает эти сигналы. Не будет работать на другой машине. Вы должны установить свойства DtrEnable и RtsEnable в true.

    if (scaleSerialPort.IsOpen)
        scaleSerialPort.Close();
    try
    {
        scaleSerialPort.Open();
        open = true;
    }

Это никогда не бывает правильно, статья MSDN для SerialPort.Close() специально предупреждает об этом. SerialPort использует рабочий поток для повышения своих событий, метод Close() не дожидается завершения этого потока. Попытка открыть его снова немедленно будет всегда терпеть неудачу, порт все еще открывается этим рабочим потоком. Просто никогда не делайте этого, откройте порт, когда ваше приложение запустится и не закроет его, пока оно не закончится.

    scaleSerialPort.ReadTimeout = 200; 

Категорически избегайте использования тайм-аутов, которые приближают время, в течение которого ваша программа может стать кататонической, потому что машина занята, или важные части вашего процесса выгружаются. Вы не должны использовать таймауты вообще, поскольку вы полагаетесь на DataReceived. Если вы все равно хотите их использовать, всегда делайте их в 10 раз больше, чем в худшем случае. Не опускайтесь ниже 5000.

    catch (Exception e)
    {
        if (debug) Console.WriteLine(e);
        Console.WriteLine("Could not connect to scale.");
        SendKeys.SendWait("^a");
        SendKeys.SendWait("Error-C23");
    }

Просто никогда не делай этого. Если вы не можете открыть порт, тогда вам не составит никакого смысла поддерживать вашу программу. Не забудьте громко рухнуть, напишите обработчик событий для AppDomain.CurrentDomain.UnhandledException, чтобы рассказать историю. Никогда не толкайте такие нажатия клавиш, вы не знаете, куда они идут.

    Double.TryParse(...);

Возвращаемое значение этого метода не является необязательным. Впадая вперед и игнорируя неудавшееся преобразование, просто производит недиагностируемое неправильное поведение.

    return SetWindowsHookEx(WH_KEYBOARD_LL, proc, GetModuleHandle(curModule.ModuleName), 0);

.NET 4.0 больше не эмулирует дескрипторы модулей для управляемого кода. Вы должны использовать LoadLibrary ("user32"), чтобы получить действительный дескриптор. Не проблема в Windows 8, она принимает нулевое значение для дескриптора модуля, но проблема в более ранних версиях.

    if (nCode >= 0 && wParam == (IntPtr)WM_KEYDOWN)

Это хорошо, но что, если другой крюк клавиатуры сначала перехватывает нажатия клавиш? Вы, конечно, проиграете.

    if (pressedKey == Keys.F11 || pressedKey == Keys.F12)

Я бы настоятельно рекомендовал вам не использовать низкоуровневый клавиатурный крючок, чтобы просто уловить два нажатия клавиш. Значительно лучше мышеловка - RegisterHotKey(). У вас не будет проблем с кодом примера для Google. Автоматически решает проблему "другое приложение видит в любом случае".

Ну, это, наверное, один из тех, я не могу догадаться, что :) Удачи.

Возможно, это не решение, которое вы ожидаете, но я все равно отправлю его.

Я использую другой подход для захвата такого рода событий, Windows предоставляет возможность зарегистрировать "глобальную горячую клавишу", которая будет перехвачена где угодно, независимо от того, работает ли целевое приложение или нет (на самом деле оно подключено к операционной системе и не одно приложение).

Вы можете попробовать с этим фрагментом https://gist.github.com/bruce965/87caacfb0289d0ad1b3c, который можно вызвать следующим образом:

new HotkeyHandler(false, false, false, false, Keys.F11).Register(yourForm);
new HotkeyHandler(false, false, false, false, Keys.F12).Register(yourForm);

где yourForm - это System.Windows.Forms.Form со следующим кодом:

protected override void WndProc(ref Message m) {
    if(m.Msg == HotkeyHandler.WM_HOTKEY_MSG_ID) {
        // Handle hotkey
    }

    base.WndProc(ref m);
}

Вам не нужно, чтобы форма была открытой или видимой, просто убедитесь, что она работает с:

Application.Run(yourForm);

---

Из связанного фрагмента для быстрой справки:

[DllImport("user32.dll")]
static extern bool RegisterHotKey(IntPtr hWnd, int id, int fsModifiers, int vk);

// make sure the second parameter '1234' is unique for every registered hotkey
RegisterHotKey(yourForm.Handle, 1234, 0, (int) Keys.F11);