c++ - 奇怪的 C++ Arduino 串行行为

Question

我正在为我的 arduino 编写一些代码以从模拟操纵杆打印 4 个按钮状态和 X、Y 坐标，但我有奇怪的行为。

代码如下:

  int x,y;
  x = analogRead(joy_x);
  y = analogRead(joy_y);
  char* buf = new char[4];
  sprintf(buf, "%4d", x);
  Serial.print("X:");
  Serial.print(buf);
  Serial.print(" Y:");
  sprintf(buf, "%4d", y);
  Serial.println(buf);

  int one,two,three,four;
  one = digitalRead(touchOne);
  two = digitalRead(touchTwo);
  three = digitalRead(touchThree);
  four = digitalRead(touchFour);

  // create an array of states set to the default values
  char states[4] = {'X', 'X', 'X', 'X'};
  // change each value if it is not in the default state
  if(one   == 1) states[0] = 'O';
  if(two   == 1) states[1] = 'O';
  if(three == 1) states[2] = 'O';
  if(four  == 1) states[3] = 'O';
  // output the states to the OLED display
  //display.print(states);
  Serial.print(states);

当它运行时，串行输出如下所示:

XXXX3X: 515 Y: 520
XXXX3X: 516 Y: 520
XXXX3X: 516 Y: 519
XXXX3X: 516 Y: 520
XXXX3X: 515 Y: 520
XXXX3X: 516 Y: 519
XXXX3X: 516 Y: 520
XXXX3X: 515 Y: 519
XXXX3X: 516 Y: 520

尽管 X、Y 应该在 XXXX 之前并且数字 3 不知从何而来。 希望能解开这个谜团，谢谢。

Answer 1

您的代码中有几个缓冲区溢出。

  char* buf = new char[4];
  sprintf(buf, "%4d", x);
  ...
  sprintf(buf, "%4d", y);

这需要为 sprintf 添加的空终止符留出空间。

  char* buf = new char[5];

也更容易:无需使用此处的免费商店。

  char buf[5];

同样的事情在这里:

  char states[4] = {'X', 'X', 'X', 'X'};
  ...
  Serial.print(states);

我们需要添加一个空终止符以使其成为一个有效的字符串。

  char states[5] = {'X', 'X', 'X', 'X', '\0'};

这应该可以解决即时内存问题。