Как поменять данную конструкцию на millis()?

Question

[blackgrom]

for (int i = 0; i < 7; i++) {
    for (int n = 0; n < PIXELS_DIGITS; n++){ 
      writeSeg(n, 5);
  }
    strip.show();
    delay(100);
     for (int n = 0; n < PIXELS_DIGITS; n++){ 
      writeSeg(n, 0);
  }
    strip.show();
    delay(100);
     for (int n = 0; n < PIXELS_DIGITS; n++){ 
      writeSeg(n, 2);
  }
    strip.show();
    delay(100);
     for (int n = 0; n < PIXELS_DIGITS; n++){ 
      writeSeg(n, 0);
  }
    strip.show();
    delay(100);
     for (int n = 0; n < PIXELS_DIGITS; n++){ 
      writeSeg(n, 5);
  }
    strip.show();
    delay(100);
  }

из адресной ленты ws2812 построены большие 7 сегментные дисплеи, в этом коде пройсходит анимация то есть включаеться и выключается разные сегменты через 100 миллисекунд, как видно из кода сначала включаеться 5 сегмент потом 0 потом 2 потом снова 0 и потом 2 и снова 5(в функций writeSeg(int n, int seg ) где seg это номер сегмента). переключенте идет через 100миллисекунд с помощью функциий delay и теперь вопрос как заменить его на millis()?

Comments

[gd1xza]

берете и переписываете как еще то.

[blackgrom]

gd1xza, ну вот в этом то и вопрос как данную конструкцию переписать на millis

[blackgrom]

gd1xza, не используя delay

[gd1xza]

blackgrom, это тогда задание а не вопрос

Answer 1

[evgeniy_lm]

Не совсем понимаю зачем это нужно, но примерно так:

unsigned long t;
........................................
// вместо delay(100);
t = millis()+100;
while (t < millis());
.......................................

В принципе этот код абсолютно идентичен delay, только размер исполняемого файла будет больше, т.к. delay работает точно так же, но код более компактный.

Вообще весь ваш код пишется более компактно:

char x[5] = {5, 0, 2, 0, 5};
for (int i = 0; i < 7; i++)
{
for (int k = 0; k < 5; k++)
{
for (int n = 0; n < PIXELS_DIGITS; n++)
{
writeSeg(n, x[k]);
}
strip.show();
delay(100);
}
}

Comments

[blackgrom]

Блин вы лучший

[kalapanga]

evgeniy_lm, ну и зачем такое писать? blackgrom ведь так и будет думать, что от него требовалось именно это. Да ещё и с millis обращаетесь неправильно (переполнение).

[evgeniy_lm]

kalapanga, о чем вы вообще? Эта хрень в принципе не правильная. а вероятность переполнения 0,0000023% т.е. ей можно пренебречь. По хорошему вместо delay нужно использовать таймер

[kalapanga]

1) Для тех, кто "Не совсем понимаю зачем это нужно" объясняю. Это нужно, чтобы сделать блокирующий код неблокирующим. Вы этого не сделали.
2) "Вероятность переполнения" - глупость. Переполнение - это не случайное, а предопределённое событие, про которое абсолютно точно известно, когда оно произойдёт. Конечно, для учебного задания про абстрактную мигалку это не особо важно, но почему сразу не учиться использовать millis() правильно?
3) О таймерах. Вобще-то millis и так использует таймер. Использовать аппратный таймер в явном виде здесь нет ни малейшей необходимости.

[evgeniy_lm]

kalapanga,

Это нужно, чтобы сделать блокирующий код неблокирующим. Вы этого не сделали.

блокирующий код неблокирующим делается с помощью прерываний. delay и millis это совсем про другое.

Переполнение - это не случайное, а предопределённое событие,

Верно, не случайное - 1 раз за 50 суток непрерывной работы. Схема собранная на коленке с использованием delay и millis полтора месяца работать не будет.

почему сразу не учиться использовать millis() правильно?

Ну так научили бы.

Answer 2

[kalapanga]

Для начала повторяющиеся куски кода (зажигание одного сегмента ленты) сделать функцией, номер сегмента - параметр. Тогда в программе останется всего один delay. А его уже менять по прописи BlinkWithoutDelay.
Точнее, нюнсы реализации уже надо в целом с остальной программой смотреть.