Arduino.ru

Кажется я нашел решение. Насколько оно корректно и какие могут быть подводные камни?

 

byte a ;
byte b ;
byte c ;
byte d ;
byte e ;
byte f ;
byte g ;
byte h ;
int massive1[] = {a, b, c, d};
int massive2[] = {e, f, g, h};
const int analogInPin1 = A2;   // клавиатура 1
const int analogInPin2 = A3;   // клавиатура 2
int analog1 = 0;
int analog2 = 0;
int cifra = 25 ;
#include <LiquidCrystal.h> //Подключаем библиотеку для работы с LCD
LiquidCrystal lcd(12, 11, 10, 5, 4, 3, 2); // инициализируем LCD, указывая управляющие контакты
int i = 0;
void setup() {
  // Serial.begin(9600); //монитор порта тестовая строка
  pinMode(analogInPin1, INPUT_PULLUP);
  pinMode(analogInPin2, INPUT_PULLUP);
  lcd.begin(20, 4);// задаем размерность дисплея
}
void loop() {
  analog1 = analogRead(analogInPin1);
  if (analog1 > 100 && analog1 < 300) {
    cifra = 1 ;
   }
  if (analog1 > 350 && analog1 < 440) {
    cifra = 2 ;
  }
  if (analog1 > 450 && analog1 < 510) {
    cifra = 3 ;
  }
  if (analog1 > 520 && analog1 < 580) {
    cifra = 4 ;
  }
  if (analog1 > 585 && analog1 < 645) {
    cifra = 5 ;
  }
 // lcd.setCursor(9, 0); //выставим курсор и введем  фразу тестовая строка
 // lcd.print(cifra);   //тестовая строка
 //  Serial.println(analog1);  //тестовая строка
  if ( analog1 < 999 && cifra != 25) {
    delay(400);
        
    if (i < 4 ) {
      massive1[i] = cifra;
      lcd.setCursor(i,0); //выставим курсор и введем  фразу
  lcd.print(massive1[i]);
  i = i + 1;

    } }
  } //не трогай!!!

 

Все неправильно.  Во первых посмотрите конструкцию if (...) {}; else if (...) {}; . Это облегчит код. Во вторых предусмотрите код отсутствия нажатия кнопки, который нужно обабатывать как кнопку, иначе вы не сможете получать событие "отпускание кнопки". В третьих - по истечении антидребезговой задержки вы должны вновь опросить порт и убедится, что там сохраняется нажатой та кнопка которая запустила задержку, в противном случае нужно считать, что ничего не нажато. Не сделайте ошибку: фиксировать нужно не показания порта, а именно номер кнопки, потому что показания ADC могут плавать. 

Lictor,

1. Что означает глубокомысленная конструкция в строка 1-9?

2. Для чего объявлен массив в строке 10, если он не используется?

3. Зачем переменные в строках 13-15 и 18 объявлены глобально, если они используются только в одной функции?

4. Строки 27-41 - почему, если допустим, сработает первый if, всё ревно выполняются все остальные, хотя они заведомо ложны?

 

1 Строки 1 -9 таки правда лишние.

2 массив будет использоваться в дальнейшем

3Верно, моя ошибка

4,5 Не понял, а почему остальные if выполнятся? Интервалы поправил.

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

7 это по сути кодовый замок, скорости набора не требуется. Поэтому 400

8 Нет там стоит if который при i == 4 будет выполнять прочие действия, я его здесь не показал, что бы не загромождать пример

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

Конструкцию if else расмотрю, благодарю

Волшебно :) Напрягитесь и прикинте что будет, если в вашем коде от замка будет две одинаковых цифры подряд ! А другие отличные от этих двух.

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

 

использование керамических конденсаторов

Задержек в 400ms?

Вот ознакомьтесть, там нет чистых задержек, а есть периодические опросы - http://arduino.ru/forum/obshchii/drebezg-mozhet-komu-nibud-prigoditsya

Мда... Что такое "чистые задержки " !? Переодический опрос без задержек , это как ? Не важно как отсчитываются промежутки между чтениями - ламерским delay или засечкой времени таймером, я например в случае резистивной клавиатуры использую для отсчета прерывания по результату ADC конвертации. 

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

Можно реализовать смешной алгоритм частотной фильтрации. Можнопридумать какую нибудь производную чухню, типа измерения периода дребезга. Но зачем ? Если есть нормальное и вменяемое простое решение.

А использование конденсатора в резистивной клавиатуре искажает результат, да и не алгоритм это вовсе.

Вы решили поспорить о терминологии? Это не со мной. Мне неинтересно

Данный топик - прямое опровержение Вашего утверждения насчёт того, что это единственный алгоритм. Автор читае всего один раз, второй раз не читает и ни с чем не сравнивает. Да и статья на которую я ссылался содержит несколько иной (уже третий) подход, Вы видимо не прочитали.