Почему таймер milis ардуино не работает 2 раза?

Question

[Павел]

Предыстория: есть у меня свет на улице и в доме, на улице свет включаю кнопочкой или датчиком геркона если на улице темно, захотелось мне добавить в дом датчик движения на включение света, для этого пришлось отказаться от delay потому что тормозило весь код на период включения , написал вот такой код, но он работает не корректно, время не так считает как надо, а самое главное работает только 1 раз

потыкайте носом пожалуйста

#define knopka 2  // кнопка
#define rele_dom 3    // пин реле в доме
#define rele_ul 4    // пин реле на улице
#define dsveta 5     // пин датчика света 
#define gerkon 7  // геркон 
#define green_1 A0
#define green_2 A1
#define green_3 A2
#define green_4 A3


unsigned long previousMillis_dom = 0;        // храним время последнего переключения
unsigned long previousMillis_ul = 0;
bool rele_dom_on = false;
bool rele_ul_on = false;

void setup() {
  pinMode(green_1, OUTPUT);
  pinMode(green_2, OUTPUT);
  pinMode(green_3, OUTPUT);
  pinMode(green_4, OUTPUT);

  pinMode(rele_dom, OUTPUT);
  pinMode(rele_ul, OUTPUT);

  digitalWrite(rele_dom, HIGH); // выключаем реле по умолчанию
  digitalWrite(rele_ul, HIGH);

  pinMode(dsveta, INPUT);
  pinMode(gerkon, INPUT);
  pinMode(knopka, INPUT);


}

void loop()
{
  unsigned long currentMillis = millis(); // переменная для таймеров

  // индикация светодиодами
  if (digitalRead(dsveta) == 1) {
    digitalWrite(green_1, HIGH);
  } else {
    digitalWrite(green_1, LOW);
  }

  if (digitalRead(ddvij) == 1) {
    digitalWrite(green_2, HIGH);
  } else {
    digitalWrite(green_2, LOW);
  }

  if (digitalRead(gerkon) == 1) {
    digitalWrite(green_3, HIGH);
  } else {
    digitalWrite(green_3, LOW);
  }

  if (digitalRead(knopka) == 1) {
    digitalWrite(green_4, HIGH);
  } else {
    digitalWrite(green_4, LOW);
  }
  //

  // принудительный свет на улице от кнопки
  if (!rele_ul_on && (digitalRead(knopka) == 1)) {
    digitalWrite(rele_ul, LOW); // включить
    rele_ul_on = true; // запомнить состояние
    previousMillis_ul = currentMillis; // запомнить момент включения
  }
  if (rele_ul_on && currentMillis - previousMillis_ul > 5000) { // время в милисекундах

    digitalWrite(rele_ul, HIGH); // прошло больше N секунд - выключаем
    rele_ul_on = false;
  }
  //

  // включение света на улице от геркона и датчика света
  if (digitalRead(gerkon) == 0 && digitalRead(dsveta) == 1) {
    digitalWrite(rele_ul, LOW); // включить
    rele_ul_on = true; // запомнить состояние
    previousMillis_ul = currentMillis; // запомнить момент включения
  }
  if (rele_ul_on && currentMillis - previousMillis_ul > 7000) { // время в милисекундах
    digitalWrite(rele_ul, HIGH); // прошло больше N секунд - выключаем
    rele_ul_on = true;
  }
}

Comments

[kalapanga]

Я правильно понимаю, что раньше было два способа включить свет - кнопка и где-то установленный геркон, а сейчас должно стать три - кнопка, геркон, датчик движения?
В коде Вы проверяете геркон и датчик одним выражением. Их работа действительно как-то взаимосвязана или они работают независимо друг от друга?
Сдаётся мне Вы условие какое-то не то проверяете.

[Павел]

kalapanga, геркон работает только когда нет света, а кнопка независимо от остальных датчиков, но это не важно сейчас. Хоть бы с кнопки первое реле ключать, а с геркона второе , первое на 5000мс, а второе на 7000мс к примеру

[kalapanga]

Павел, Так это же у Вас вроде работало - в прошлом вопросе как раз фигурировали два реле. Только там были вроде как две кнопки, а теперь кнопка и геркон, но это невелика разница. Если убрать из кода всё про датчик движения, то код правильно работает или нет?
Если нет, то приведите здесь конечный код для кнопки и геркона, без датчика движения.
Или что-то ещё в условиях поменялось?

[Павел]

kalapanga, в прошлом коде были светодиоды как подопытные вместо реле, мне казалось все работало, но как дело до релюх дошло оказалось что нет, я всю ночь сидел гуглил, думал и дебажил код, путем отладки в консоль я понял что не выйдет у меня сделать это, по крайней мере с моим текущим уровнем знаний

[kalapanga]

Павел, Да ладно! Чем уж там релюшки-то удивили? Про релюшки, по сравнению со светодиодами, дополнительно нужна следующая информация. Первое - это отдельно взятые реле или модули реле, которые непосредственно к ардуино подключать можно? И второе - каким уровнем сигнала они управляются (высокий или низкий уровень нужно подать, чтобы реле включить).
И давайте последний вариант кода с двумя кнопками и двумя реле.
Там не работать-то нечему.

Answer 1

[Дмитрий Павлов]

millis()
Возвращает количество миллисекунд с момента начала выполнения текущей программы на плате Arduino.
Это количество сбрасывается на ноль, в следствие переполнения значения, приблизительно через 50 дней.

Возможно, у вас возникают проблемы из-за переполнения, плюс вы переопределяйте

unsigned long currentMillis = millis(); // переменная для таймеров

переменную currentMillis
с началом каждого прохода void loop(). Также советую вам пересмотреть

// включение света на улице от геркона и датчика света
  if (digitalRead(gerkon) == 0 && digitalRead(dsveta) == 1) {
    digitalWrite(rele_ul, LOW); // включить
    rele_ul_on = true; // запомнить состояние
    previousMillis_ul = currentMillis; // запомнить момент включения
  }

, так как

previousMillis_ul = currentMillis; // запомнить момент включения

будет присваивать время старта loop.

Comments

[Павел]

что вы пытаетесь этим сказать? Надо обнулять это значение или значение previousMillis_ul ? Или совсем не использовать?

[Дмитрий Павлов]

Павел, То что millis может косячить когда произойдет переполнение и учитывать это в работе программы

[Дмитрий Павлов]

Павел, Как вариант, когда требуется длительный период мониторить временной интервал, используют часы реального времени, но зависит от того , насколько для вашего устройства критичен сбой (сброс) таймера контроллера.

[Павел]

Дмитрий Павлов, у меня там будет таймер максимум 15 минут, но их будет 2, какой вариант мне подойдет? То есть пока первый таймер работает допустим на 10 минут, во время него чтоб можно было запустить второй на 2 минуты 4 раза к примеру

[Дмитрий Павлов]

Павел, Как вариант, если использовать часы реального времени, то можно создать две переменные DataTime home,strit,prow1,prow2 и перед запуском 1 к примеру считывать текущее время, присваивать переменной prow1 текущее значение 1 переменной с прибавленные интервалом и когда текущее время дойдет до контрольного значения, выполнять действие (включать/выключать свет), альтернативно 2 таймер реализовать через вторую основную и контрольную переменную , при этом таймер нужен всего один.