Как сделать правильный входной класс в GLFW для игрового движка

Следует отметить, что GLFW обеспечивает обработку ввода как на основе опроса, так и на основе обратного вызова. В вашем примере вы используете оба: опрос для обработки ключей А и Б и обратный вызов для обработки ключа побег. Я полагаю, что вам будет проще работать с обратным вызовом при создании отдельного класса ввода (как для клавиатуры, так и для мыши).

Существует множество способов создания класса KeyInput с использованием метода обратного вызова GLFW. Вот мой предпочтительный метод (написанный в соответствии с вашим кодом). Он допускает экземпляры несколькоKeyInput (в отличие от синглтона), что означает, что у вас может быть экземпляр KeyInput для ключей пользовательского интерфейса, экземпляр для внутриигровых ключей и т. д.

Подводя итог: каждый экземпляр KeyInput отслеживает нажатое состояние списка клавиш, определяемых пользователем при построении. У него есть геттер и сеттер для доступа к нажатому состоянию для любой клавиши (хотя геттер и сеттер работают только с отслеживаемыми клавишами). Всякий раз, когда вызывается GLFW key-input-callback (статический), он вызывает этот установщик для все экземпляры из KeyInput. Экземпляры хранятся в статическом векторе, который добавляется при построении и удаляется при уничтожении.

Вход.ч

#include "../extern/GLFW/glfw3.h"
#include <map>
#include <vector>

class KeyInput {
  // Main KeyInput functionality
  public:
    // Takes a list of which keys to keep state for
    KeyInput(std::vector<int> keysToMonitor);
    ~KeyInput();
    // If this KeyInput is enabled and the given key is monitored,
    // returns pressed state.  Else returns false.
    bool getIsKeyDown(int key);
    // See _isEnabled for details
    bool getIsEnabled() { return _isEnabled; }
    void setIsEnabled(bool value) { _isEnabled = value; }
  private:
    // Used internally to update key states.  Called by the GLFW callback.
    void setIsKeyDown(int key, bool isDown);
    // Map from monitored keyes to their pressed states
    std::map<int, bool> _keys;
    // If disabled, KeyInput.getIsKeyDown always returns false
    bool _isEnabled;

  // Workaround for C++ class using a c-style-callback
  public:
    // Must be called before any KeyInput instances will work
    static void setupKeyInputs(Window& window);
  private:
    // The GLFW callback for key events.  Sends events to all KeyInput instances
    static void callback(
      GLFWwindow* window, int key, int scancode, int action, int mods);
    // Keep a list of all KeyInput instances and notify them all of key events
    static std::vector<KeyInput*> _instances;
};

Ввод.cpp

#include "KeyInput.h"
#include <algorithm>

std::vector<KeyInput*> KeyInput::_instances;

KeyInput::KeyInput(std::vector<int> keysToMonitor) : _isEnabled(true) {
  for (int key : keysToMonitor) {
    _keys[key] = false;
  }
  // Add this instance to the list of instances
  KeyInput::_instances.push_back(this);
}

KeyInput::~KeyInput() {
  // Remove this instance from the list of instances
  _instances.erase(std::remove(_instances.begin(), _instances.end(), this), _instances.end());
}

bool KeyInput::getIsKeyDown(int key) {
  bool result = false;
  if (_isEnabled) {
    std::map<int,bool>::iterator it = _keys.find(key);
    if (it != _keys.end()) {
      result = _keys[key];
    }
  }
  return result;
}

void KeyInput::setIsKeyDown(int key, bool isDown) {
  std::map<int,bool>::iterator it = _keys.find(key);
  if (it != _keys.end()) {
    _keys[key] = isDown;
  }
}

void KeyInput::setupKeyInputs(Window& window) {
  glfwSetKeyCallback(window.getWindow(), KeyInput::callback);
}

void KeyInput::callback(GLFWwindow* window, int key, int scancode, int action, int mods) {
  // Send key event to all KeyInput instances
  for (KeyInput* keyInput : _instances) {
    keyInput->setIsKeyDown(key, action != GLFW_RELEASE);
  }
}