Почему не работает код?

Каждые 10 сек включить светодиод на 3 сек.

Код (C++):

#define LED_PIN    13
unsigned long x = 10000;
unsigned long y =  3000;

void setup()
{
  pinMode(LED_PIN, OUTPUT);
}

void loop()
{
  static unsigned long OnTime = millis();
  static unsigned long OffTime = 0;

  if (millis() - OnTime > x) // каждые X сек включаем
  {
    OnTime = millis();
    OffTime = millis() + y; // задаем время отключения
    digitalWrite(LED_PIN, HIGH);
  }

  if (millis() > OffTime)    // по прошествии Y сек отключаем
  {
    OnTime = millis();        // сбрасываем счетчик
    OffTime = millis() + x;
    digitalWrite(LED_PIN, LOW);
  }
}
 

Однако включается и не отключается.
Что не так?

 

Код почти правильный (светодиод должен моргать), вот только период будет увеличиваться с каждой итерацией на 3 секунды.
Что бы период был точно 10 сек, нужно исправить функцию 'loop()':

Код (C++):

void loop()
{
  static unsigned long OnTime = millis();
  static unsigned long OffTime = OnTime + y;

  if (millis() - OnTime > x) // каждые X сек включаем
  {
    OnTime = millis();
    OffTime = millis() + y; // задаем время отключения
    digitalWrite(LED_PIN, HIGH);
  }

  if (millis() > OffTime)    // по прошествии Y сек отключаем
  {
    digitalWrite(LED_PIN, LOW);
  }
}

Если у вас не моргает, то проблема скорее в схеме, в плате Arduino или в компиляторе....

 

СПАСИБО!
Все работает, как и задумано.

 

Не могли бы вы Объяснить подробно эту конструкцию:

Код (C++):

static unsigned long OnTime = millis();

Для каких целей тут слово static и что оно делает?

 

Какой вычурный код у вас получился! А вообще, если программа ичего больше не делает то сойдет и такое:

Код (C++):

#define LED_PIN    13
unsigned long x = 10000;
unsigned long y =  3000;

void setup()
{
  pinMode(LED_PIN, OUTPUT);
  digitalWrite(LED_PIN, LOW);
}

void loop() {
  delay(10000);
  digitalWrite(LED_PIN, HIGH);
  delay(3000);
  digitalWrite(LED_PIN, LOW);
}

Это же блинк обычный с несимметричными паузами. Да и через millis() можно было как то проще сделать.

 

А почему не вынести в глобальные переменные, тем более, если автору надо будет изменить из другой функции период, он сможет это сделать без проблем.

 

Нате вам такой вариант еще:

Код (C++):

#define LED_PIN    13
unsigned int x = 10000;
unsigned int y =  3000;
unsigned long timerXY;
bool v = 0;

void setup()
{
  pinMode(LED_PIN, OUTPUT);
  digitalWrite(LED_PIN, LOW);
  timerXY = millis() + x;
}

void loop() {
  if (millis() > timerXY) {
    v = !v;
    digitalWrite(LED_PIN, v);
    timerXY = millis() + (v ? y : x);
  }
}

По мне так ничего плохого в глобальных переменных нет. Если не путаться в них, конечно.

 

Плохо не понимать, что делаешь. А в глобальных переменных особо плохого нету.
Ну и из опыта:

Код (C++):

if (millis() - OnTime > x)

после, по моему, 52 дней работы, этот код перестанет работать и придется либо перезагрузить ардуину, либо переделывать код.

 

Так она сбрасываться будет каждую итерацию loop().

 

Один раз в месяц глюкнет мигалка, или темно будет не 10 сек или светло не 3 сек, а меньше. Вот и все.

 

В вашем случае(коде) - да.
А в случае нахождения разности, она в if вообще перестанет попадать после срока достижения максимального значения, и, соответственно, перестанет устанавливать OnTime, который будет огромный, а milis() будет мал.

 

Ну я все же предпочитаю глобальные флаги в таких случаях. )

 

А еще мой вариант экономит машинное время, вычисляя разницу один раз за период, а не каждый раз. )

Насчет зависания даже сразу не соображу, разница в примере будет огромным минусом и она будет меньше х. Что это значит?

 

Я так делаю:

Код (C++):

timeoff = millis() + INTRVAL;
...
if (millis() > timeoff) {
..
}
 

Работает и правильно и быстро.

 

В общем, там что то с переполнением.
Правда я сейчас подумал, какая разница, когда оно будет происходить при вычитании, или при сложении.
Одно точно: при сложении оно произойдёт один раз и пока milis() не переполнится, программа попадёт в цикл.
А вот при вычитании поведение программы менее очевидно.