arkhnchul писал(а):constexpr-ы обычно лаконичнее и понятнее развесистых макросов
Код: Выделить всё
#define bitrev(x) (((x) & 0x01 ? 0x80 : 0) | \
((x) & 0x02 ? 0x40 : 0) | \
((x) & 0x04 ? 0x20 : 0) | \
((x) & 0x08 ? 0x10 : 0) | \
((x) & 0x10 ? 0x08 : 0) | \
((x) & 0x20 ? 0x04 : 0) | \
((x) & 0x40 ? 0x02 : 0) | \
((x) & 0x80 ? 0x01 : 0))
Код: Выделить всё
tmp = bitrev((3<<A)|(4<<B)|(5<<C))
Код: Выделить всё
*(volatile uint8_t *)&SPI1->DR = bitrev(SPI1->DR);так я и спрашиваю о слабостях.VladislavS писал(а):Это, кстати, было не единственное слабое место макроса
вы приводили листинг С++ функции, которая в частном случае не привела к порождению кода.VladislavS писал(а):Я же приводил листинг
только в том случае, если будет особым образом оформлена и использована ТОЛЬКО с константным аргументом (и то не факт). во всех прочих случаях - будет порождать код.VladislavS писал(а):это будет обычная C функция, которая с константным аргументом тоже не будет порождать код
я лишь рассчитываю, что каждый говорящий в состоянии аргументировать свои слова. но если не в состоянии - я умолкаю...VladislavS писал(а):И после этого рассчитываете что вас учить будут?
Код: Выделить всё
{STATE1, EVENT1, handler_1},
{STATE1, EVENT1, handler_2},
{STATE2, EVENT2, handler_3}
Код: Выделить всё
eState fsNormalBypass() {
...
}
...
// Transition table type
typedef eState (*trans_t[ST_LAST][EV_LAST])(void);
// Transition table
trans_t trans = {
[ST_NORMAL][EV_BTN_SHORT] = fsNormalBypass,
[ST_NORMAL][EV_BTN_LONG] = fsNormalClosed,
[ST_BYPASS][EV_BTN_SHORT] = fsBypassNormal
...
};
// Get event in order of priority
eEvent fsGetEvent() {
eEvent ret = EV_NONE;
...
return ret;
}
// Run the transition based on event
eRetCode fsTransition() {
#ifdef VERBOSE_LOGS
LOG("Transition: st %d, ev %d\r\n", state.cur_state, state.event);
#endif
if ((state.event < EV_LAST) && trans[state.cur_state][state.event] != NULL) {
state.prev_state = state.cur_state; // save previous state
state.cur_state = trans[state.cur_state][state.event](); // run the transition
return RET_OK;
}
return RET_ERROR;
}
int main(void)
{
...
while(1)
{
state.event = fsGetEvent();
if (state.event != EV_NONE)
fsTransition();
}
}
Код: Выделить всё
typedef enum {ST_NONE, ST_IDLE, ST_CALIBRATION, ST_NORMAL, ST_BYPASS, ST_OVERFLOW, ST_RESTORATION, ST_MAINTAINCE, ST_LAST} eState;
typedef enum {EV_NONE, EV_BTN_SHORT, EV_BTN_LONG, EV_BAT_LOW, EV_AC_SHORTAGE, EV_TIMEOUT, EV_LAST} eEvent;
