上下文

问题的背景是我目前正在编写一个与 Arduino 一起使用的小型库,以便充当游戏 Controller 。然而,我遇到的问题更多地与 C++ 有关,而不是 Arduino 特定的任何问题。

我在下面包含了库的标题和源代码,然后是 Arduino 代码。我尽可能地截断了它。

问题

简而言之,只有我定义的最后一个开关/ Action 才能得到正确处理。

这些操作在 Arduino 设置函数中定义。例如:

controller.addSwitchContinuous(10, 0);  // Pin 10; btn index 0

意味着引脚 10 被映射到按钮 0。当引脚 10 关闭时,这被视为按钮被按下。这适用于单个 Action ,但是当我开始添加更多 Action 时,只有最后一个 Action 才真正起作用。因此,在以下示例中,仅识别引脚 9:
controller.addSwitchContinuous(10, 0);  // <-- Doesn't work
controller.addSwitchContinuous(9, 1);   // <-- Works

这适用于任意数量的操作:
controller.addSwitchContinuous(10, 0);  // <-- Doesn't work
controller.addSwitchContinuous(9, 1);   // <-- Doesn't work
controller.addSwitchContinuous(8, 2);   // <-- Doesn't work
controller.addSwitchContinuous(7, 3);   // <-- Works

潜在原因

我对 C++ 相当新手,所以我怀疑我在指针上做错了。更具体地说, Joystick_ 实例的传递方式似乎有问题。

我一直在摆弄构造函数并尝试使用引用而不是指针,但我无法让它正常工作。

我可以确认 JFSF::loop 中的迭代确实遍历所有操作,如果我修改它:
void JFSF::loop()
    {
        for (int n = 0; n < _nextActionIndex; n++)
        {
            if (_actions[n])
            {
                _actions[n]->loop();
                _joystick->setButton(n, PRESSED);  // Debug: Set button pressed, regardless of switch.
            }
        }
        if (_doSendState)
        {
            _joystick->sendState();
        }
    }

然后按预期按下按钮 0 到 n。 loop() 可能没有被正确调用,但我希望它在 N = 1 的情况下也会失败。此外,最后一个 Action 总是成功的事实表明迭代没问题。

完整代码
// JFSF.h
#ifndef JFSF_h
#define JFSF_h

// ... include for Arduino.h and Joystick.h; bunch of defines

namespace JFSF_PRIV
{
class AbstractAction
{
public:
  virtual void loop();
};

/* A Switch that essentially acts as a push button. */
class SwitchContinuousAction : public AbstractAction
{
public:
  SwitchContinuousAction(Joystick_ *joystick, int pin, int btnIndex);
  void loop();

private:
  Joystick_ *_joystick;
  int _pin;
  int _btnIndex;
};

} // namespace JFSF_PRIV

class JFSF
{
public:
  JFSF(Joystick_ *joystick, bool doSendState); // doSendState should be true if Joystick_ does not auto send state.
  void loop();
  void addSwitchContinuous(int inputPin, int btnIndex);

private:
  Joystick_ *_joystick;
  JFSF_PRIV::AbstractAction *_actions[MAX_ACTIONS];
  int _nextActionIndex;
  bool _doSendState;
};

#endif

源文件(修剪):
// JFSF.cpp
#include "Arduino.h"
#include "Joystick.h"
#include "JFSF.h"

#define PRESSED 1
#define RELEASED 0

// Private classes
namespace JFSF_PRIV
{

SwitchContinuousAction::SwitchContinuousAction(Joystick_ *joystick, int pin, int btnIndex)
{
    _joystick = joystick;
    _pin = pin;
    _btnIndex = btnIndex;
    pinMode(_pin, INPUT_PULLUP);
}

void SwitchContinuousAction::loop()
{
    int _state = digitalRead(_pin) == LOW ? PRESSED : RELEASED;
    _joystick->setButton(_btnIndex, _state);
}

} // namespace JFSF_PRIV

JFSF::JFSF(Joystick_ *joystick, bool doSendState)
{
    _joystick = joystick;
    _nextActionIndex = 0;
    _doSendState = doSendState;
}

void JFSF::addSwitchContinuous(int inputPin, int btnIndex)
{
    JFSF_PRIV::SwitchContinuousAction newBtnAction(_joystick, inputPin, btnIndex);
    _actions[_nextActionIndex++] = &newBtnAction;
}

void JFSF::loop()
{
    for (int n = 0; n < _nextActionIndex; n++)
    {
        if (_actions[n])
        {
            _actions[n]->loop();
        }
    }
    if (_doSendState)
    {
        _joystick->sendState();
    }
}

为了完整起见,这是 Arduino 的代码,但它几乎只是声明:
#include <JFSF.h>

// ... A bunch of const declarations used below. These are pretty self explanatory.

// See: https://github.com/MHeironimus/ArduinoJoystickLibrary#joystick-library-api
Joystick_ joystick(HID_REPORT_ID,
  JOYSTICK_TYPE_JOYSTICK, // _JOYSTICK, _GAMEPAD or _MULTI_AXIS
  BTN_COUNT, HAT_SWITCH_COUNT,
  INCLUDE_X_AXIS, INCLUDE_Y_AXIS, INCLUDE_Z_AXIS,
  INCLUDE_RX_AXIS, INCLUDE_RY_AXIS, INCLUDE_RZ_AXIS,
  INCLUDE_RUDDER, INCLUDE_THROTTLE,
  INCLUDE_ACCELERATOR, INCLUDE_BRAKE, INCLUDE_STEERING);

JFSF controller(&joystick, !DO_AUTO_SEND_STATE);

void setup() {
  joystick.begin(DO_AUTO_SEND_STATE);
  controller.addSwitchContinuous(10, 0);  // <-- Doesn't work
  controller.addSwitchContinuous(9, 1);   // <-- Works
}

void loop() {
  controller.loop();
}

引用资料

ArduinoJoystickLibrary(Joystick_ 的来源)可以在这里找到:https://github.com/MHeironimus/ArduinoJoystickLibrary#joystick-library-api

最佳答案

我真的不明白你的代码。请阅读 How to create a Minimal, Complete and Verifiable example 。无论如何,以下肯定是错误的,并且可能是您问题的原因:

void JFSF::addSwitchContinuous(int inputPin, int btnIndex)
{
    JFSF_PRIV::SwitchContinuousAction newBtnAction(_joystick, inputPin, btnIndex);
    _actions[_nextActionIndex++] = &newBtnAction;
}

为了清楚起见,让我们稍微重写一下:
void foo(){
    T bar;
    container[index] = &bar;
}

这里发生的是 bar 在超出范围时被销毁,因此您放入容器的指针指向垃圾。大概在您的代码中的其他地方,您正在取消引用这些指针,这是未定义的行为(也就是任何事情都可能发生)。

长话短说:过度使用指针是 C++ 初学者的常见模式。很可能您应该使 container 成为对象而不是指针的容器,并使用自动内存管理而不是试图对抗它。

关于C++ 引用在函数作用域内创建的实例,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/53752096/

10-11 23:22
查看更多