Проблема с таймером Arduino, почему не работает?

Question

[Tiamon]

Не могу понять, почему не работает код

Выдержки, имеющие отношение к вопросу:

Есть небольшой проект, реле включаются по таймеру. Решил изменить по быстрому.
Сделать так, что бы после включения ардуинки, реле включалось сразу на заданное время, после чего через заданный интервал включалось снова на заданное время и так по циклу.

Вроде сделал всё очень просто, но не работает, не включается, не могу понять почему.

const long DurationCh_2 = 120;                  //ДЛИТЕЛЬНОСТЬ срабатывания реле

const long StartViaCh_2 = 200;                  //Включится через 

int Xer = 0;
int StartRelCn_2;

void loop()                                        // ПРОГРАММЫй безусловный ЦИКЛ
{
  DateTime myTime = RTC.now();                     //Читаем данные времени из RTC

  long utime = myTime.unixtime();                  //сохраняем время в формате UNIX
  utime %= 86400;                                  //Сохраняем в переменной остаток деления на кол-во секнд в сутках,
                                                   //Это дает количество секунд с начала текущих суток


    
   while(Xer < 1)				//Должно выполнятся один раз
      {
        Xer ++;
        StartRelCn_2 = utime;          		//Присвоить текущее время
      }

      
   if ((utime >= StartRelCn_2) &&
       (utime < (StartRelCn_2+DurationCh_2)))
      {
        digitalWrite(RelayChn2,LOW);                 //Устанавливаем на 2 входе релейного модуля НИЗКИЙ уровень - реле срабатывает
        StartRelCn_2 = StartRelCn_2 + StartViaCh_2;  // Повторно включаем через заданный интервал времени
      }
    else
      {
          digitalWrite(RelayChn2,HIGH);              //Устанавливаем на 2 входе релейного модуля ВЫСОКИЙ уровень - реле выключается
      }

Answer 1

[Tiamon]

Разобрался сам, дело оказалось в "long" то есть мне нужно было указать переменную в этом "формате"
Так как у меня время в нем

void loop()                                        // ПРОГРАММЫй безусловный ЦИКЛ
{
  long utime = myTime.unixtime();                  //сохраняем время в формате UNIX

То есть верный код такой:

int long DurationCh_2 = 120;                  //ДЛИТЕЛЬНОСТЬ срабатывания реле
int long StartViaCh_2 = 200;                  //Включится через 
int Xer = 0;
int long StartRelCn_2;

Тогда идёт корректное сравнение и присваивание переменных

Answer 2

[Александр Гусев]

Верни всё как было, а одноразовое включение сделай в цикле инициализации :)
Xer + 1; ничего не сделает с переменной Xer. Эта строка вычислит результат, но никуда его не присвоит.
Xer ++; или Xer += 1; попробуй ради интереса. Особо в логику не вникал)

Comments

[Tiamon]

Да, ошибка, исправил, но всё равно не работает.

Не понял про цикл инициализации, если можно, подробнее

[Александр Гусев]

Tiamon, помимо ф-ции loop(), есть функция setup(), где конфигурируются выводы и периферия (порты на ввод-вывод и т.п.). Она вызывается только один раз после запуска МК.

[Tiamon]

Александр Гусев, это не будет работать, так как переменная utime есть в loop(), я не могу получить текущее время с модуля реального времени в setup()

[Александр Гусев]

Tiamon, сделать глобальной не запрещено :)
Ну хорошо, что сам разобрался. Я косяк в инкременте увидел, а в алгоритм как-то влом вникать уже было. :)

[Tiamon]

Александр Гусев, сделал глобальной а дальше то что? Если в setup считывается один раз ДО того как ей присвоено время, которое мне нужно считать\назначить один раз и до того как инициирован модуль из которого оно считывается.

Без обид, но если тебе в лом разбираться, зачем тогда пишешь? )