Форум arduino.ua

sxstalker пише:

заменил digitalWrite(13, LOW); на digitalWrite(13, HIGH);  в самом начале
первый цикл все работает верно

ладно если Вам это сложно, может так будет проще для понимания

unsigned long last_time;

void setup() {
  pinMode(13, OUTPUT);
  digitalWrite(13, LOW); 

}

bool on = false; // поточний стан. з початку - вимкнено;

void loop() {
  unsigned long now = millis();
  static unsigned char task=0;
switch (task)
  {
	case : 0  
	   digitalWrite(13, HIGH); // вмикаєм 
       if (now - last_time >= 2000) { // пройшло 1000 мс ?
       last_time = now;
       on = true; // зберігаєм поточний стан, що увімкнено
       }
       break; 
    case : 1 
          digitalWrite(13, LOW); // вимикаєм       
       if (now - last_time >= 200) { // пройшло 300 мс ?
          last_time = now;
          on = false; // зберігаєм поточний стан, що вимкнено
          }
       break; 
    case : 2    
          digitalWrite(13, HIGH); // вмикаєм	
       if (now - last_time >= 500) { // пройшло 1000 мс ?
          last_time = now;
          on = true; // зберігаєм поточний стан, що увімкнено
        }
       break; 
    case : 3 
           digitalWrite(13, LOW); // вимикаєм	
        if (now - last_time >= 2000) { // пройшло 300 мс ?
           last_time = now;
           on = false; // зберігаєм поточний стан, що вимкнено
        }
       break; 
    default :
       break; 	
  } 


}

Ця конструкція:

sxstalker пише:

  if (!on) {
...
  } else if (on) {
...
  } else if (!on) {
...
  } else if (on) {
...
  } 

не має сенсу. Якщо перша умова "if (!on)" виконалась, то решта вже не виконуються. А якщо не виконалась, то обовʼязково виконається друга - "if (on)", бо вона протилежна першій. А дві останні - ніколи не виконаються.

В попередньому прикладі у вас було всього два стани: світлодіод вимкнено, і світлодіод увімкнено. Тому було достатньо змінної типу bool, у якої всього два можливих значення.

В останньому прикладі у вас чотири стани:
* вимкнено перший раз (на 2000 мс)
* увімкнено перший раз (на 200 мс)
* вимкнено другий раз (на 500 мс)
* увімкнено другий раз (на 2000 мс)

Тому для збереження стану потрібна змінна такого типу, щоб могла приймати як мінімум 4 значення. Для простоти візьмем int. Стани будем нумерувати від 0 до 3. І для ясності перейменуєм змінну з "on" на "state".

int state = 0; // поточний стан. з початку - "0";

void loop() {
  unsigned long now = millis();

  if (state == 0) { // в стані "0"?
    if (now - last_time >= 2000) {
      digitalWrite(13, HIGH);
      last_time = now;
      state = 1; // переходим в стан "1"
    }
  } else if (state == 1) { // в стані "1"?
     if (now - last_time >= 200) {
      digitalWrite(13, LOW);
      last_time = now;
      state = 2; // переходим в стан "2"
    }
  } else if (state == 2) { // в стані "2"?
    if (now - last_time >= 500) {
      digitalWrite(13, HIGH);
      last_time = now;
      state = 3; // переходим в стан "3"
    }
  } else if (state == 3) { // в стані "3" ?
     if (now - last_time >= 2000) {
        digitalWrite(13, LOW);
       last_time = now;
       state = 0; // переходим в початковий стан "0"
    }
  }
}

Абсолютно те ж саме, тільки switch/case замість if/else. Так більш читабельно:

int state = 0; // поточний стан. з початку - "0";

void loop() {
  unsigned long now = millis();

  switch (state)
  {
    case 0:
      if (now - last_time >= 2000) {
        digitalWrite(13, HIGH);
        last_time = now;
        state = 1;
      }
      break;
    case 1:
      if (now - last_time >= 200) {
        digitalWrite(13, LOW);
        last_time = now;
        state = 2;
      }
      break;
    case 2:
      if (now - last_time >= 500) {
        digitalWrite(13, HIGH);
        last_time = now;
        state = 3;
      }
      break;
    case 3:
      if (now - last_time >= 2000) {
        digitalWrite(13, LOW);
        last_time = now;
        state = 0; // переходим в початковий стан "0"
      }
      break;
  }
}

Тепер зверніть увагу на те, що дії при переході між станами практично однакові. Відрізняються тільки
тривалості: 2000, 200, 500, 2000
значення, що передається у digitalWrite: HIGH, LOW, HIGH, LOW
номер наступного стану: 1, 2, 3, 0, 1, 2 ...

Таким чином можна використати один і той же код для циклічного переходу між станами. Тривалості та значення для digitalWrite помістимо у масиви, а значення з масивів вибиратимем по номеру стану:

unsigned int dur[4] = { 2000, 200, 500, 2000 };  // масив тривалостей (duration) для кожного стану
byte led[4] = { HIGH, LOW, HIGH, LOW };          // масив значень digitalWrite для кожного стану
int state = 0; // поточний стан. з початку - "0";

void loop() {
  unsigned long now = millis();

  if (now - last_time >= dur[state]) { // якщо час поточного стану вийшов
    digitalWrite(13, led[state]);      // вмикаєм або вимикаєм світлодіод
    last_time = now;
    state = state + 1; // переходим у наступний стан
    // але максимально можливий номер стану - "3"
    // якщо це був останній - переходим у початковий стан "0"
    if (state > 3) {
      state = 0;
    }
   }
}

Тепер зверніть увагу, що в масиві led всього два варіанти значень для digitalWrite: HIGH і LOW, і вони чергуються. Можна обійтись без цього масиву. Як його позбутись - вам в якості самостійного завдання

как-то так...

unsigned long last_time;

void setup() {
  pinMode(13, OUTPUT);
  digitalWrite(13, LOW); 

}

bool on = false; // поточний стан. з початку - вимкнено;

void loop() {
  unsigned long now = millis();
  static unsigned char task=0;
switch (task)
  {
	  case 0:   
       if (now - last_time >= 2000) { // пройшло 1000 мс ?
          digitalWrite(13, HIGH); // вмикаєм
          last_time = now;
          task=1; // переходим к следующей задаче
        }
       break; 
    case 1:   
       if (now - last_time >= 200) { // пройшло 300 мс ?
          digitalWrite(13, LOW); // вимикаєм
          last_time = now;
          task=2; 
        }
       break; 
    case 2:     
       if (now - last_time >= 500) { // пройшло 1000 мс ?
          digitalWrite(13, HIGH); // вмикаєм
          last_time = now;
          task=3;
        }
       break; 
    case 3: 
        if (now - last_time >= 2000) { // пройшло 300 мс ?
           digitalWrite(13, LOW); // вимикаєм
           last_time = now;
           task=0;
        }
       break; 
    default :
       break; 	
  } 


}

Ця конструкція:

sxstalker пише:

  if (!on) {
...
  } else if (on) {
...
  } else if (!on) {
...
  } else if (on) {
...
  } 

не має сенсу. Якщо перша умова "if (!on)" виконалась, то решта вже не виконуються. А якщо не виконалась, то обовʼязково виконається друга - "if (on)", бо вона протилежна першій. А дві останні - ніколи не виконаються.

В попередньому прикладі у вас було всього два стани: світлодіод вимкнено, і світлодіод увімкнено. Тому було достатньо змінної типу bool, у якої всього два можливих значення.

В останньому прикладі у вас чотири стани:
* вимкнено перший раз (на 2000 мс)
* увімкнено перший раз (на 200 мс)
* вимкнено другий раз (на 500 мс)
* увімкнено другий раз (на 2000 мс)

Тому для збереження стану потрібна змінна такого типу, щоб могла приймати як мінімум 4 значення. Для простоти візьмем int. Стани будем нумерувати від 0 до 3. І для ясності перейменуєм змінну з "on" на "state".

int state = 0; // поточний стан. з початку - "0";

void loop() {
  unsigned long now = millis();

  if (state == 0) { // в стані "0"?
    if (now - last_time >= 2000) {
      digitalWrite(13, HIGH);
      last_time = now;
      state = 1; // переходим в стан "1"
    }
  } else if (state == 1) { // в стані "1"?
     if (now - last_time >= 200) {
      digitalWrite(13, LOW);
      last_time = now;
      state = 2; // переходим в стан "2"
    }
  } else if (state == 2) { // в стані "2"?
    if (now - last_time >= 500) {
      digitalWrite(13, HIGH);
      last_time = now;
      state = 3; // переходим в стан "3"
    }
  } else if (state == 3) { // в стані "3" ?
     if (now - last_time >= 2000) {
        digitalWrite(13, LOW);
       last_time = now;
       state = 0; // переходим в початковий стан "0"
    }
  }
}

Абсолютно те ж саме, тільки switch/case замість if/else. Так більш читабельно:

int state = 0; // поточний стан. з початку - "0";

void loop() {
  unsigned long now = millis();

  switch (state)
  {
    case 0:
      if (now - last_time >= 2000) {
        digitalWrite(13, HIGH);
        last_time = now;
        state = 1;
      }
      break;
    case 1:
      if (now - last_time >= 200) {
        digitalWrite(13, LOW);
        last_time = now;
        state = 2;
      }
      break;
    case 2:
      if (now - last_time >= 500) {
        digitalWrite(13, HIGH);
        last_time = now;
        state = 3;
      }
      break;
    case 3:
      if (now - last_time >= 2000) {
        digitalWrite(13, LOW);
        last_time = now;
        state = 0; // переходим в початковий стан "0"
      }
      break;
  }
}

Тепер зверніть увагу на те, що дії при переході між станами практично однакові. Відрізняються тільки
тривалості: 2000, 200, 500, 2000
значення, що передається у digitalWrite: HIGH, LOW, HIGH, LOW
номер наступного стану: 1, 2, 3, 0, 1, 2 ...

Таким чином можна використати один і той же код для циклічного переходу між станами. Тривалості та значення для digitalWrite помістимо у масиви, а значення з масивів вибиратимем по номеру стану:

unsigned int dur[4] = { 2000, 200, 500, 2000 };  // масив тривалостей (duration) для кожного стану
byte led[4] = { HIGH, LOW, HIGH, LOW };          // масив значень digitalWrite для кожного стану
int state = 0; // поточний стан. з початку - "0";

void loop() {
  unsigned long now = millis();

  if (now - last_time >= dur[state]) { // якщо час поточного стану вийшов
    digitalWrite(13, led[state]);      // вмикаєм або вимикаєм світлодіод
    last_time = now;
    state = state + 1; // переходим у наступний стан
    // але максимально можливий номер стану - "3"
    // якщо це був останній - переходим у початковий стан "0"
    if (state > 3) {
      state = 0;
    }
   }
}

Тепер зверніть увагу, що в масиві led всього два варіанти значень для digitalWrite: HIGH і LOW, і вони чергуються. Можна обійтись без цього масиву. Як його позбутись - вам в якості самостійного завдання