Выключение светодиода

1. pushpressed устанавливается, но никогда не сбрасывается.

Он устанавливается в stopPins() и stopPins2 (), если вход HIGH. Поскольку он никогда не сбрасывается обратно в ноль, как только он установлен, логика в startBuzz и startBuzz2 будет следовать за одним проходом. Вы создали защелкнутый переключатель.

---

Общие вещи

---

2. Непоследовательное именование: затрудняет чтение кода

Выберите snake_case или camelCase или smashitalltogether (не рекомендуется) и придерживайтесь его :) То же самое относится и к "типам" переменных: некоторые вещи называются xxx pin xxx, некоторые-xxx state xxx, некоторые-нет. Это яснее, когда одно и то же-вещи называются-одно и то же.

3. Дублирование: затрудняет понимание кода

Наряду с именованием (по существу) дублированный код затрудняет отслеживание проблем, как вы заметили, пытаясь отладить эту проблему. Существует масса способов уменьшить код, чтобы сделать его понятным. Ореховая скорлупа состоит в том, чтобы определить различия в подобном коде и сделать меньше сразу.

(И если вы хотите довести его до логического завершения, то упаковка каждого сигнала тревоги в класс устранит все дублирование и большинство параметров функции :)

---

Досадное отвлечение объяснений

---

Давайте посмотрим на startBuzz и startBuzz2.

void startBuzz() {
  if (pushpressed == 0) {
    if (state_led == LOW) {
      state_led = HIGH;
      digitalWrite(led1_pin, state_led);
    }
  } else if (pushpressed == 1) {
    state_led = LOW;
    digitalWrite(led1_pin, state_led);
  }
}

void startBuzz2() {
  if (pushpressed == 1) {
    if (state_led2 == LOW) {
      state_led2 = HIGH;
      digitalWrite(led2_pin, state_led2);
    }
  } else if (pushpressed == 0) {
    state_led2 = LOW;
    digitalWrite(led2_pin, state_led2);
  }
}

Единственное, что здесь отличается, - это вывод светодиода и состояние светодиода. Давайте нормализуем наше именование, чтобы все стало ясно, используя startBuzz в качестве примера. Мы также предположим, что pushPressed может быть только 0 или 1 (ВЫСОКИЙ или НИЗКИЙ):

void startBuzz() {
  if (pushPressed == LOW) {
    if (led1State == LOW) {
      led1State = HIGH;
      digitalWrite(led1Pin, led1State);
    }
  } else {
    led1State = LOW;
    digitalWrite(led1Pin, led1State);
  }
}

Мы можем сделать эту работу для любого из светодиодов, выполнив две вещи: принимая pin и текущее состояние и возвращая новое состояние. Зачем возвращать новое состояние вместо того, чтобы установить его здесь? Это упрощает основной код (и позволяет избежать передачи ссылки).

void startBuzz(int ledPin, int ledState) {
  int newLedState = ledState;
  
  if (pushPressed == LOW) {
    if (ledState == LOW) {
      newLedState = HIGH;
      digitalWrite(ledPin, newLedState);
    }
  } else {
    newLedState = LOW;
    digitalWrite(ledPin, newLedState);
  }
  
  return newLedState;
}

Затем, когда мы вызываем startBuzz или startBuzz2, мы заменяем текущий код следующим:

led1State = startBuzz(led1Pin, led1State);

Мы еще не закончили.

Поскольку мы сохраняем исходное состояние ledState, мы можем удалить дублированные вызовы digitalWrite; в худшем случае у нас будет лишняя запись при нажатии pushPressed != LOW && ledState != HIGH.

Поскольку все условия, кроме одного, приводят к НИЗКОМУ значению, нам даже не нужно сохранять исходное состояние. И так как это два условия, чтобы перевернуть его HIGH, мы можем объединить их. И если мы вытащим цифровую запись, то на самом деле просто рассчитаем следующее состояние-оно не зависит от светодиодного штифта.

void getNextLedState(int ledState) {
  return (pushPressed == LOW && ledState == LOW) ? HIGH : LOW;
}

А сайты вызовов выглядят примерно так:

led1State = getNextLedState(led1State);
digitalWrite(led1Pin, led1State);

Аналогично для сигналов тревоги: создайте функцию checkAlarm, которая принимает часы и минуты и возвращает логическое значение. В цикле вызовите checkAlarm(h1, m1) и, если true, вызовите runAlarm1(). (Даже это можно было бы уменьшить еще больше, но вы поняли идею.) стопПинс отличается только строками, записанными на жидкокристаллический дисплей. И так далее.