Делаем из кнопок и светодиодов минипанельку управления. Кнопки 6X6X10MM 4PIN без фиксации.

bool func(bool someParam) {
    if (someParam.ToString().Length == 4)
        return true;
    else
        return false;

int max(int a, int b)
{
  int c = 0;
  while (c < a && c < b)
    c++;
  if (c == a)
    return a;
  if (c == b)
    return b;
  if (c != a && c != b)
    return c;
}

А что здесь комментировать?

Если вам в том коде надо ставить комменты, то вы или домашний программер или вообще к этому делу кроме как диванного эксперта отношения не имеете.

что код, написанный лет 10 назад, не всегда понятен

В начале функции идет описание, мол делается то и то, вводные параметры такие, выдает то и то.

то читать чужой код вам ни когда не стоит (а может и свой).

...

	switch (a)
	{
		case 2:
			i += 8;
			break;
		case 3:
			i += 9;
			break;
		case 5:
			i += 10;
			break;
		case 8:
			i += 11;
			break;
		case 12:
			i += 12;
			break;
		case 13:
			i += 13;
			break;
	}

	if (a == 2)
	{
		i += 8;
	}
	else
	{
		if (a == 3)
		{
			i += 9;
		}
		else
		{
			if (a == 5)
			{
				i += 10;
			}
			else
			{
				if (a == 8)
				{
					i += 11;
				}
				else
				{
					if (a == 12)
					{
						i += 12;
					}
					else
					{
						if (a == 13)
						{
							i += 13;
						}
					}
				}
			}
		}
	}

if (a == 2) i += 8;
else if (a == 3) i += 9;
else if (a == 5) i += 10;
else if (a == 8) i += 11;
else if (a == 12) i += 12;
else if (a == 13) i += 13;

if (a == 2) {
  i += 8;
} else if (a == 3) {
  i += 9;
} else if (a == 5) {
  i += 10;
} else if (a == 8) {
  i += 11;
} else if (a == 12) {
  i += 12;
} else if (a == 13) {
  i += 13;
}

	switch (a)
	{
		case 1:
			i += 8;
			break;
		case 2:
			i += 9;
			break;
		case 3:
			i += 10;
			break;
		case 4:
			i += 11;
			break;
		case 5:
			i += 12;
			break;
		case 6:
			i += 13;
			break;
		default:
			i += 14;
	}

	if (a == 1)
	{
		i += 8;
	}
	else
	{
		if (a == 2)
		{
			i += 9;
		}
		else
		{
			if (a == 3)
			{
				i += 10;
			}
			else
			{
				if (a == 4)
				{
					i += 11;
				}
				else
				{
					if (a == 5)
					{
						i += 12;
					}
					else
					{
						if (a == 6)
						{
							i += 13;
						}
						else
						{
							i += 14;
						}
					}
				}
			}
		}
	}

main:
        LDR.N    R0,??main_1
        LDRB     R1,[R0, #+0]
        SUBS     R1,R1,#+1
        CMP      R1,#+5
        BHI.N    ??main_2
        TBB      [PC, R1]
        DATA
??main_0:
        DC8      0x3,0x7,0xB,0xF
        DC8      0x13,0x17
        THUMB
??main_3:
        LDRH     R1,[R0, #+2]
        ADDS     R1,R1,#+8
        STRH     R1,[R0, #+2]
        B.N      ??main_4
??main_5:
        LDRH     R1,[R0, #+2]
        ADDS     R1,R1,#+9
        STRH     R1,[R0, #+2]
        B.N      ??main_4
??main_6:
        LDRH     R1,[R0, #+2]
        ADDS     R1,R1,#+10
        STRH     R1,[R0, #+2]
        B.N      ??main_4
??main_7:
        LDRH     R1,[R0, #+2]
        ADDS     R1,R1,#+11
        STRH     R1,[R0, #+2]
        B.N      ??main_4
??main_8:
        LDRH     R1,[R0, #+2]
        ADDS     R1,R1,#+12
        STRH     R1,[R0, #+2]
        B.N      ??main_4
??main_9:
        LDRH     R1,[R0, #+2]
        ADDS     R1,R1,#+13
        STRH     R1,[R0, #+2]
        B.N      ??main_4
??main_2:
        LDRH     R1,[R0, #+2]
        ADDS     R1,R1,#+14
        STRH     R1,[R0, #+2]
??main_4:
        MOVS     R0,#+0
        BX       LR               ;; return

main:
        LDR.N    R0,??main_0
        LDRH     R1,[R0, #+2]
        LDRB     R2,[R0, #+0]
        CMP      R2,#+1
        BNE.N    ??main_1
        ADDS     R1,R1,#+8
        STRH     R1,[R0, #+2]
        B.N      ??main_2
??main_1:
        CMP      R2,#+2
        BNE.N    ??main_3
        ADDS     R1,R1,#+9
        STRH     R1,[R0, #+2]
        B.N      ??main_2
??main_3:
        CMP      R2,#+3
        BNE.N    ??main_4
        ADDS     R1,R1,#+10
        STRH     R1,[R0, #+2]
        B.N      ??main_2
??main_4:
        CMP      R2,#+4
        BNE.N    ??main_5
        ADDS     R1,R1,#+11
        STRH     R1,[R0, #+2]
        B.N      ??main_2
??main_5:
        CMP      R2,#+5
        BNE.N    ??main_6
        ADDS     R1,R1,#+12
        STRH     R1,[R0, #+2]
        B.N      ??main_2
??main_6:
        CMP      R2,#+6
        BNE.N    ??main_7
        ADDS     R1,R1,#+13
        STRH     R1,[R0, #+2]
        B.N      ??main_2
??main_7:
        ADDS     R1,R1,#+14
        STRH     R1,[R0, #+2]
??main_2:
        MOVS     R0,#+0
        BX       LR               ;; return

if (1<a & а<7)
i=14
else
i+=a;
Как это будет выглядить в асме?

main:
        LDR.N    R0,??main_0
        LDRB     R1,[R0, #+0]
        SUBS     R2,R1,#+2
        CMP      R2,#+5
        BCS.N    ??main_1
        MOVS     R1,#+14
        STRH     R1,[R0, #+2]
        B.N      ??main_2
??main_1:
        LDRH     R2,[R0, #+2]
        ADDS     R1,R1,R2
        STRH     R1,[R0, #+2]
??main_2:
        MOVS     R0,#+0
        BX       LR               ;; return

совсем не аналогичен по результату моему примеру

как раз у Вас частный случай оптимизации конкретного примера

Чем же он не аналогичен?

первому так же аналогичен, только опять же я с «И» пролетел и a целочисленная

if (1<a & а<7)
  i=14
else
  i+=a;

if (0<a && а<7)
  i+=14
else
  {
    i+=a;
    i+=7;
  }

компилятор превратил switch из переходов в бинарное дерево. Но опять же это тоже частый случай.

каждое меню — структура — какой отображать текст, какую переменную, и указатель на функцию.

typedef struct
{
	Int8U 		name_font;
	Int8U 		val_font;
	Int16U		back_color[2];		// normal, selected
	Int16U 		name_color[3];		// normal, selected, disabled
	Int16U 		val_color[4];		// normal, selected, disabled, editing
	Int8U		l_indent;
	Int8U		r_indent;
	Int16S		x;
	Int16S		y;
	Int16S		w;
	Int16S		h;
	Int8U		items_count;
	Int8U 		selected_item;
	Int16U		flags;
	MENU_LINES_ITEM	*items;
	void		*prevmenu;
	void		*nextmenu;
	void		*parentmenu;
	Int32S		oldnval;
	char		oldcval[16];
	menu_func	menudrawfunc;
} MENU_LINES_STRUCT;

typedef struct
{
	const char 	*text;
	char		cval[16];
	Int32S		nval;
	Int16S		x;
	Int16S		y;
	Int16S		w;
	Int16S		h;
	Int16U		flags;
	void*		childmenu;
	menuitem_func	activatefunc;
	menuitem_func	drawfunc;
	menuitem_func	changevaluefunc;
} MENU_LINES_ITEM;

Редактирование параметра у меня одна кнопка, и не важно что это за пункт

функция инкремента переменной увеличивает значение, если вышло за рамки допустимого для «типа» — обнуляет

void	SomeValueChange(void *menu, Int8U item, void *param)
{
	MENU_LINES_STRUCT *mn = (MENU_LINES_STRUCT*)menu;
	Int32S *val = &(mn->items[mn->selected_item].nval);
	Int32S oldval = *val;
	Int32U encspd = KEYB_GetEncSpeed();
	// time
	if (mn->selected_item == 0)
	{
		if (encspd > 150)
			*val += KEYB_GetEncCount();
		else
			if (encspd > 100)
				*val += KEYB_GetEncCount() * 2;
			else
				if (encspd > 60)
					*val += KEYB_GetEncCount() * 10;
				else
					*val += KEYB_GetEncCount() * 100;
		if (*val < 1)
			*val = 1;
		if (*val > 5000)
			*val = 5000;
	}
	// power
	else
	{
		if (mn->selected_item == 1)
		{
			if (encspd > 100)
				*val += KEYB_GetEncCount()*5;
			else
				*val += KEYB_GetEncCount() * 10;
			if (*val < 5)
				*val = 5;
			if (*val > 100)
				*val = 100;
		}
	}
	if (oldval == *val)
		return;
	sprintf(mn->items[mn->selected_item].cval, "%d", *val);
	mn->items[mn->selected_item].drawfunc(menu, item, param);
}

А функция вывода на экран знает как выводить ту или иную переменную, в зависимости от «типа» (числом или текстом и каким).

одну сделал универсальной — указатель на вызов функции «записать/применить/выполнить/Enter»

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

Из основных различий: указателей на функции у меня только один, все значения константны (указатели, текст и неизменяемые параметры меню) и удалось все запихнуть в EEPROM (AVR).

const char* STRING_Otmena = "Отмена";
const char* STRING_Vyhod = "Выход";
const char* STRING_Sohranit = "Сохранить";
const char* STRING_SOHRANIT = "СОХРАНИТЬ";
const char* STRING_Zagruzit = "Загрузить";
const char* STRING_ZAGRUZIT = "ЗАГРУЗИТЬ";
...

У меня не обнуляет, а просто перестает увеличивать или уменьшать по достижении пределов:

switch (pgm_read_byte(&menu[menu_cur].maximum))
		{	case maximum_32:			if(32		 <lcd_tmp_var)lcd_tmp_var=32;return;
			case maximum_8:			if(8		 <lcd_tmp_var)lcd_tmp_var=8;return;
			case maximum_20000:		if(20000	 <lcd_tmp_var)lcd_tmp_var=20000;return;
			case maximum_sl:			if(2000000000<lcd_tmp_var)lcd_tmp_var=2000000000;return;
			case maximum_255_or_disable:if(255 		 <lcd_tmp_var)lcd_tmp_var=255;return;
			case maximum_500:			if(500 	 <lcd_tmp_var)lcd_tmp_var=500;return;
			case maximum_1440:		if(1440	 <lcd_tmp_var)lcd_tmp_var=1440;return;
		}
// Ах сколько магических констант в коде. Ну ничего, проект старый... и мой самый долгоиграющий, еще не все плохие свои методики от туда удалил.

Сохранение текущих настроек

Не смейтесь над именами — я в них пишу буквальную транскрипцию для максимально точного понимания что это за строка :)))

#define uart_receiver_buf_size_8 8

    Module                    ro code  ro data  rw data  rw data
                                                          (abs)
    ------                    -------  -------  -------  -------
D:\Electronics\Projects\welding\v0.1\PRG_flash\Debug\Obj: [1]
    allmenu.o                   4 380             2 868
    buzzer.o                      648                68
    flash.o                     1 088                         16
    font_1b62h_splash.o                  1 547       16
    ili9225.o                   4 772        2       22
    keyb.o                        792       19       12
    main.o                      8 124      497       66
    menus.o                     3 400
    startup_stm32f0xx.o           286        2
    stm32f0xx_gpio.o              372
    stm32f0xx_it.o                 28
    stm32f0xx_misc.o              120
    stm32f0xx_rcc.o               220
    stm32f0xx_spi.o               284
    stm32f0xx_tim.o             1 076
    strings_def.o                          490      144
    system_stm32f0xx.o            372
    tft_font_1b12h.o                     1 995       16
    tft_font_1b18h.o                     4 003       16
    tft_font_1b22h_digits.o                980       16
    tft_font_1b22h_symbols.o               741       16
    timer.o                       640               140
    ------------------------------------------------------------
    Total:                     26 602   10 276    3 400       16

Иконки у меня тоже символами, потому что на 1604 никак по-другому, но неплохо выходит, даже анимация.

а это факт )

погрешность в миллисекунду — никак не скажется на результате

поэтому фокусы с регулировками дополнительными не дают плодов

сборная солянка из десятков просмотренных схем

вижу знакомые детали ) то есть вы решили клон моей штуки сделать

С учетом текущего положения вашего кода выходит совсем мелочь домалевать для работы с сенсором.

ок и отмена я так понимаю пункты меню и каждый раз они описываются сами по себе?

А что не так у автора?

И откажитесь вы уже от свича. If. Доказано что он быстрее.

Ну и не все состояния будут отрабатываться корректно когда, например, главный цикл где-то задержался и пользователь успел нажать и отпустить кнопку.

sir0ta: А что не так у автора? Все те же состояния кнопки.

sir0ta:И откажитесь вы уже от свича. If. Доказано что он быстрее.

AndyBig: свич прыгнет сразу в нужную ветку.

Для них все равно — кнопок 20 или одна с разными способами нажатия, у них 20 событий и все.

Давно читал что switch может разворачиваться в бинарное дерево примерно так:

Я даже не понял как switch был оптимизирован, даже отрицательные числа появились

ммм… не едут лыжи… Что значит события? Получается вы функции приписываете какое-то событие. Затем в каком-то месте типа проходим по сопоставлениям и если событие произошло то запускаем функцию? Даже не представлю как по другому.

// псевдокод
int getkey(){
if (button1_short) return event1;
if (button2_short) return event2;
if (button2_long) return event3;
if (button1 && button2) return event4;}

if (pin1_short) return button1;
if (pin2_short) return button2;
if (pin2_long) return button3;
if (pin1 && pin2) return button4;

а break ни кто не забыл в кейсах?

Не будем спорить, юзайте в удовольствие )