Arduino.ru

Зарядите Serial на 115200 и после строки 22 временно поставьте Serial.println(timer);

Это, конечно, вызовет тошноту у программистов микроконтроллеров, но поможет понять, почему не выполняется условие.

Не понял. 

Типы одинаковые.

Почему-то таймер не обнуляется.

Как заставить это работать? 

 

Ну так оно работает, ровно так, как написано. В прерывании вы должны только проверить длительность и количество щелчков, и ничего другого. В лупе меняйте состояние выхода.

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

Ничего не выходит.

Можете помочь с кодом?

Покажите, как пытаетесь исправить.
Я бы попробывал замерить длительность сигнала с датчика, и на основании этого уже определять хлопки. Делать это конечно лучше таймером, а это тема довольно сложная, если вы новичок, то попробуйте для начала в loop'е это всё сделать, а в прерывании менять состоянии переменной, для отсчета времени. Не знаю, что за датчик, может надо применить не миллис а микрос

могу, только словами скажите, что хотите сделать, не увидел в коде двойного щелчка совсем

а вот так?
 

#define pinLed  13
#define pinINT0 2
#define Scetchik 1

void ledInvert(void)
{
  static unsigned long j = Scetchik;
  if(!j--){ 
    digitalWrite(pinLed, ! digitalRead(pinLed));
   j=Scetchik;
    }
}

void setup() {
  pinMode(pinLed,OUTPUT);
  attachInterrupt(0, ledInvert, RISING );
 }

void loop() {

}

 

Проблема в том, что я не могу победить дребезг- не нашёл конденсатор, и теперь процесс поиска ошибок превращается в ад.
Хочу, чтобы при поступлении лог.1 за 4 секунды дважды, прерываясь, зажигался диод.

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

Какой дребезг? У вас там механические контакты? Если вы имеете ввиду шум, то, ещё раз повторяю, меряйте длительность шума, потом тишину, опять шум, и если попал подходящий вам интервал, включайте/выключайте свой выход. Если датчик постоянно вам посылает помехи на порт, то без дополнительных элементов не добиться стабильной работы.

Я менял, не помогает.

Внутри функции обработки прерывания не работает delay(), значения возвращаемые millis() не изменяются. Возможна потеря данный передаваемых по последовательному соединению (Serial data) в момент выполнения функциии обработки прерывания. Переменные, изменяемые в функции, должным быть объявлены как volatile.

Ну да, я там все без задержек написал.
С переменными вчера разобрался.
Возможно такое, что, если отключить сериал, она заработает нормально?

нет, логика не соответствует требуемой, начните с моего скетча, где отрабатывается два щелчка

Отключил сериал, стабильность работы стала лучше, но программа снова не работает.
Как найти этот скетч?

#define pinLed  13
#define pinINT0 2
#define Scetchik 1
volatile unsigned int j = Scetchik;
unsigned old_millis = 0;
#define timer 4000
int flag = 0;

void ledInvert(void)
{
    if(!j--){ 
    digitalWrite(pinLed, ! digitalRead(pinLed));
    j=Scetchik;
    }
}

void setup() {
  pinMode(pinLed,OUTPUT);
  attachInterrupt(0, ledInvert, RISING );
 }

void loop() {
if(j==0 && flag==0){old_millis=millis();flag=1;}
if(millis()-old_millis > timer && flag==1){j=Scetchik; flag = 0;} // начинай сначала )))

}

 

#define pinLed  13
#define pinINT0 2
#define Scetchik 1
#define tau_drebezg 200
#define timer 4000

volatile unsigned int j = Scetchik;
unsigned old_millis = 0;
unsigned tau_millis = 0;
byte flag = 0;

void ledInvert(void)
{
    if(millis()-tau_millis > tau_drebezg){
        if(!j--){ 
        digitalWrite(pinLed, ! digitalRead(pinLed));
        j=Scetchik;
                }
                tau_millis=millis();
                                         } // 
}
    
void setup() {
  pinMode(pinLed,OUTPUT);
  pinMode(3,OUTPUT);
  digitalWrite(3,HIGH);
  delay(20);
  attachInterrupt(0, ledInvert, RISING );
 }

void loop() {
if(j==0 && flag==0){old_millis=millis();flag=1;}  //ждём 4 секунды 2-й хлопок
if(millis()-old_millis > timer && flag==1){j=Scetchik; flag = 0;} // нет хлопка -начинай сначала )))

}

 

ННП НФВКЩИС.

И, да, как исправить ошибку я уже написал в #17, а ещё есть замечание по объёму кода.

Вот эта конструкция

if (flag == 0) {
	digitalWrite(led, HIGH);
	flag = 1;
} else {
	digitalWrite(led, LOW);
	flag = 0;
}

легко заменяется на существенно более короткую и прозрачную

digitalWrite (led, flag = !flag);

несколько спорное утверждение

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

Не "несколько", а на полноценный холивар в котором все будут правы :)

(смеясь) ну да, ну да =)

зы. ua6em, чуть позже отвечу. некогда щас

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

Антидребезг в прерывании - Может так

#define button 2
bool kn = 0;
volatile uint32_t time;

void setup()
{
  pinMode(button, INPUT_PULLUP);
  attachInterrupt(0, butDreb, CHANGE);
}

void loop()
{
  if(kn)
  {
    // bla-bla-bla
    kn = 0;
  }
}

void butDreb()
{
  if(!digitalRead(button))time = millis(); 
  if(digitalRead(button)){
    if(millis() - time > 50) kn = 1;
  }
}

 

А если дребезга нет, то как будет второй раз вызван butDreb() и выставлен kn = 1?

Вернусь к этому как-нибудь потом.  

Когда достану конденсатор.

Спасибо всем, кто пытался помочь, но мой говнокод на 2й день не читается.

Кто его писал?

Если был клик то будет и единица

Ну а обработку проведем в лооп и сбросим в 0

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

недавно здесь описыал рабочий алгоритм одного из вариантов антидребезга с прерыванием 

http://arduino.ru/forum/pesochnitsa-razdel-dlya-novichkov/obrabotka-knopki-na-preryvaniipower-down-mode-gde-oshibka#comment-432995

ua6em, вроде недавно искал и использовал для энкодера код от Димакса, (ежели память е изменяет), я его тоже тестил с год назад, но то ли у меня больно глючный энк. попался, то еще хз что, но пропуски и ошибки с тем кодом  были. В результате написал такой, и он меня вполне устроил. конденсаторы ко все пинам энка по 0,1 мкф, подтяжка может быть только внутренними резисторами ардуины на голом энке.

​void encoder()  //  Обработка прерывания энкодера
{
  char j=0;
  for(byte i=0; i<7; i++)                 // несколько раз читаем состояния пина dt
    {if(digitalRead(DT)) j++; else j--; } // для защиты от дребезга контактов
      if(j>0) enc--;  else enc++;         // и по результатам тестов принимаем решение

  switch(MENU)                            // ограничение вводимых параметров
     {  case 1:  if(enc <0) enc = 0; 

И что? код из поста #16 не работает?

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

Он работает, но тоже с дебезгом, если подуть - мигает.

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

Подуть это у ДПС, настраивайте на хлопок руками )))
Он работает так, как написали, а задумывали, так как написано у меня ...

это и так понятно, если после первого включения до срабатывания прерывания прошло более 4 секунд код никогда не выполнится

согласен, невнимательно посмотрел, сорри =)

но тоже обработка со своими нюансами.

В таком случае это не столько защита от дребезга, сколько определение клика. Потому что просто push оно ловить не сможет.

не может + там множественный вызов прерывания в случае дребезга

7/8 предыдущео поста стер, чтобы не влезать во все эти хреновы тонкости, бо и так понятно, что где-то такой вариант годится, а где-то нет.

почему тему про се ЕвгенийП  и обозвал -холиваром

Ну я так и написал в #27 что это клик, а если надо удержание и клик то надо добавить еще одну переменную. Например так

//Антидребезг в прерывании (НАЖАТИЕ КЛИК)

#define button 2
volatile bool kn = 0;
volatile bool fl_kn = 0;
volatile uint32_t time;

void setup()
{
  pinMode(button, INPUT_PULLUP);
  attachInterrupt(0, butDreb, CHANGE);
}

void butDreb()
{
  if(!digitalRead(button)){//Прерывание по нажатию
    time = millis(); 
    fl_kn = 1;
  }
  if(digitalRead(button)){ // Прерывание по отпусканию или дребезг
    if(millis() - time > 50) kn = fl_kn;//Был клик (Если прога длинная и в лооп не успели)
    fl_kn = 0;
  }
}

void loop()
{
  if(fl_kn)
  {
    if(millis() - time > 50) //Было нажатие
    {
      kn = 1;
      fl_kn = 0;
    }
  }
  if(kn)
  {
    // bla-bla-bla
    kn = 0;
  }
}

 

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