Счетчик времени за 10 импульсов с прерываниями

Молодец Вася, ты правда СУПЕР.
Проблема в том, что ошибок в этом коде значительно больше.
Что касается деления, проще записать так:
float qwerty = (oldTime - newTime)/1000.0;

 

Оно не записывает в переменные, дополнил в код

Код (C++):

//#define irDiod 2 // приемник на 2 пине
//int nom = 10; // количество импульсов
int val = 1; //переменная для хранения импульсов
float oldTime = 0;
float newTime = 0;
float qwerty= 0;

void setup() {
  //pinMode(irDiod, INPUT);
  Serial.begin(9600);
}

void loop() {
  if (digitalRead(2) == HIGH && val == 1)
  {
    float oldTime = millis();
    val++;
  }
  if (digitalRead(2) == HIGH && val < 10)
  {
    val++;
  }
  else if (val == 10)
  {
    float newTime = millis();
  }
  qwerty = float(newTime - oldTime)/1000;
  Serial.print("qwerty:   ");
  Serial.println(qwerty);
  Serial.print("val:    ");
  Serial.println(val);
  Serial.print("oldTime:   ");
  Serial.println(oldTime);
  Serial.print("newTime:   ");
  Serial.println(newTime);
  delay(1000);
}
 

 

маленький совет - кмк, правильнее будет весь вывод включить в else if (val == 10), чтобы данные выводились только после десятого отсчета

 

Да. кстати, на тему "не записывает в переменные" - зачем в циклах ЗАНОВО переопределяете переменные? - потому и не записывает

 

Да вы были правы
Но теперь другая проблема.После подачи одного импульса, он сам докручивает до 10.

Код (C++):

//#define irDiod 2 // приемник на 2 пине
//int nom = 10; // количество импульсов
int val = 0; //переменная для хранения импульсов
float oldTime = 0;
float newTime = 0;
float qwerty = 0;

void setup() {
  //pinMode(irDiod, INPUT);
  Serial.begin(9600);
}

void loop() {
  if (digitalRead(2) == HIGH && val == 1)
  {
    float oldTime = millis();
    val=val+1;
    Serial.print("oldTime:   ");
    Serial.println(oldTime);
  }
  if (digitalRead(2) == HIGH && val < 10)
  {
    val=val+1;
  }
  else if (val == 10)
  {
    float newTime = millis();
    qwerty = (newTime - oldTime) / 1000;
   
    Serial.print("qwerty:   ");
    Serial.println(qwerty);
   
    Serial.print("newTime:   ");
    Serial.println(newTime);
    delay(1000);
  }
  //Serial.print("val:    ");
  //Serial.println(val);
}

 

Вот с сериала данные

 

Я параллельно кнопке поставил конденсаторы. Это не могу быть дребезги.

 

таки я не гуру, но сдается мне, что вы считаете циклы во время нажатия). надо добавить всетаки LOW наверное. HIGH + LOW это и будет один импульс причем начало первого хая) бертся за отсчет времени. хай или лоу 10 как конец отсчета. да и еще, определенно стоит добавить в скетч инициализацию вашей кнопки или что у вас там и прописать в сетапе что она делает INPUT или OUTPUT). скорее всего первое наверное). я так думаю.
(тихо молчу про внешние прерывания)

 

До внешних прерываний скоро наверное доберемся)

 

tschernenko, слушайте СуперВасю, он дело говорит. Надо ловить не только нажатия, но и отпускания кнопки, иначе Вы не отличите 10 коротких нажатий от одного длинного

 

Он самый. Можно в обработчик прерывания добавить программное подавление дребезга при помощи функции 'millis()' (антидеребезг вносится именно в обработчик прерывания, а не в функцию 'loop()') или нужно бороться с дребезгом аппаратно.

Если фронт нарастает плавно, то скорее будет.

 

Код (C++):

int val = 0; //переменная для хранения импульсов
float oldTime = 0;
float newTime = 0;
float qwerty = 0;

void setup() {
    pinMode(2, INPUT);
    attachInterrupt(0, count, RISING);
    Serial.begin(9600);
}

void loop() {
    if (val == 1) {
        oldTime = millis();
    }
    if (val == 10) {
        float newTime = millis();
        qwerty = float(newTime - oldTime) / 1000;
        Serial.print("qwerty:   ");
        Serial.println(qwerty);
        val = 0;
    }
}

// задерджка для подавления дребезга; зависит от частоты импульсов;
#define BOUNCE_DELAY 200
unsigned long last = 0;

void count() {
    unsigned long current = millis();
    if ((current - last) > BOUNCE_DELAY) {
        last = current;
        val++;
    }
}