Question

我正在研究Arduino的LED推子。为此，我写了一个定义所有内容的类。但是从昨天开始我遇到了问题，我无法解决。到目前为止，这是我的代码：

class LED {
    int pins[4];
    int r = 0;
    int g = 0;
    int b = 0;
    int task = 1;
    int taskgoal = 255;
  public:
    void newTask() {
      ...
    }
    void newGoal() {
      ...
    }
    boolean taskFinished() {
      ...
    }
    void runTask() {
      if (taskFinished()) {
        newTask();
        newGoal();
        cout << "New Task and new Goal.\n";
      }
      int current;
      switch (task) {
        ...
      }
      if (current < taskgoal) {
        switch (task) {
          case 1:
            cout << r << "+1" << "\n";
            this->redefine(this->r + 1, this->g, this->b);
            cout << r << "+1" << "\n";
            break;
          case 2:
            cout << g << "+1" << "\n";
            this->redefine(this->g, this->g + 1, this->b);
            break;
          case 3:
            cout << b << "+1" << "\n";
            this->redefine(this->r, this->g, this->b + 1);
            break;
        }
      } else if (current > taskgoal) {
        ...
      }
      analogWrite(pins[0], this->r);
      analogWrite(pins[1], this->g);
      analogWrite(pins[2], this->b);
      return;
    }
    void ledinit(int p[]) {
      this->r = random(1, 255);
      this->g = random(1, 255);
      this->b = random(1, 255);
      for (int i = 1; i < 3; i++) {
        pins[i] = p[i];
        pinMode(pins[i], OUTPUT);
      }
      pins[3] = p[3];
      analogWrite(pins[0], this->r);
      analogWrite(pins[1], this->g);
      analogWrite(pins[2], this->b);
      analogWrite(pins[3], LOW);
      newTask();
      newGoal();
      runTask();
      return;
    }
    void redefine(int rr, int gg, int bb) {
      r = rr;
      g = gg;
      b = bb;
      return;
    }
};
std::vector< LED >leds;

void setup() {
  Serial.begin(9600);
  LED l1;
  int l1pins[4] = {11, 9, 8, 10};
  l1.ledinit(l1pins);
  leds.push_back(l1);
}

void loop() {
  for (LED l : leds) {
    l.runTask();
    cout << "Run Task" << "\n";
  }
  delay(100);
}

（Pastebin：http://pastebin.com/pQMEkG1a）在第76行中，我尝试使用我的函数redefine将r加1。我也直接使用this->r++;或this->r += 1;进行了尝试，但所有结果都相同。它会将1添加到r变量中。但是在下一个循环中，它与开始时的值相同。这是输出的样子：

Run Task
45+1
46+1
Run Task
45+1
46+1
Run Task
45+1
46+1

但为什么每次都是45？我认为这是一个非常容易的错误，但我找不到它。

Answer 1

查看代码，我希望您的问题在for (LED l : leds)

之内

这个基于范围的循环遍历LED矢量，从每个元素中获取一个副本，您将在该副本上运行任务。

我建议编写以下内容：for (LED &l : leds)以防止复制。

由于您的编译器似乎支持C ++ 11，您可能希望以后通过向类LED添加以下成员函数来防止类似问题：LED(const LED &) = delete;以及可能来自{{3}的其他几个}。这将阻止来自led的任何副本。可能你需要一些move-constructor才能在向量中添加LED。

类中的变量本身是重置的

1 个答案: