if (LightLcd){
if ((millis()-LcdTimeStart)>15000){ //Тушим подсветку при простое более 15 сек
lcd.noDisplay();
digitalWrite(8,LOW);
}
}
void LCDBacklight(byte v) { // Управление подсветкой
if (v == 0) { // Выключить подсветку
BacklightOffTime = millis();
lcd.noBacklight();
}
else if (v == 1) { //Включить подсветку
BacklightOffTime = millis() + BacklightDelay;
lcd.backlight();
}
else { // Выключить если время вышло
if (BacklightOffTime < millis())
lcd.noBacklight();
else
lcd.backlight();
}
}прошу прощения, Эдуард, но строчка 7 сразу ставит под сомнение профессионализм автора. Так делать нельзя.
Почему же ? вполне можно вычислить время выключения.. но вот если будет превышено максимальное значение, то будут глюки.
ну вот именно поэтому и нельзя так делать
void LCDBacklight(byte v) { // Управление подсветкой
if (v == 0) { // Выключить подсветку
lcd.noBacklight();
}
else if (v == 1) { //Включить подсветку
BacklightONTime = millis();
lcd.backlight();
}
else { // Выключить если время вышло
if (millis() - BacklightONTime > BacklightDelay) lcd.noBacklight();
}
}
не исключено, что это правильная идея.Сделать удобный пользовательсткий интерфейс на ардуине - имхо, задачка посложнее самого контроллера освещения будет.
Кстати! Хорошая идея для коммерческих кодов, ставишь такой код, и раз 50 дней все это глючит. А если взять знаковое число, то глючить будет раз в 25 дней. Убираешь, если полностью доволен оплатой. Нет оплаты - пардоньте
#include <LiquidCrystal.h>
class Matrixbutton {
public:
Matrixbutton(byte setPinX[], byte setCountX, byte setPinY[], byte setCountY){
_x=setCountX;
_y=setCountY;
for (byte ix=0; ix<_x;ix++){
pin_x[ix]=setPinX[ix];
}
for (byte iy=0; iy<_y;iy++){
pin_y[iy]=setPinY[iy];
}
}
void read(){
for (byte xm=0;xm<_x;xm++){ // перебираем столбики
digitalWrite(pin_x[xm],1);
delay(20); //Физика, при длинных проводах обязательно делать небольшую задержку
for (byte ym=0;ym<_y;ym++){ //Читаем строчки
current_state[xm][ym]=digitalRead(pin_y[ym]);
if (last_state[xm][ym]==0 && current_state[xm][ym]==1){ //Если предыдущее 0, а новое 1, значит клавиша нажата
event_down[xm][ym]=1;
eventdown_start[xm][ym]=millis(); //Стартуем таймер, дабы отследить длительность нажатия кнопки
}
if (eventdown_start[xm][ym]!=0){ //Если клавиша долго нажата то перестаём пытаться регистрировать события быстрого и долгого клика
if ((millis()-eventdown_start[xm][ym])>= 5000){
eventdown_start[xm][ym]=0;
}
}
if (last_state[xm][ym]==1 && current_state[xm][ym]==0){ //Если предыдущая 1, а новое 0, значит клавиша отпущена
event_up[xm][ym]=1; //Регистрируем события отпускания клавиши, но ещё необходимо проверить события быстрого или долгого клика
if (eventdown_start[xm][ym]!=0){
if ((millis()-eventdown_start[xm][ym]) < 1100) { // Если клик меньше 1.1 сек, то будем считать это быстрым кликом, если от 1.1 до 5 сек, то долгий
event_fastclick[xm][ym]=1;
eventdown_start[xm][ym]=0;
} else {
event_longclick[xm][ym]=1;
eventdown_start[xm][ym]=0;
}
}
}
last_state[xm][ym]=current_state[xm][ym];
}
digitalWrite(pin_x[xm],0); //реверс
}
}
void init(){
for (byte i=0; i<_x;i++){ //Инициализируем пины
pinMode(pin_x[i],OUTPUT);
digitalWrite(pin_x[i],0); //На тот случай, если делать puulup
}
for (byte i=0; i<_y;i++){
pinMode(pin_y[i],INPUT); //Не забываем подтянуть строки к земле через резисторы, иначе будет весело, можно правда сделать puulup, но придётся немного программу изменить
}
for (byte xm=0;xm<_x;xm++){ //Обнуляем матрицы состояний и таймеров
for (byte ym=0;ym<_y;ym++){
current_state[xm][ym]=0;
last_state[xm][ym]=0;
event_up[xm][ym]=0;
event_down[xm][ym]=0;
event_fastclick[xm][ym]=0;
event_longclick[xm][ym]=0;
eventdown_start[xm][ym]=0;
}
}
}
boolean clickUp(byte button){ //Просто события, после считывания события - сбрасываем событие
if (event_up[button/_x][button%_y] == 1){
event_up[button/_x][button%_y] =0;
return true;
} else {
return false;
}
}
boolean clickDown(byte button){
if (event_down[button/_x][button%_y] == 1){
event_down[button/_x][button%_y] =0;
return true;
} else {
return false;
}
}
boolean fastclick(byte button){
if (event_fastclick[button/_x][button%_y] == 1){
event_fastclick[button/_x][button%_y] =0;
return true;
} else {
return false;
}
}
boolean longclick(byte button){
if (event_longclick[button/_x][button%_y] == 1){
event_longclick[button/_x][button%_y] =0;
return true;
} else {
return false;
}
}
private:
byte _x; //Количество столбиков
byte _y; //Количество строк
byte pin_x[9]; //Пины столбиков (Выходы)
byte pin_y[9]; //Пины строк (Входы)
boolean current_state[9][9]; //Размеры массивов должны быть не менее _x и _y
boolean last_state[9][9];
boolean event_up[9][9];
boolean event_down[9][9];
boolean event_fastclick[9][9];
boolean event_longclick[9][9];
unsigned long eventdown_start[9][9];
};
byte pinx_Str[]={37,39,41,43,45,47,49,51,53};
byte piny_Cool[]={36,38,40,42,44,46,48,50,52};
Matrixbutton knop(pinx_Str,9,piny_Cool,9);
LiquidCrystal lcd(12,11,5,4,3,2);
String str,str1,str2;
bool fl_cur,LightLcd;
unsigned long LcdTimeStart;
void setup() {
lcd.begin(16, 2);
fl_cur=false;
pinMode(8,OUTPUT); //Управление подсветкой LCD
lcd.noDisplay();
pinMode (13,OUTPUT);
knop.init();
Serial.begin(9600); // отладка
}
void loop() {
knop.read(); //Читаем кнопки
byte f=0;
while (f<=80){ //Если была нажата любая кнопка, то условие в цикле сработает
if (knop.clickUp(f)) {
str=str+"Up="+String(f)+" ";
fl_cur=true;
}
if (knop.longclick(f)){
str=str+"LongClick="+String(f)+" ";
fl_cur=true;
}
if (knop.fastclick(f)){
str=str+"FastClick="+String(f)+" ";
fl_cur=true;
}
if (knop.clickDown(f)){
str=str+"DOWN="+String(f)+" ";
fl_cur=true;
}
f++;
}
if (LightLcd){
if ((millis()-LcdTimeStart)>15000){ //Тушим подсветку при простое более 15 сек
lcd.noDisplay();
digitalWrite(8,LOW);
LightLcd=false;
}
}
if (fl_cur){ //Если были события с кнопок, кидаем их на экран
Serial.println(str);
digitalWrite(8,HIGH);
lcd.display();
LcdTimeStart=millis();
LightLcd=true;
lcd.clear();
str1=str2;
str2=str;
lcd.setCursor(0,0);
lcd.print(str1);
lcd.setCursor(0,1);
lcd.print(str2);
fl_cur=false;
str="";
}
}Я так же подумал, просто на схеме к подсветки тянется +3,3В, сейчас проверил ток потребления подсветки 3 милиампера, как я понимаю надо поставить резистор номиналом в 1,7/0,003=566 Ом, воткнул сперва напрямую, как то сильно ярко засветился, поставил через сопротивление на 510 Ом, сейчас всё в норме, просто много схем вчера в инете нарыл, зачастую все через транзистор подсветку врубают.
Чёт меня это начинает напрягать (не Вы, а именно этот факт про 50 дней), 50 дней, это как я понимаю предел millis(), но на сколько помню на другом форуме мне объяснили что можно плевать на переход через 50 дней и всё будет работать как надо... И да, флаг включенной подсветки в коде подправил, спасибо! Сейчас в рабочей системе нет у меня никаких millis и lcd, может я тут сижу и начинаю творить код для будущих граблей?Кстати! Хорошая идея для коммерческих кодов, ставишь такой код, и раз 50 дней все это глючит. А если взять знаковое число, то глючить будет раз в 25 дней. Убираешь, если полностью доволен оплатой. Нет оплаты - пардоньте
49 дней с хвостиком. Но все зависит от метода реализации. в 40-ом посте будет плевать, а в 37 нет.
Массивы передавать накладно, передавайте либо указатели на массивы, либо ссылки, либо используйте кучу (new/delete).
class Matrixbutton {
public:
Matrixbutton(byte* setPinX, byte setCountX, byte* setPinY, byte setCountY){
_x=setCountX;
_y=setCountY;
pin_x=setPinX;
pin_y=setPinY;
}
......
private:
byte _x; //Количество столбиков
byte _y; //Количество строк
byte* pin_x; //Пины столбиков (Выходы)
byte* pin_y;
......Это я уже понимаю, спасибо.
Класс видит все свои члены вне зависимости указанной области видимости. Она нужна лишь для доступа снаружи/наследования. Но наружу тоже можно через get-метод отдать, вернув ту же ссылку.
Я где то не понимаю, у меня есть
private:
boolean current_state[9][9]; //Размеры массивов должны соответствовать _x и _y
boolean last_state[9][9];
boolean event_up[9][9];
boolean event_down[9][9];
boolean event_fastclick[9][9];
boolean event_longclick[9][9];
unsigned long eventdown_start[9][9]; boolean current_state[9][9]; //Размеры массивов должны соответствовать _x и _y
boolean last_state[9][9];
boolean event_up[9][9];
boolean event_down[9][9];
boolean event_fastclick[9][9];
boolean event_longclick[9][9];Думаю у меня просто нет таланта правильно объяснять.
class Matrixbutton {
public:
Matrixbutton(byte* setPinX, byte setCountX, byte* setPinY, byte setCountY){
_x=setCountX;
_y=setCountY;
pin_x=setPinX;
pin_y=setPinY;
}
static const byte nr=81;
void read(){
for (byte xm=0;xm<_x;xm++){ // перебираем столбики
digitalWrite(pin_x[xm],1);
delay(20); //Физика, при длинных проводах обязательно делать небольшую задержку
for (byte ym=0;ym<_y;ym++){ //Читаем строчки
current_state[xm*_x+ym]=digitalRead(pin_y[ym]);
if (last_state[xm*_x+ym]==0 && current_state[xm*_x+ym]==1){ //Если предыдущее 0, а новое 1, значит клавиша нажата
event_down[xm*_x+ym]=1;
eventdown_start[xm*_x+ym]=millis(); //Стартуем таймер, дабы отследить длительность нажатия кнопки
}
if (eventdown_start[xm*_x+ym]!=0){ //Если клавиша долго нажата то перестаём пытаться регистрировать события быстрого и долгого клика
if ((millis()-eventdown_start[xm*_x+ym])>= 5000){
eventdown_start[xm*_x+ym]=0;
}
}
if (last_state[xm*_x+ym]==1 && current_state[xm*_x+ym]==0){ //Если предыдущая 1, а новое 0, значит клавиша отпущена
event_up[xm*_x+ym]=1; //Регистрируем события отпускания клавиши, но ещё необходимо проверить события быстрого или долгого клика
if (eventdown_start[xm*_x+ym]!=0){
if ((millis()-eventdown_start[xm*_x+ym]) < 1100) { // Если клик меньше 1.1 сек, то будем считать это быстрым кликом, если от 1.1 до 5 сек, то долгий
event_fastclick[xm*_x+ym]=1;
eventdown_start[xm*_x+ym]=0;
} else {
event_longclick[xm*_x+ym]=1;
eventdown_start[xm*_x+ym]=0;
}
}
}
last_state[xm*_x+ym]=current_state[xm*_x+ym];
}
digitalWrite(pin_x[xm],0); //реверс
}
}
void init(){
for (byte i=0; i<_x;i++){ //Инициализируем пины
pinMode(pin_x[i],OUTPUT);
digitalWrite(pin_x[i],0); //На тот случай, если делать puulup
}
for (byte i=0; i<_y;i++){
pinMode(pin_y[i],INPUT); //Не забываем подтянуть строки к земле через резисторы, иначе будет весело, можно правда сделать puulup, но придётся немного программу изменить
}
for (byte xm=0;xm<(_x*_y);xm++){ //Обнуляем матрицы состояний и таймеров
current_state[xm]=0;
last_state[xm]=0;
event_up[xm]=0;
event_down[xm]=0;
event_fastclick[xm]=0;
event_longclick[xm]=0;
eventdown_start[xm]=0;
}
}
boolean clickUp(byte button){ //Просто события, после считывания события - сбрасываем событие
if (event_up[button] == 1){
event_up[button] =0;
return true;
} else {
return false;
}
}
boolean clickDown(byte button){
if (event_down[button] == 1){
event_down[button] =0;
return true;
} else {
return false;
}
}
boolean fastclick(byte button){
if (event_fastclick[button] == 1){
event_fastclick[button] =0;
return true;
} else {
return false;
}
}
boolean longclick(byte button){
if (event_longclick[button] == 1){
event_longclick[button] =0;
return true;
} else {
return false;
}
}
private:
byte _x; //Количество столбиков
byte _y; //Количество строк
byte* pin_x; //Пины столбиков (Выходы)
byte* pin_y; //Пины строк (Входы)
boolean current_state[nr]; //Размеры массивов должны быть не менее _x*_y
boolean last_state[nr];
boolean event_up[nr];
boolean event_down[nr];
boolean event_fastclick[nr];
boolean event_longclick[nr];
unsigned long eventdown_start[nr];
};#include <LiquidCrystal.h>
class Matrixbutton {
public:
Matrixbutton(byte* setPinX, byte setCountX, byte* setPinY, byte setCountY){
_x=setCountX;
_y=setCountY;
pin_x=setPinX;
pin_y=setPinY;
byte nnr=_x*_y;
boolean *current_state0 = new boolean [nnr]; //высвобождаем место в памяти под массивы нужной длины
current_state=current_state0; //ссылку на освобождённое место передаём в private, для
boolean *last_state0 = new boolean [nnr]; //организации доступа к данной области памяти из других функций класса
last_state=last_state0;
boolean *event_up0 = new boolean [nnr];
event_up=event_up0;
boolean *event_down0 = new boolean [nnr];
event_down=event_down0;
boolean *event_fastclick0 = new boolean [nnr];
event_fastclick=event_fastclick0;
boolean *event_longclick0 = new boolean [nnr];
event_longclick=event_longclick0;
unsigned long *eventdown_start0 = new unsigned long [nnr];
eventdown_start=eventdown_start0;
}
void read(){
for (byte xm=0;xm<_x;xm++){ // перебираем столбики
digitalWrite(pin_x[xm],1);
delay(20); //Физика, при длинных проводах обязательно делать небольшую задержку
for (byte ym=0;ym<_y;ym++){ //Читаем строчки
current_state[xm*_x+ym]=digitalRead(pin_y[ym]);
if (last_state[xm*_x+ym]==0 && current_state[xm*_x+ym]==1){ //Если предыдущее 0, а новое 1, значит клавиша нажата
event_down[xm*_x+ym]=1;
eventdown_start[xm*_x+ym]=millis(); //Стартуем таймер, дабы отследить длительность нажатия кнопки
}
if (eventdown_start[xm*_x+ym]!=0){ //Если клавиша долго нажата то перестаём пытаться регистрировать события быстрого и долгого клика
if ((millis()-eventdown_start[xm*_x+ym])>= 5000){
eventdown_start[xm*_x+ym]=0;
}
}
if (last_state[xm*_x+ym]==1 && current_state[xm*_x+ym]==0){ //Если предыдущая 1, а новое 0, значит клавиша отпущена
event_up[xm*_x+ym]=1; //Регистрируем события отпускания клавиши, но ещё необходимо проверить события быстрого или долгого клика
if (eventdown_start[xm*_x+ym]!=0){
if ((millis()-eventdown_start[xm*_x+ym]) < 1100) { // Если клик меньше 1.1 сек, то будем считать это быстрым кликом, если от 1.1 до 5 сек, то долгий
event_fastclick[xm*_x+ym]=1;
eventdown_start[xm*_x+ym]=0;
} else {
event_longclick[xm*_x+ym]=1;
eventdown_start[xm*_x+ym]=0;
}
}
}
last_state[xm*_x+ym]=current_state[xm*_x+ym];
}
digitalWrite(pin_x[xm],0); //реверс
}
}
void init(){
for (byte i=0; i<_x;i++){ //Инициализируем пины
pinMode(pin_x[i],OUTPUT);
digitalWrite(pin_x[i],0); //На тот случай, если делать puulup
}
for (byte i=0; i<_y;i++){
pinMode(pin_y[i],INPUT); //Не забываем подтянуть строки к земле через резисторы, иначе будет весело, можно правда сделать puulup, но придётся немного программу изменить
}
for (byte xm=0;xm<(_x*_y);xm++){ //Обнуляем матрицы состояний и таймеров
current_state[xm]=0;
last_state[xm]=0;
event_up[xm]=0;
event_down[xm]=0;
event_fastclick[xm]=0;
event_longclick[xm]=0;
eventdown_start[xm]=0;
}
}
boolean clickUp(byte button){ //Просто события, после считывания события - сбрасываем событие
if (event_up[button] == 1){
event_up[button] =0;
return true;
} else {
return false;
}
}
boolean clickDown(byte button){
if (event_down[button] == 1){
event_down[button] =0;
return true;
} else {
return false;
}
}
boolean fastclick(byte button){
if (event_fastclick[button] == 1){
event_fastclick[button] =0;
return true;
} else {
return false;
}
}
boolean longclick(byte button){
if (event_longclick[button] == 1){
event_longclick[button] =0;
return true;
} else {
return false;
}
}
private:
byte _x; //Количество столбиков
byte _y; //Количество строк
byte* pin_x; //Пины столбиков (Выходы)
byte* pin_y; //Пины строк (Входы)
boolean* current_state;
boolean* last_state;
boolean* event_up;
boolean* event_down;
boolean* event_fastclick;
boolean* event_longclick;
unsigned long* eventdown_start;
};
byte pinx_Str[]={37,39,41,43,45,47,49,51,53};
byte piny_Cool[]={36,38,40,42,44,46,48,50,52};
Matrixbutton knop(pinx_Str,9,piny_Cool,9);
LiquidCrystal lcd(12,11,5,4,3,2);
String str,str1,str2;
bool fl_cur,LightLcd;
unsigned long LcdTimeStart;
void setup() {
lcd.begin(16, 2);
fl_cur=false;
pinMode(8,OUTPUT); //Управление подсветкой LCD
lcd.noDisplay();
pinMode (13,OUTPUT);
knop.init();
Serial.begin(9600); // отладка
}
void loop() {
knop.read(); //Читаем кнопки
byte f=0;
while (f<=80){ //Если была нажата любая кнопка, то условие в цикле сработает
if (knop.clickUp(f)) {
str=str+"Up="+String(f)+" ";
fl_cur=true;
}
if (knop.longclick(f)){
str=str+"LongClick="+String(f)+" ";
fl_cur=true;
}
if (knop.fastclick(f)){
str=str+"FastClick="+String(f)+" ";
fl_cur=true;
}
if (knop.clickDown(f)){
str=str+"DOWN="+String(f)+" ";
fl_cur=true;
}
f++;
}
if (LightLcd){
if ((millis()-LcdTimeStart)>15000){ //Тушим подсветку при простое более 15 сек
lcd.noDisplay();
digitalWrite(8,LOW);
LightLcd=false;
}
}
if (fl_cur){ //Если были события с кнопок, кидаем их на экран
Serial.println(str);
digitalWrite(8,HIGH);
lcd.display();
LcdTimeStart=millis();
LightLcd=true;
lcd.clear();
str1=str2;
str2=str;
lcd.setCursor(0,0);
lcd.print(str1);
lcd.setCursor(0,1);
lcd.print(str2);
fl_cur=false;
str="";
}
}Вот тут самое интересное. Когда вызывается деструктор ? сразу как цикл loop закончится ? Нет. а если он закончился, то значит что то пошло не так и надо жать ресет. А там все чистится. Или я тут где то заблуждаюсь ?
В коде ТС все так и есть - у него один экземпляр класса " навсегда"
, и работает он до выключения ардуины. По сути ему деструктор не нужен.
Фатальная ошибка. Хабр не только полезен, но и вреден.
У тебя же лежит в щитке резервный контроллер? И меняется он лёгким движением руки? Или прям горячий резерв?
Не очень понятна проблема с одновременным нажиманием кнопок. Семья бегает по дому и постоянно нажимает кнопки? Какова вероятность одновременного нажатия? И что за беда случится если их таки нажмут? Но в любом случае по диоду на каждую кнопку не надо, достаточно одного на линию: http://ns2.arduino.ru/sites/default/files/u1788/with_diode.png