【问题标题】:Arduino stops executing after about 9 loopsArduino 在大约 9 个循环后停止执行
【发布时间】:2014-07-06 05:38:31
【问题描述】:

我有一个草图,它将在停止之前执行大约 9 次。我尝试释放我在主循环中声明的所有数组,但这在停止之前将循环数减少到 2。

根据 MemoryFree 的说法,我让它检查可用内存,看看它是否已经用完,但似乎没有泄漏。也许我编码错了?

// nrf24_reliable_datagram_client.pde
// -*- mode: C++ -*-
// Example sketch showing how to create a simple addressed, reliable messaging client
// with the RHReliableDatagram class, using the RH_NRF24 driver to control a NRF24 radio.
// It is designed to work with the other example nrf24_reliable_datagram_server
// Tested on Uno with Sparkfun WRL-00691 NRF24L01 module
// Tested on Teensy with Sparkfun WRL-00691 NRF24L01 module
// Tested on Anarduino Mini (http://www.anarduino.com/mini/) with RFM73 module
// Tested on Arduino Mega with Sparkfun WRL-00691 NRF25L01 module

#include <RHReliableDatagram.h>
#include <RH_NRF24.h>
#include <SPI.h>
#include <DHT.h>
#include <MemoryFree.h>

DHT dht(5, DHT22);
#define CLIENT_ADDRESS 1
#define SERVER_ADDRESS 2

// Singleton instance of the radio driver
RH_NRF24 driver;
// RH_NRF24 driver(8, 7);   // For RFM73 on Anarduino Mini

// Class to manage message delivery and receipt, using the driver declared above
RHReliableDatagram manager(driver, CLIENT_ADDRESS);

void setup() 
{
  pinMode(13,OUTPUT);
  Serial.begin(9600);
  if (!manager.init())
    Serial.println("init failed");
  // Defaults after init are 2.402 GHz (channel 2), 2Mbps, 0dBm
  dht.begin();
}

uint8_t data[70];
// Dont put this on the stack:
uint8_t buf[RH_NRF24_MAX_MESSAGE_LEN];

void looop()
{
  byte data[2];

  getdat(&data[0]);
}

void getdat(byte *pdata)
{
  pdata[0] = 'a';
  pdata[1] = 'b';
}

double getAvgRead(int readDelay, int iterations, double* result){
  int startMillis = millis();
  int iterationStart, i;
  for (i = 0 ; i < iterations; i++){
    iterationStart = millis();

    digitalWrite(13,HIGH);
    result[0] += dht.readTemperature(true);
    //Serial.println(result[0]);
    result[1] += dht.readHumidity();
    digitalWrite(13,LOW);

    delay(readDelay-millis()+iterationStart);
  }

  result[0] /= i; //divide by iterations to produce the average
  result[1] /= i;
  //Serial.println(result[0]);
}

int i = 0;
void loop()
{
  Serial.println("Sending, loop " + String(i++) + ", mem: " + String(freeMemory()));
  //int s = millis();
  double values[2] = {0,0};
  getAvgRead(2020,1,values);
  //Serial.println((millis()-s));

  uint8_t data[28];
  char str[28];
  sprintf(str,"U,DHT;T,%d;H,%d", (int)(values[0] * 100.0), (int)(values[1] * 100.0));
  memcpy(data,(uint8_t*)str,sizeof(str));
  Serial.println((char*)data);

  // Send a message to manager_server
  if (manager.sendtoWait(data, sizeof(data), SERVER_ADDRESS))
  {
    // Now wait for a reply from the server
    uint8_t len = sizeof(buf);
    uint8_t from;   

    manager.recvfromAckTimeout(buf, &len, 2000, &from);
  }
  else
    Serial.println("sendtoWait failed");
}

引入 getAvgRead 函数导致了这个错误。我可能错过了什么?

编辑:此草图旨在从 DHT 传感器读取多次,平均读数,然后将其传输到另一个 arduino。

【问题讨论】:

    标签: c arduino


    【解决方案1】:

    我不太确定你想用这个草图实现什么,但看过代码后我怀疑你的问题将是:

    delay(readDelay-millis()+iterationStart);
    

    我怀疑您正试图使每次迭代精确地花费readDelay 毫秒。如果迭代花费的时间超过readDelay,则延迟将为负数。我假设一个负的delay 参数将环绕成一个非常大的无符号整数。也许您应该检查计算的延迟,如果它是负数,则跳过延迟。

    此外,正如 @m0nk3y 在 cmets 中建议的那样,您还应该将您的 readDelaystartMillisiterationStart 变量更改为 unsigned longs。

    这个delay 肯定是我首先要开始寻找的地方。当然,我也完全有可能找错树了。

    【讨论】:

    • 该行告诉它在剩余的交互持续时间内暂停。应该和delay(readDelay-(millis()-iterationStart))一样。它计算自迭代开始以来经过了多少毫秒,然后等待迭代持续 X 毫秒。 readDelay 名称错误,因为我更改了参数并忘记了它。
    • 是的,我刚刚意识到这一点。如果您只是将延迟设置为 readDelay,会发生什么情况。草图是否停止运行。
    • 如果你硬编码延迟值会发生什么,也许只是将它设置为readDelay?我还更新了我的答案,建议延迟可能最终是负面的。
    • 我只是将其更改为 delay(readDelay) 并且它执行的时间是以前的两倍,这可能就是它。我难住了。我正在尝试delay(readDelay-(millis()-iterationStart)) 看看是否效果更好。
    • 将您的millis变量更改为unsigned long有帮助吗?来自link:请注意,millis 的参数是无符号长整数,如果程序员尝试使用其他数据类型(例如 int)进行数学运算,可能会产生错误。
    猜你喜欢
    • 2019-06-22
    • 1970-01-01
    • 2022-01-16
    • 2014-05-30
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    相关资源
    最近更新 更多