Как "красиво" написать этот кусочек?, Хочется писать красиво... Опции

[Pyku_He_oTTyda]

Вот такой кусочек кода, как написать его покрасивее?
Заранее благодарен за внимание!

Код

void read_key(void)
{
if(PINA==0xFE)
{
up_press=1;
}
else
{
up_press=0;
};

if(PINA==0xFD)
{
down_press=1;
}
else
{
down_press=0;
};

if(PINA==0xFB)
{
left_press=1;
}
else
{
left_press=0;
};

if(PINA==0xF7)
{
right_press=1;
}
else
{
right_press=0;
};

if(PINA==0xEF)
{
mode_press=1;
}
else
{
mode_press=0;
};

if(PINA==0xDF)
{
set_press=1;
}
else
{
set_press=0;
};
}

[Pyku_He_oTTyda]

Спасибо!
Несколько кнопок не могут быть нажаты, это считается ошибкой.

З.Ы. Прошу прощения, с использованием копи-пасте пропустил ошибку, key_press в моем коде не работает, счас поправлю

[Сергей Борщ]

Спасибо!
Несколько кнопок не могут быть нажаты, это считается ошибкой.

Тогда вполне мой второй вариант подходит. Зачем держать семь переменных, для которых валидным будет только семь значений на всех? Можно еще поправить:

Код

enum keypressed_t {RELEASED, UP, DOWN, LEFT, RIGHT, MODE, SET, MAX_KEY};
void read_key()
{
   uint8_t Port_Val = ~PINA;
   uint8_t Mask = 1 << (MAX_KEY - 1);
   uint8_t Key_tmp = MAX_KEY - 1;
   do
   {
     if(Port_Val & Mask)
       break;
     Mask >>= 1;
     Key_tmp--;
   }
   while(Mask);
   Key = Key_tmp;
}

[singlskv]

Тогда вполне мой второй вариант подходит. Зачем держать семь переменных, для которых валидным будет только семь значений на всех? Можно еще поправить:

Код

     if(Port_Val & Mask)
       break;
}

так мы ловим первую по списку нажатую
чтобы проверить что нажата только 1 нужно заменить break; на например tmp++;
и в конце цикла проверять tmp<2

[Сергей Борщ]

так мы ловим первую по списку нажатую
чтобы проверить что нажата только 1 нужно заменить break; на например tmp++;
и в конце цикла проверять tmp<2

Да, про то, что две нажатые считаются как "ни одной" на момент написания не было. Тогда можно или считать, или так:

Код

void read_key()
{
   uint8_t Port_Val = ~PINA;
   Port_Val &= (1<<MAX_KEYS) - 1;    // mask unused bits
   if(!(Port_Val & (Port_Val - 1)))  // if only one key pressed
   {
      uint8_t Mask = 1 << (MAX_KEY - 1);
      uint8_t Key_tmp = MAX_KEY - 1;
      do
      {
        if(Port_Val & Mask)
           break;
        Mask >>= 1;
        Key_tmp--;
      }
      while(Mask);
      Key = Key_tmp;
   }
   else
      Key = RELEASED;
}