ARDUINO Помогите с кодом

Старик Похабыч · 27 Окт 2020

А зачем надо было туда лезть , если все работало ?

ASM · 27 Окт 2020

Старик Похабыч написал(а):
А зачем надо было туда лезть , если все работало ?

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

DAK · 28 Окт 2020

ASM написал(а):
этот же код работает)

C++:

if (GetSecond()>=16 && GetSecond()<=20 || (GetSecond()>=41 &&GetSecond()<=45)) {TempStreetToArray();} else if (GetSecond()>=8 && GetSecond()<=15 || (GetSecond()>=34 &&GetSecond()<=40)) {TempToArray();} else if (GetSecond()>=21&& GetSecond()<=25 || (GetSecond()>=46 && GetSecond()<=50)) {PressToArray();} else {TimeToArray();}

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

Просто для себя хотел узнать. Получается что за это условие Вы вызываете функцию GetSecond() 12 раз, у меня даже возникла мысль, что в процессе выполнения условия результат данной функции может поменяться. Просто хотел у ГУРУ уточнить, что в этом случае лучше делать, один раз опросить функцию, передав её значение в переменную перед условием и потом прогнать условие, или так тоже вполне адекватно?

Старик Похабыч · 28 Окт 2020

Может поменяться. Но вероятность в данном случае мала. Тем не менее, лучше сначала получить в переменную и с ней работать, так будет правильнее и , скорее всего, даже немного быстрее.
Привыкать так писать не хорошо, т.к. можно напороться. И вообще когда что то делаешь надо четко понимать что будет в результате, так во всем.

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

ASM · 28 Окт 2020

DAK написал(а):
у меня даже возникла мысль, что в процессе выполнения условия результат данной функции может поменяться

бывало такое, на короткое время) ну этот код не я придумал, он был уже, только доделал под свои нужды) больше датчиков, больше запросов)

DAK · 28 Окт 2020

ASM написал(а):
бывало такое, на короткое время) ну этот код не я придумал, он был уже, только доделал под свои нужды) больше датчиков, больше запросов)

Да я всё понимаю, просто для себя узнал. Глаз резало, а главное никто не поправлял, я уж засомневался в своей адекватности.

ASM · 28 Окт 2020

DAK написал(а):
Да я всё понимаю, просто для себя узнал. Глаз резало, а главное никто не поправлял, я уж засомневался в своей адекватности.

Хорошо, когда знаешь и понимаешь теорию)
P.S.
Я вот хочу на esp с wifi перевести без ардуино, для управления с телефона) получится осуществить, без переписывания всего кода?) Лампу Gyver делал, в настройках с wifi разберусь)
Если возможно, надо заказывать платку, зимой будет чем заняться)

DAK · 28 Окт 2020

В этом случае, как я понимаю самое главное, это написать приложение под Андроид. По мне, следующие, что я буду делать, так это играть с структурами и еепром... Сперва надо до конца понять все эти типы данных, всякие классы, структуры, бинарные операции, а потом уже думать о вафле и прочем....

ASM · 28 Окт 2020

@Старик Похабыч, подскажи, как проще организовать тумблер для включения смены цветов и спецэффектов.
в коде есть две функции void fadeall() и void cylon (). Чтобы они работали, в коде часов, датчиков прописаны cylon(); ledColor = ColorTable[random(16)];
и закрыт ledColor = ColorTable[2]; (любое значение, свое или из таблицы) что выключить, нужно его открыть и закрыть спецэффекты.
Хотелось бы два тумблера
Cylon = true/false
LedColor = 0/1
1 будет random
0 будет фиксированным

через if (Cylon = true), то выполнять код, а если нет, то пропускает?)
аналогично if ( LedColor = 1), то выполняем, если нет, то берем готовый?)

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

ASM · 28 Окт 2020

@DAK, так-то есть готовые программы для управления по WiFi через AT команды

ASM · 28 Окт 2020

bool CYLON = false;
bool FADEAL = false;

void fadeall() {
if (FADEAL=true){... } }

void cylon () {
if (CYLON=true){...
fadeal;
...}}

даже если false, функция работает) я же указал, что работать при true)

Старик Похабыч · 28 Окт 2020

если у отображения спецэффектов есть 3 состояния, то bool я бы не брал. взять byte, тогда 0 - все выключено, 1- 1-ый спец, 2 - 2-ой, и всегда можно легко добавить 3-ий
Ну а в коде switch (nomer) и нужное делать

void cylon () {
if (!CYLON) return;
...
}

ASM · 28 Окт 2020

Нету трёх состояний) в коде Cylon, который ссылается для переливашек на fadeal, и LedColor, который можно сделать статичным и рандомным, надеюсь понятно)

Старик Похабыч · 28 Окт 2020

Вот тут у каждой переменной 2 состояния, соотв есть не 3 а 4 возможных состояния всего.
bool CYLON = false;
bool FADEAL = false;

Я пока не понимаю.. завтра на свежую голову прочитаю

ASM · 29 Окт 2020

Еще баг словил, установка времени теперь работает только один раз за цикл (11сек)
Батарейку заменил и всё по нулям) через settime установил)
Подробнее вчерашний разговор...

cylon:

void cylon () {
  static uint8_t hue = 0;
  Serial.print("x");
  // First slide the led in one direction
  for(int i = 0; i < NUM_LEDS; i++) {
    // Set the i'th led to red
    leds[i] = CHSV(hue++, 255, 255);
    // Show the leds
    FastLED.show();
    // now that we've shown the leds, reset the i'th led to black
    // leds[i] = CRGB::Black;
    fadeall();
    // Wait a little bit before we loop around and do it again
    delay(10);
  }
  Serial.print("x");

  // Now go in the other direction. 
  for(int i = (NUM_LEDS)-1; i >= 0; i--) {
    // Set the i'th led to red
    leds[i] = CHSV(hue++, 255, 255);
    // Show the leds
    FastLED.show();
    // now that we've shown the leds, reset the i'th led to black
    // leds[i] = CRGB::Black;
    fadeall();
    // Wait a little bit before we loop around and do it again
    delay(10);
  }
}

C++:

void fadeall() { for(int i = 0; i < NUM_LEDS; i++) { leds[i].nscale8(250); } }

кусочек из кода, для восприятия

C++:

      for(int k=0; k<=27;k++){
        if (digits[digit][k] == 1){leds[cursor]=ledColor;}
         else if (digits[digit][k]==0){leds[cursor]=0x000000;};
         cursor ++;
        };
      if (digit != last_digit)
      {
        //cylon();
        //ledColor =  ColorTable[random(16)];
          ledColor =  ColorTable[2];

меняя комментарии включаю/выключаю "фишки")

kostyamat · 29 Окт 2020

ASM написал(а):
bool CYLON = false;
bool FADEAL = false;

void fadeall() {
if (FADEAL=true){... } }

void cylon () {
if (CYLON=true){...
fadeal;
...}}

даже если false, функция работает) я же указал, что работать при true)

А если вот так?
void fadeall() {
if (FADEAL==true){... } }

void cylon () {
if (CYLON==true){...
fadeal;
...}}

Старик Похабыч · 29 Окт 2020

kostyamat написал(а):
А если вот так?
void fadeall() {
if (FADEAL==true){... } }

void cylon () {
if (CYLON==true){...
fadeal;
...}}

А так будет работать.. типичная ошибка... А я уже не замтил... До 5 утра починял ребенку ноут, что бы мог смотреть свои дистанционные уроки..
А лучше всего так:

void fadeall() {
if (FADEAL){... } }

void cylon () {
if (CYLON){...
fadeal;
...}}

ASM · 29 Окт 2020

@Старик Похабыч, я так сначала хотел сделать, но добавил, чтобы явно видно было)
работает в таком виде) сейчас тогда с LedColor помучаю, на byte (0 или 1) и буду спецэффекты допиливать, не так работают, как бы мне хотелось)

Старик Похабыч · 29 Окт 2020

в примерах лента перерисовывается после изменения каждого свтодиода. может так и надо. но от delay надо избавиться

ASM · 29 Окт 2020

@Старик Похабыч, пока так не работает)
byte LEDCOLOR = 1;

C++:

void color(){
   if (LEDCOLOR==0){
      long ledColor = CRGB::Aqua;}
      else if (LEDCOLOR==1){
      long ledColor = ColorTable[random(16)];}
    }

в loop color();
в коде ledColor; и ничего не кажет)

Старик Похабыч · 29 Окт 2020

а точно ledColor надо лонг ?

C++:

CRGB randomcolor;
randomcolor  = CHSV(random(255), random(255), 255);

kostyamat · 29 Окт 2020

Старик Похабыч написал(а):
а точно ledColor надо лонг ?

Точно нет. Потому как правильно
CRGB, а не long. Хотя фастлед и такое может сконвертировать, вот только зачем? CRGB это юнион и занимает три байта, а лонг - 4-ре.

Старик Похабыч · 29 Окт 2020

Вообще с расширением ASM надо марку держать

ASM · 29 Окт 2020

Старик Похабыч написал(а):
а точно ledColor надо лонг ?

из кода

C++:

long ledColor = CRGB::Aqua;

long ColorTable[16] = {
  CRGB::Amethyst,
  CRGB::Aqua,
  CRGB::Blue,
  CRGB::Chartreuse,
  CRGB::DarkGreen,
  CRGB::DarkMagenta,
  CRGB::DarkOrange,
  CRGB::DeepPink,
  CRGB::Fuchsia,
  CRGB::Gold,
  CRGB::GreenYellow,
  CRGB::LightCoral,
  CRGB::Tomato,
  CRGB::Salmon,
  CRGB::Red,
  CRGB::Orchid
};

kostyamat · 29 Окт 2020

@ASM, и шо?
Замените long на CRGB, скомпилируйте, и оцените разницу. Потом напишите static const CRGB ColorTable[16] = FLPROGMEM {} (надо перед include FastLED задать
#define FASTLED_USE_PROGMEM 1) и оцените еще раз.

ARDUINO Помогите с кодом

★★★★★★★

★★★★★✩✩

★★★✩✩✩✩

★★★★★★★

★★★★★✩✩

★★★✩✩✩✩

★★★★★✩✩

★★★✩✩✩✩

★★★★★✩✩

★★★★★✩✩

★★★★★✩✩

★★★★★★★

★★★★★✩✩

★★★★★★★

★★★★★✩✩

★★★★★★✩

★★★★★★★

★★★★★✩✩

★★★★★★★

★★★★★✩✩

★★★★★★★

★★★★★★✩

★★★★★★★

★★★★★✩✩

★★★★★★✩