Arduino.ru

Команда goto не дает опрашивать состояния кнопок.

Куто вас вообще надоумил?

Да и с вложенными условиями у вас похоже что-то не то...

чето я не понял как именно устройство должно работать....

жмёшь кпопы и оно работает.

Станднртаная кухонная вытяжка из котороый был выкинут вентилятор (стоит приточка).

По результатам переделки - есть 4 кнопки (используем пока 3) и два реле (220В)

Одно реле отвечает rel2 отвечает как на всех вытяжках за освещение. Т.е нажал кнопку btn4 - свет есть. Нажал - света нет.

Второе реле rel 1 подает 220В на клапан вентиляции (при подаче закрывается - отрубая другие ветки).

И вот по нажатию кнопки btn2 и btn3 подает напряжение на 1ое реле на полчаса и час соответсвенно.

то есть 4 кнопки....

первая - включает и выключает реле

вторая - включает и выключает клапан вытяжки

третья - включает клапан вытяжки на пол часа

четвертая - включает клапан вытяжки на час

я правильно все понял?????

Да.

Но "вторая - включает и выключает клапан вытяжки" - сейчас в коде не прописана

а если вытяжка уже работает (по кнопкам 2-4, неважно кто инициатор), и повторно нажата кнопка 2 или 3, то что делать? Остановить или продлить еще на полчаса/час?

На самом деле не так важно. Сейчас бы запустить базовый функционал.

В идеале думаю - продлить на полчаса.

Отменить все и вернутся в исходное состояние думаю как раз будет со временем 4я кнопка

Получился вот такой код

enum { nkinds=2, debounced=5 }; // # of activity kinds;  debounce-criterion
const byte outPins[nkinds] = {11}; // Output pin numbers
const byte inPins[nkinds]  = {2, 3}; // Input pin numbers
const unsigned int actLens[nkinds] = {1800000, 3600000}; // Activity lengths, ms
byte states[nkinds] = {0, 0};
unsigned long actEnds[nkinds] = {0, 0};
int lightState = 0;
const int lightPin = 4;
const int lightrelePin =  13;


void setup() {
// Set input/output modes of pins
  for (byte i=0; i<nkinds; ++i) {
    pinMode(outPins[i], OUTPUT);
    pinMode(inPins[i], INPUT);
  }
  pinMode(lightrelePin, OUTPUT);
  pinMode(lightPin, INPUT);
}

// Once per millisecond, for each kind of activity
// check for new beginning or deal with ongoing action
void loop() {
  lightState = digitalRead(lightPin);
  if (lightState == HIGH) {
    digitalWrite(lightrelePin, HIGH);  
  } 
  else {
    // turn LED off:
    digitalWrite(lightrelePin, LOW); 
  }
  
  unsigned long now = millis();
  for (byte i=0; i<nkinds; ++i) {
    if (states[i] < debounced) {  // Is activity off?
      // It's off, look for a turnon
      if (digitalRead(inPins[i])) {
        ++states[i];        // Button is closed at the moment
        // See if button has been debounced
        if (states[i] >= debounced) { // If so, start its action
          digitalWrite(outPins[i], HIGH);
          actEnds[i] = now + actLens[i];
        }
      }
      else {
        states[i] = 0;      // Button is not closed at moment
      }
    }
    else { // It's on, see if end-time has arrived
      if (now >= actEnds[i]) {
        digitalWrite(outPins[i], LOW); // Clear output when done
        states[i] = 0;        // Return to button-seeking state
      }
    }
  }
  while (now == millis()) {};       // Finish current millisecond
}

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

for (byte i=0; i<nkinds; ++i) {
pinMode(outPins[i], OUTPUT);

При таком раскладе получаем обращение к outPins[1], т.е. второму элементу массива, а у тебя изначально задан только один элемент: {11}.

Аналогично обращаемся всегда ко второму элементу states[i] , но никогда не используем первый элемент. Зачем тогд тратить под него память? 

В примере изначально был массив из 3х элекментов. 3 кнопки - 3события на выводе. У меня же элемент один.

Здесь проблемы только в чистоте кода - или все таки код получается рабчоим?