Arduino.ru

В коде не разбирался, но 99.9% кривой, копайте в нем. Вставьте сериалы в контрольных точках, смотрите какие значения какая переменная принимает. Кварц конечно на МК не супер, но его отклонения могут на суточных интервалах сказываться, а не на секундных. Где то у вас интервал проскакивает свое значение, попробуйте на 5000 изменить и посмотрите.

Да нет,bwn, этот гений время начала своих танцев не фиксирует. Когда подряд, то успевает к фиксации в строках 60 и 77. А «чуть погодя» запомненное время становится очень старым. И оба диодо стабатывают одновременно.... Задержка то уже случилась. Гы!

wdrakula, я в первом if-е забуксовал и даже читать дальше не стал. Serial.println наше фсе.

Зря! Там песня, а не код! Получил большое удовольствие.

А как прекрасны операторы через запятую?

Спасибо что развеяли сомнения на счет кварца. 
Прямизной кода правда похвастаться не могу, в "теме" букавльно пару месяцев, и факультета IT за плечами нет. Гуманитарий я :)  Но хочу учиться) 

Пробовал поменять интервал на 5000 - задержка начинает работать, но 5с - это много, я собирался настраивать ее в пределах до 1с. 

Издеваетесь?

в 55 и 72 строки добавьте    previous-bla-bla-bla=millis()

----------------

остальные свои косяки сами ищите... или Женя поможет - он добрый. ;) Клапауция еще можно попросить.

а у меня возник вопрос - Вы вообще как это создавали? Просто есть серьезное подозрение, что Вы не очень понимаете, что в этом коде происходит?

То есть Вы что-то взяли за основу, не понимая, а потом стали приделывать костыли из говна и палок. Так?

1. К Лешаку, находим там и читаем про миллис, пока не наступит просветление. Правда сейчас что то у меня не сконнектилось, попозже попробуйте.
2. Что стать адептом тележурнала "Хочу все знать", берем сериалпринты и расставляем в контрольных точках, какие значения у наших переменных.
3. На основании полученной информации, курим что нибудь забористое и начинаем креативить на предмет, почему моя переменная делает не так.
 

Так вроде уже всё выкурено. Миллис вон уже нестабилен.

Помнится в 80ы-е юзера так задолбали апломбными заявлениями, что у нас что-то не так работает, что на дверь комнаты дежурной смены мы повесили табличку: "Да, у нас фортран не работает, и чо?"

У нас компьютер отчет стер.
Что делали?
Ничего, он сам.
Вот такие были у нас брутальные компьютеры.)))) ИИ отдыхает.
И это уже 486 и первые пеньки. Что в 80-х было вообще не представляю, я тогда еще деревянный в виде линейки юзал.

в 84-ом я распечатал себе свой первый экземпляр Кернигана и Ричи на АЦПУ, на СМ-4 (вроде Мера польская была).

Вы все врети. Стока люди не живут. :)

Эт план такой, пенсионный фонд обанкротить.))))

Слёзы счастья!)))

Спасибо wdrakula,  про 55 и 72 строки - это было круто!)  Теперь все работает идеально

Даже видео снял на радостях)

https://vimeo.com/202012131

Простите, только сейчас понял, на сколько провокационен заголовок!) 
Да и китайцы не виноваты, отличные платы делают!))) 

Спасибо за ресурс Лешака, буду вчитываться. Что касается остального кода, то пока одолевать людей не буду, раз он работает, возможно скоро сам буду так же смеяться над своими ошибками ;) 

А я получается, писал эту фразу, писал,  "previous-bla-bla-bla=millis()" , но не в действии, а где-то между ними. Поэтому, она не выполнялась или выполнялась не правильно...  

это потому, что какой-то "гей" научил Вас писать операторы через запятую.

Поэтому чудо, что у Вас хоть что-то работает.

оператор if используется так:

if (a>b && c<d || x==y)
   {
   operator1;
   operator2;
   operator3;
   operator4;
   }
else
  {
   operator7;
   operator8;
  }

if (a>b && c<d || x==y)
   {
   operator1;
   operator2;
   operator3;
   operator4;
   }
else
  {
   operator7;
   operator8;
  }

Да, спасибо! Теперь только так! 

Наверно уже надоел, но все же спрошу)

Есть еще одна часть кода того же моего "масштабного" проекта. 

#define ledPinRok 13 // Пин спуска

#define tataPin 10  // Спуск, соединяется с RC приемником
#define switPin 11  // Переключение функции спуска, соед с RC приемником

int ledStateRok = LOW;             // Переменная спуска
long interval_Rok = 2000;           // интервал спуска
long previousMillisRok = 0;        // храним время последнего спуска
int RS = 0; //Переменая для работы спуска
static int OneRok = 0; //Переменная для совершения цикла спуска один раз


int tata = 0;  // Переменная кнопки
int swit = 0;  // переменная переключателя функции спуска

void setup() {

  pinMode (tataPin, INPUT); //Входной пин спуск
  pinMode (switPin, INPUT); //Входной пин переключение

  pinMode (ledPinRok, OUTPUT);
  
}


void loop()
  {

    /////////////////// Блок обсчета входящего ШИМ!!!!!!!!!!!!!!!!


    {
      tata = pulseIn (tataPin, HIGH, 37880); //Обсчет входящего ШИМ
      swit = pulseIn(switPin, HIGH, 37880); //Обсчет входящего ШИМ
    }

    ////////////////////// Блок вывода значенй ШИМ на каналах на сериал монитор\\\\\\\\\\\\\\\\\\\

    Serial.print ("Channel #3: ");
    Serial.println(tata);
    Serial.print ("Channel #4: ");
    Serial.println(swit);




  //!!!!!!!!!!!!Блок переменной zRok !!!!!!!!!!!!!!!!!!

  static int zRok = 0; //Переменая для работы спуска

  if ((tata > 1200) && (swit > 1500))

  {
    zRok = 1;
  }

  if (tata < 1200)  // Если кнопка выключена
  {
    zRok  = 0; // спуск всегда выключено
  }

  //!!!!!!!!!!!!Блок переменной OneRok  !!!!!!!!!!!!!!!!!!

  if (OneRok == 1)  // Есть цикл один раз выполнялся,
  {
    ledStateRok = LOW; //то второй раз не заработает  до того как не выключится кнопка
  }

  //!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!Блок управления спуском!!!!!!!!!!!!!!!!!!!!!!!////////

  digitalWrite(ledPinRok, ledStateRok); //Присваиваем пину спуска значение перменной

  unsigned long currentMillisRok = millis(); //Объявляем переменную времени

  if ((OneRok == 0) && (zRok  == 1)) //Если кнопка нажата и цикл не выполнялся

  {
    ledStateRok = HIGH; // спуск срабатывает
    currentMillisRok  = millis();   // !!!запоминаем "время начала танцев"!!!
    RS = 1;
  }

  if ( (RS == 1) && (currentMillisRok - previousMillisRok > interval_Rok)) //Если спуск включен,
    // ИИ прошло время больше интервала
  {
    ledStateRok = LOW; //Спуск выключается
    previousMillisRok  = currentMillisRok ;  //Запоминаем время срабатывания
    OneRok = 1; //   переменная однократного исполнения цикла получает=1
    RS = 0;   // обнуляем переменную спуска


  }
  //!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!Блок обнуленя переменной "OneRok"!!!!!!!!!!!!!!!!!!!!!!!////////
  if ((OneRok) == 1 && (zRok == 0)) //Если цикл один раз выполнялся, и уровень оборотов пересек нижнюю границу
  {
    OneRok = 0; // то переменная одноразового совершения цикла обнулется (цикл можно выполнить снова)
  }

   }  //Крайняя фигурная скобка

Задачей было сделать так, чтобы светодиод включился один раз на определенное время (interval_Rok) и погас,  до выключения и новго включения тумблера. Примерный код (кривой) у меня уже был. Сделал по тому же принципу, через millis (), постарался учесть замечания. 

И всё вроде бы работает, вот только при превичном включении тумблера на пульте RC, диод не всегда загорается (в большинстве случаев не загорается). 

В чем может быть дело? Кажется я не учел какой-то принципиальный момент.  
По сериал-монитору значения на входе изменяются. А вот где его воткнуть еще раз, чтобы посмотреть переменные - не соображу.

Да, еще такой симптом - время включения светодиода - не всегда одинковое, не смотря на то, что интервал ничего не именяет. Пробовал объявлять интервал const, пробовал unsigned  - та же петрушка

79 строка. Не  каррент, а привиус.

представьте  вместо контроллера самого себя, только ускоренного,как Флеш ;). с секундомером. Каррент - это то, что показывает секундомер сейчас, а привиус - то, что вы записали на бумажке. Так становится нагляднее?

Вы их таки не пробуйте, они не черешня, а разберитесь и усвойте, какое магическое слово что значит.))))

Не, у ТС что то все петрушка получается. Было б сало, еще куда ни шло.)))

Гораздо))) А я всё представляю время как что-то связанное с бесконечной вселенной))) 
 

Но если в 79 строку ввернуть previousMillisRok вместо currentMillisRok, получается странная картина: 

Диод загорается и горит. А когда кнопку выключаешь - через интервал гаснет. Но иногда и правильно стработает пару раз, а потом опять за своё. 

Сижу, разбираюсь, но пока без сала:) 

Вы, признак того, что цикл начался (OneRok) j обнуляете при окончании цикла, а нужно при НАЧАЛЕ.

Перенесите OneRok=0;  в первый if, рядом с 79 строкой.

Понятно? Тогда это условие сработает ровно один раз, при нажатии кнопки.

wdrakula, добрый вы. Вот сериалпринты ТС явно пользовать не желает, предпочитает спросить, а это не гут.

Никакие отладчики, сериалпринты и прочие эти ваши джитэги никогда не заменят живого человеческого общения! Долой виртуальную землю, резиновых женщин и безалкогольное пиво!

Не, я лимит вопросов предпочитаю приберегать на случай, когда действительно в тупик залезу (мой личный).

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

Время начала танцев обозначаю..., все по схеме вроде делаю...  Странно, не пойму почему так. 

Из кода убрал лишние переменные,  теперь он сталь короче 

#define ledPinRok 13 // Пин спуска

#define tataPin 10  // Спуск, соединяется с RC приемником
#define switPin 11  // Переключение функции спуска, соед с RC приемником





 int tata = 0;  // Переменная кнопки
int swit = 0;  // переменная переключателя функции спуска

void setup() {

  pinMode (tataPin, INPUT); //Входной пин спуск
  pinMode (switPin, INPUT); //Входной пин переключение

  pinMode (ledPinRok, OUTPUT);

   Serial.begin(9600);

  
}


void loop()
  {

    /////////////////// Блок обсчета входящего ШИМ!!!!!!!!!!!!!!!!


    {
      tata = pulseIn (tataPin, HIGH, 37880); //Обсчет входящего ШИМ
      swit = pulseIn(switPin, HIGH, 37880); //Обсчет входящего ШИМ
    }

    ////////////////////// Блок вывода значенй ШИМ на каналах на сериал монитор\\\\\\\\\\\\\\\\\\\


   // Serial.print ("Channel #3: ");
   // Serial.println(tata);
  //  Serial.print ("Channel #4: ");
  //  Serial.println(swit);
  //  Serial.println("------------");

 


  //!!!!!!!!!!!!Блок переменной zRok !!!!!!!!!!!!!!!!!!
static int zRok = 0; //Переменая для работы спуска
 static int ledStateRok = LOW;             // Переменная спуска
unsigned long interval_Rok = 2000;           // интервал спуска
  unsigned long previousMillisRok = 0;        // храним время последнего спуска

  unsigned long currentMillisRok = millis(); //Объявляем переменную времени



  //!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!Блок управления спуском!!!!!!!!!!!!!!!!!!!!!!!////////


 Serial.print ("Svet ");
  Serial.println(ledStateRok);
   
  Serial.print ("Peremenneya: ");
  Serial.println(zRok);

 
 if ((tata > 1200) && (swit > 1500)&& (zRok != 2)) 
 {
  zRok = 1;
 }

if (( zRok  == 2) && (tata < 1200)) //Если цикл один раз выполнялся, и кнопка выключена
  {
  zRok = 0; // то переменная одноразового совершения цикла обнулется (цикл можно выполнить снова)
  }

  
  if  (zRok  == 1)//Переменная =1

  {
    ledStateRok = HIGH; // спуск срабатывает
     previousMillisRok  = millis();   // !!!запоминаем "время начала танцев"!!! ??? - поменял на превиус...
     
  } 
else
{
  ledStateRok = LOW;
}

  if ((ledStateRok == HIGH)&& (currentMillisRok - previousMillisRok > interval_Rok)) //Если спуск включен,
    // ИИ прошло время больше интервала
  {
    ledStateRok = LOW;
     previousMillisRok  = currentMillisRok ;  //Запоминаем время срабатывания
     zRok = 2;
    
  }

 
digitalWrite(ledPinRok, ledStateRok); //Присваиваем пину спуска значение перменной





   }  //Крайняя фигурная скобка

Вы, все таки доведете меня до греха!

В первой версии кода, не надо ничего делать КРОМЕ ТОГО, что в 79 строке СТАРОГО, мля, кода, написать :

previousMillisRok  = millis(); OneRok=1;

Всё. На этом мы прекратим изучение Ваших бессмерных творений. Оставьте их потомкам. Я - устал.

Хорошо, Вы уже и так очень помогли!)