Arduino.ru

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

Ну это не код, а фрагмент кода ...

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

Ну, это франмент кода который приводит к ошибке. Вернее, работает не как хочется.

Пилю универсвльный пульт управляние на nrf24. Есть экран, на котором отображается номер выбранной модели и "расход", (не знаю, как правильно назвать, вроде как consumption, т.е. максимальная скорость). Длительное нажатие кнопок меню "влево" и "вправо" по идее должно увеличивать и уменьшать "расход". При этом меняется отображаемая информация на экране.

Короткое нажатие на кнопки работает как триггер для каждого канала, отображение на экране не требуется.

наверное потому что для остальных перерисовка экрана не прописана.

Попробую "велосипед"

Так меняю порядок команд, ставлю первыми без прописовки экрана. Работают отлично, а те, что с экраном - нет.

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

А чем они различаются? 2 кнопки с отрабатыванием короткого и длинного нажатия, две - с отрабатыванием просто короткого...
Проблема в том, что отрабатываются только ЛЮБЫЕ 2 кнопки в коде...

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

1) Если кнопка 1 будет нажата и отпущена в течении 1 секунды, то я включаю лампочку

2) Если кнопка 2 будет нажата и отпущена в течении 1 секунды, то я включаю бомбу

3) Если кнопка 1 будет нажата и более 1 секунды, то я выключаю лампочку

....

и. т.д.

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

Блин... Ну разве можно так читать?

#include <Button.h>
Button test;

#define Button_Apin    7     // пин кнопки А
#define Button_Bpin    8     // пин кнопки B
#define Button_Cpin    9     // пин кнопки C
#define Button_Dpin    10    // пин кнопки D

#define Button_A       0     // программный номер кнопки А
#define Button_B       1     // программный номер кнопки B
#define Button_C       2     // программный номер кнопки C
#define Button_D       3     // программный номер кнопки D

void setup() {
Serial.begin(9600);
test.NO();                         // кнопки нормально разомкнутые 
test.pullUp();                     // включена внутрення подтяжка к питанию (кнопки при нажатии замыкают на GND)
test.duration_bounce       (  50); // время антидребезга
test.duration_click_Db     ( 250); // время дабл клика 
test.duration_inactivity_Up(5000); // время неактивности отпущенной кнопки
test.duration_inactivity_Dn(1000); // время неактивности нажатой кнопки
test.duration_press        ( 500); // время длительного нажатия

// ниже через запятую пишутся пины кнопок, соответственно в скетче программные номера у этих кнопок будут 0, 1 , 2 и т.д. и обращение к ним далее уже будет по программым номерам, поэтому выше рядом с дефайнами пинов сделаны дефайны программных номеров: 
test.button(Button_Apin, Button_Bpin, Button_Cpin, Button_Dpin);
}
void loop (){ 
test.read(); 

if (test.event_press_short  (Button_A)) {Serial.println("shortA");}
if (test.event_press_long   (Button_A)) {Serial.println("longA");}

if (test.event_press_short  (Button_B)) {Serial.println("shortB");}
if (test.event_press_long   (Button_B)) {Serial.println("longB");}

if (test.event_press_short  (Button_C)) {Serial.println("shortC");}
if (test.event_press_long   (Button_C)) {Serial.println("longC");}

if (test.event_press_short  (Button_D)) {Serial.println("shortD");}
if (test.event_press_long   (Button_D)) {Serial.println("longD");}


}

Спасибо за разные варианты.

Попробовал, но ни в одном случае не смог добиться 100% повторяемости результата. Иной раз приходится повторно нажимать на кнопку, чтоб свершилось "чудо". Это так и надо или у меня слишком завышенные требования к результату?

Если же судить по исходному сообщению, то там у Вас куча delay(200) - с ним ничего устойчиво работать и не должно.

Не куча, а 2... Это при обработке состояния "долго нажато" изменялись значения счетчика и выводилось значение на экран. Задержка нужна, чтоб  цифры не так быстро бежали. 

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

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

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

Зато я узнал еще несколько методов обработки событий с кнопками:)

Но никто из мЭтров не сказал, "слыш, нуб, тут у тебя все нормально, ищи в другом месте"

А вообще, интересно - с чего это Вы решили, что "тут у тебя все нормально"? По выложенному фрагменту этого не скажешь.

Я не спорю, программы пишу... Ну не долго. Очень не долго. Но я учусь.

Код, может быть и не идеален, но он работает.

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

vdm, Вы, вообще-то, начинаете совсем не с того.

Начинать нужно с подробного описания алгоритма. По-русски.

Вы же предлагаете взять код с ошибками и "исправить" его. 

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

Вы понимаете разницу между "сейчас работает" и "должен работать"?

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

Дальше: отладка - процесс примерно на порядок более трудоемкий, чем написание кода. Следовательно, код надо писать так, чтобы его легче было отлаживать. Следовательно, во многих случаях переписать код с нуля - рациональнее, чем пытаться исправить существующий код. А чтобы написать с нуля, опять же, нужно описание алгоритма, и совершенно не нужен неправильно работающий код.

Хорошо, учту. Исправлюсь.

Код должен обабатывать кнопки btnLEFT, btnRIGHT, btnPROG и btnOK.

В результате короткого нажатия инвертируется состояние leftTurn, rightTurn, headLight и specialLight соответственно.

Длительное нажатие и удержание кнопок btnLEFT и btnRIGHT приводит к уменьшению и увеличению переменной consumption.

Переменные leftTurn, rightTurn, headLight и specialLight типа boolean.

- integer

Значения переменных leftTurn, rightTurn, headLight, specialLight и consumption записываются с массив и отправоляются в модуль nrf24l01.

int port[6];

Изначально был обьявлен массив port[6] куда я все и записывал. При этом возникала ошибка в переменных headLight, specialLight и consumption. 

  port[0] = thortle;                  // Записываем газ
  port[1] = steering;                 // записываем положение руля
  port[2] = leftTurn;                 // Левый поворотник
  port[3] = rightTurn;                // правый поворотник
  port[4] = headLight;                // фары
  port[5] = specialLight;             // спецсигнал
  port[6] = consumption;              // расходы

Когда изменил размер массива на 8, ошибка ушла.

Вероятнее всего вся проблема была в том, что я записывал в массив как значения int, так и boolean

если у вас так:

  port[0] = thortle;                  // Записываем газ
  port[1] = steering;                 // записываем положение руля
  port[2] = leftTurn;                 // Левый поворотник
  port[3] = rightTurn;                // правый поворотник
  port[4] = headLight;                // фары
  port[5] = specialLight;             // спецсигнал
  port[6] = consumption;              // расходы

то вообще-то нужно создавать массив int port[7],  если что. 

Ну правильно... Если [6] то и пунктов 6.... [0] то не считаем... В)))))