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

[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;
};
}

[SasaVitebsk]

А если так???

Код

#define                CNT_KEY        6                        // Число кнопок
__no_init    struct
{
uint8_t            Up            : 1,                    // Вверх
                            Down        : 1,                    // Вниз
                            Left        : 1,                    // Влево
                            Right        : 1,                    // Вправо
                            Mode        : 1,                    // Мода
                            Set            : 1;                    // Установить
} Key;



void read_key()
{
uint8_t            i,j,k;


Key = ~PINA;
i = ~Key;
k=0;
for(j=0;j<CNT_KEY;j++) {k += i & 1; i >>= 1};
if(k>1) Key = 0;
}


// А анализировать так

if(Key.Up)
{
....
}

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

А если так???

Код

Key = ~PINA;
i = ~Key;

Присваивание целого структуре и наоборот недопустимо. Тогда уж

Код

union
{
    struct
   {
       uint8_t            Up            : 1,                    // Вверх
       uint8_t            Down        : 1,                    // Вниз
       ..........
    },
    uint8_t raw;
} Key;

Key.raw = ~PINA;
i = ~Key.raw;

[_sR_]

Присваивание целого структуре и наоборот недопустимо. Тогда уж

Код

union...

поддерживаю
кроме того, не помню, чтобы стандарт гарантировал прямое соответствие битов значения целого и битов
поля.
хотя признаю, что отклонения не встречал, но ведь вопрос был в красоте

[SasaVitebsk]

поддерживаю
кроме того, не помню, чтобы стандарт гарантировал прямое соответствие битов значения целого и битов
поля.
хотя признаю, что отклонения не встречал, но ведь вопрос был в красоте

А помоему это красивое решение (хотя понятие субъективное). И что ещё важнее - точно отражает суть. То есть у вас признаки нажатия. Очень читабелен текст.

Код

if(Key.Up)
{
   MotorVert(On);
}

Что тут написано вспомнишь ч/з год.

[_sR_]

А помоему это красивое решение...

Никто не сказал, что Ваше решение не красиво.
Более того, оно мне тоже нравится и в одном из проектов сделал точно так как Вы. Человек, который в последствии поддерживал этот код, понял все сразу.
Сергей Борщ просто показал как это записать "по циркулю" .
Я же предупредил, что такое присваивание, может содержать сюрприз, т.к. стандарт не определяет явно соответствие битовых полей и битов слова в котором они хранятся. К счастью для нас, все разработчики компиляторов думают в этом случии одинаково и надо просто не забывать о возможных граблях.