Помогите с обработкой кнопок в CVAVR

[Dimitrij]

Сразу сильно не пинайте, я только учусь.
Обрабатываю однократное нажатие кнопок в CVAVR:

if(!PINB.1){            //ели нажата кнопка 1, то однократно   
       if(!PINB.2) Key=3;  //отрабатывается Key=4, а если кнопка2,  
       else Key=4;}          //то отрабатывается Key=3
     else Key=0;             //ели ничего не нажато 

Кнопка на PINB.1 отрабатывается нормально, а на PINB.1 не работает.
Что не так?

[Saadov]

Исправте последнее предложение.
У вас PINB.2 будет отрабатываться, если нажата PINB.1.
Задумывалось то как?

[Dimitrij]

Задумывалось то как?

Если нажать на кнопку1 (PINB.1), то однократно сработает Key=4,
а ели на кнопку2 (PINB.2), то однократно сработает Key=3.
Если код такой:

if(!PINB.1) Key=4;
   else Key=0;
if(!PINB.2) Key=3;
  else Key=0; 

то срабатывает только Key=3
Что-то совсем запутался.

[Saadov]

int kn, Key = 0;
while(1)
{
if(!PINB.1){ kn = 0; delay_ms(20); while (PINB.1 == kn); Key=4; };
if(!PINB.2){ kn = 0; delay_ms(20); while (PINB.2 == kn); Key=3; };
}; 

В вашем случае мы нажали PINB.1 и присвоили переменной Key значение 4, дальше у нас по ходу действия программы проверяется нажата ли PINB.2, если не нажата, то Key присваивается значение 0, т.е вы почти сразу же обнуляете Key.

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

[Saadov]

Ах да, если ни одна кнопка не нажата, то у вас Key будет равен 0, если хоть одна нажата, то при обработке значения Key, просто обнуляете Key, т.е.

switch(Key){
case 4:  Key = 0;  //делаем что-то, если нажата кнопка 1
break;
case 3:  Key = 0;  //делаем что-то, если нажата кнопка 2
break;
}; 

[Dimitrij]

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

Но мне нужно, чтобы при отпускании кнопки значение переменной вернулось в исходное (Key=0), а при Вашем коде оно не возвращается.
У меня загвоздка как раз в том, чтобы при отпускании кнопки вернуть исходное значение.
Может, я изначально не так поставил вопрос?
Код функции:

unsigned char KEYPRESS (void);

int kn, Key = 0;
unsigned char KEYPRESS ()
{
if(!PINB.1){ kn = 0; delay_ms(20); while (PINB.1 == kn); Key=4; };
if(!PINB.2){ kn = 0; delay_ms(20); while (PINB.2 == kn); Key=3; };

return(Key);
}; 

[Yurkin2007]

Можно так сделать: при нажатии однократно вернётся ненулевое значение, в остальных случаях возвращает 0

unsigned char KEYPRESS (void); 

unsigned char oldPIN1=1;
unsigned char oldPIN2=1;

unsigned char KEYPRESS () 
{ 
  if(!PINB.1)
  { 
    if(oldPIN1)
    {
      oldPIN1=0;
      return 4;
    }
  }
  else oldPIN1=1;
  
  if(!PINB.2)
  { 
    if(oldPIN2)
    {
      oldPIN2=0;
      return 3;
    }
  }
  else oldPIN2=1;
 
  return 0;
} 

[Dimitrij]

делаем что-то, если нажата кнопка

Что-то в этом коде я ничего не пойму. Как программа поймет, на каком порту какая кнопка нажата?

[Wiza_]

Key=0;
if(!PINB.1) Key=4;
if(!PINB.2) Key=3;

При одновременно нажатых кнопках Key=3;

Если надо обрабатывать обе кнопки - вам к битовым операциям
Key=0;
if(!PINB.1) Key |= (1 ‹‹ 1);
if(!PINB.2) Key |= (1 ‹‹ 2);

Нажатия конопок:
ничего не нажато - 0
Нажата первая кнопка - 2
Нажата вторая кнопка - 4
Нажаты обе кнопки - 6

[Boba_spb]

Key=0;
if(!PINB.1) Key=4;
if(!PINB.2) Key=3;

При одновременно нажатых кнопках Key=3;

Если надо обрабатывать обе кнопки - вам к битовым операциям
Key=0;
if(!PINB.1) Key |= (1 ‹‹ 1);
if(!PINB.2) Key |= (1 ‹‹ 2);

Нажатия конопок:
ничего не нажато - 0
Нажата первая кнопка - 2
Нажата вторая кнопка - 4
Нажаты обе кнопки - 6

сдвиги то зачем?
Key = PINB & 6;