Оптимизация кода на С, возможно ли для этого кода?

Всем привет. Можно ли как то оптимизировать и сократить этот код? У кого какие идеи? А то что то громоздкий получился.

Спойлер

Код:: #define F_CPU 32000000UL #include <inttypes.h> #include <avr/io.h> #include <avr/interrupt.h> #include <avr/sleep.h> #include <util/delay.h> #include "keyboard.h" #include "lcd_lib.h" #define PAUSE 150 int k = 0; int b = 0; int i = 0; int c = 0; int cont = 0; int col[5] = {0b00000000, 0b00000001, 0b00000010, 0b00000100, 0b00001000}; int row[6] = {0b00011111, 0b00011110, 0b00011101, 0b00011011, 0b00010111, 0b00001111}; unsigned char buf = 0; unsigned char buttons[4]; void selectKey(int r, int c, int delay) { PORTD = col[r]; PORTC = row[c]; _delay_ms(PAUSE); PORTC = row[0]; PORTD = col[0]; _delay_ms(delay); } void send(void) { selectKey(1, 5, PAUSE); // RES selectKey(1, 4, PAUSE); // PRI selectKey(1, 1, PAUSE); // TOT selectKey(1, 2, PAUSE); // CAL int i; for ( i = 0; i < 4; i++) { selectKey(3, 5, PAUSE); // 0 } selectKey(4, 5, PAUSE); // OK selectKey(2, 2, PAUSE); // 1 } void end(void) { selectKey(1, 4, PAUSE); // PRI selectKey(1, 5, PAUSE); // RES } void fin(void) { LCD_Clear(); LCD_Goto(0, 0); LCD_SendString("Kod otpravlen! "); _delay_ms(1000); LCD_Clear(); b = 0; k = 0; for (i = 0; i < 4; i++) { buttons[i] = 0; } cont = 0; } int main(void) { DDRD=0xFF; DDRC=0xFF; LCD_Init(); LCD_Goto(0, 0); LCD_SendString("Vvedite "); LCD_Goto(4, 1); LCD_SendString("Koefitsent: "); _delay_ms(2000); LCD_Clear(); KEYB_Init(); while (1) { KEYB_ScanKeyboard(); buf = KEYB_GetKey(); if (buf) { if (b < 4 && buf != '*'&& buf != '#') { // Проверяем, есть ли еще место в массиве и выполняем ограничения на ввод if (b == 0 && buf != '1') { LCD_Goto(0, 0); LCD_SendString("Oshibka"); _delay_ms(2000); LCD_Clear(); b = 0; k = 0; for (i = 0; i < 4; i++) { buttons[i] = 0; } continue; // Пропускаем ввод, если первый адрес массива не равен единице } if (b == 1 && (buf != '0' && buf != '1')) { LCD_Clear(); LCD_Goto(0, 0); LCD_SendString("Oshibka"); _delay_ms(2000); LCD_Clear(); b = 0; k = 0; for (i = 0; i < 4; i++) { buttons[i] = 0; } continue; // Пропускаем ввод, если второй адрес массива не равен нулю или единице } buttons[b++] = buf; LCD_Goto(k++, 1); LCD_WriteData(buf); } if (buf == '*') { for (c = 0; c < 4; c++) { if (buttons[c] >= '0' && buttons[c] <= '9') { int digit = buttons[c] - '0'; cont = cont * 10 + digit; } } switch(cont) { case 1001: send(); selectKey(3, 5, PAUSE); selectKey(3, 5, PAUSE); selectKey(2, 2, PAUSE); end(); fin(); break; case 1002: send(); selectKey(3, 5, PAUSE); selectKey(3, 5, PAUSE); selectKey(3, 2, PAUSE); end(); fin(); break; case 1003: send(); selectKey(3, 5, PAUSE); selectKey(3, 5, PAUSE); selectKey(4, 2, PAUSE); end(); fin(); break; case 1004: send(); selectKey(3, 5, PAUSE); selectKey(3, 5, PAUSE); selectKey(2, 3, PAUSE); end(); fin(); break; case 1005: send(); selectKey(3, 5, PAUSE); selectKey(3, 5, PAUSE); selectKey(3, 3, PAUSE); end(); fin(); break; case 1006: send(); selectKey(3, 5, PAUSE); selectKey(3, 5, PAUSE); selectKey(4, 3, PAUSE); end(); fin(); break; case 1007: send(); selectKey(3, 5, PAUSE); selectKey(3, 5, PAUSE); selectKey(2, 4, PAUSE); end(); fin(); break; case 1008: send(); selectKey(3, 5, PAUSE); selectKey(3, 5, PAUSE); selectKey(3, 4, PAUSE); end(); fin(); break; case 1009: send(); selectKey(3, 5, PAUSE); selectKey(3, 5, PAUSE); selectKey(4, 4, PAUSE); end(); fin(); break; case 1010: send(); selectKey(3, 5, PAUSE); selectKey(2, 2, PAUSE); selectKey(3, 5, PAUSE); end(); fin(); break; case 1011: send(); selectKey(3, 5, PAUSE); selectKey(2, 2, PAUSE); selectKey(2, 2, PAUSE); end(); fin(); break; case 1012: send(); selectKey(3, 5, PAUSE); selectKey(2, 2, PAUSE); selectKey(3, 2, PAUSE); end(); fin(); break; case 1013: send(); selectKey(3, 5, PAUSE); selectKey(2, 2, PAUSE); selectKey(4, 2, PAUSE); end(); fin(); break; case 1014: send(); selectKey(3, 5, PAUSE); selectKey(2, 2, PAUSE); selectKey(2, 3, PAUSE); end(); fin(); break; case 1015: send(); selectKey(3, 5, PAUSE); selectKey(2, 2, PAUSE); selectKey(3, 3, PAUSE); end(); fin(); break; case 1016: send(); selectKey(3, 5, PAUSE); selectKey(2, 2, PAUSE); selectKey(4, 3, PAUSE); end(); fin(); break; case 1017: send(); selectKey(3, 5, PAUSE); selectKey(2, 2, PAUSE); selectKey(2, 4, PAUSE); end(); fin(); break; case 1018: send(); selectKey(3, 5, PAUSE); selectKey(2, 2, PAUSE); selectKey(3, 4, PAUSE); end(); fin(); break; case 1019: send(); selectKey(3, 5, PAUSE); selectKey(2, 2, PAUSE); selectKey(4, 4, PAUSE); end(); fin(); break; case 1020: send(); selectKey(3, 5, PAUSE); selectKey(3, 2, PAUSE); selectKey(3, 5, PAUSE); end(); fin(); break; case 1021: send(); selectKey(3, 5, PAUSE); selectKey(3, 2, PAUSE); selectKey(2, 2, PAUSE); end(); fin(); break; case 1022: send(); selectKey(3, 5, PAUSE); selectKey(3, 2, PAUSE); selectKey(3, 2, PAUSE); end(); fin(); break; case 1023: send(); selectKey(3, 5, PAUSE); selectKey(3, 2, PAUSE); selectKey(4, 2, PAUSE); end(); fin(); break; case 1024: send(); selectKey(3, 5, PAUSE); selectKey(3, 2, PAUSE); selectKey(2, 3, PAUSE); end(); fin(); break; case 1025: send(); selectKey(3, 5, PAUSE); selectKey(3, 2, PAUSE); selectKey(3, 3, PAUSE); end(); fin(); break; case 1026: send(); selectKey(3, 5, PAUSE); selectKey(3, 2, PAUSE); selectKey(4, 3, PAUSE); end(); fin(); break; case 1027: send(); selectKey(3, 5, PAUSE); selectKey(3, 2, PAUSE); selectKey(2, 4, PAUSE); end(); fin(); break; case 1028: send(); selectKey(3, 5, PAUSE); selectKey(3, 2, PAUSE); selectKey(3, 4, PAUSE); end(); fin(); break; case 1029: send(); selectKey(3, 5, PAUSE); selectKey(3, 2, PAUSE); selectKey(4, 4, PAUSE); end(); fin(); break; case 1030: send(); selectKey(3, 5, PAUSE); selectKey(4, 2, PAUSE); selectKey(3, 5, PAUSE); end(); fin(); break; case 1031: send(); selectKey(3, 5, PAUSE); selectKey(4, 2, PAUSE); selectKey(2, 2, PAUSE); end(); fin(); break; case 1032: send(); selectKey(3, 5, PAUSE); selectKey(4, 2, PAUSE); selectKey(3, 2, PAUSE); end(); fin(); break; case 1033: send(); selectKey(3, 5, PAUSE); selectKey(4, 2, PAUSE); selectKey(4, 2, PAUSE); end(); fin(); break; case 1034: send(); selectKey(3, 5, PAUSE); selectKey(4, 2, PAUSE); selectKey(2, 3, PAUSE); end(); fin(); break; case 1035: send(); selectKey(3, 5, PAUSE); selectKey(4, 2, PAUSE); selectKey(3, 3, PAUSE); end(); fin(); break; case 1036: send(); selectKey(3, 5, PAUSE); selectKey(4, 2, PAUSE); selectKey(4, 3, PAUSE); end(); fin(); break; case 1037: send(); selectKey(3, 5, PAUSE); selectKey(4, 2, PAUSE); selectKey(2, 4, PAUSE); end(); fin(); break; case 1038: send(); selectKey(3, 5, PAUSE); selectKey(4, 2, PAUSE); selectKey(3, 4, PAUSE); end(); fin(); break; case 1039: send(); selectKey(3, 5, PAUSE); selectKey(4, 2, PAUSE); selectKey(4, 4, PAUSE); end(); fin(); break; case 1040: send(); selectKey(3, 5, PAUSE); selectKey(2, 3, PAUSE); selectKey(3, 5, PAUSE); end(); fin(); break; case 1041: send(); selectKey(3, 5, PAUSE); selectKey(2, 3, PAUSE); selectKey(2, 2, PAUSE); end(); fin(); break; case 1042: send(); selectKey(3, 5, PAUSE); selectKey(2, 3, PAUSE); selectKey(3, 2, PAUSE); end(); fin(); break; case 1043: send(); selectKey(3, 5, PAUSE); selectKey(2, 3, PAUSE); selectKey(4, 2, PAUSE); end(); fin(); break; case 1044: send(); selectKey(3, 5, PAUSE); selectKey(2, 3, PAUSE); selectKey(2, 3, PAUSE); end(); fin(); break; case 1045: send(); selectKey(3, 5, PAUSE); selectKey(2, 3, PAUSE); selectKey(3, 3, PAUSE); end(); fin(); break; case 1046: send(); selectKey(3, 5, PAUSE); selectKey(2, 3, PAUSE); selectKey(4, 3, PAUSE); end(); fin(); break; case 1047: send(); selectKey(3, 5, PAUSE); selectKey(2, 3, PAUSE); selectKey(2, 4, PAUSE); end(); fin(); break; case 1048: send(); selectKey(3, 5, PAUSE); selectKey(2, 3, PAUSE); selectKey(3, 3, PAUSE); end(); fin(); break; case 1049: send(); selectKey(3, 5, PAUSE); selectKey(2, 3, PAUSE); selectKey(4, 3, PAUSE); end(); fin(); break; case 1050: send(); selectKey(3, 5, PAUSE); selectKey(3, 3, PAUSE); selectKey(3, 5, PAUSE); end(); fin(); break; case 1051: send(); selectKey(3, 5, PAUSE); selectKey(3, 3, PAUSE); selectKey(2, 2, PAUSE); end(); fin(); break; case 1052: send(); selectKey(3, 5, PAUSE); selectKey(3, 3, PAUSE); selectKey(3, 2, PAUSE); end(); fin(); break; case 1053: send(); selectKey(3, 5, PAUSE); selectKey(3, 3, PAUSE); selectKey(4, 2, PAUSE); end(); fin(); break; case 1054: send(); selectKey(3, 5, PAUSE); selectKey(3, 3, PAUSE); selectKey(2, 3, PAUSE); end(); fin(); break; case 1055: send(); selectKey(3, 5, PAUSE); selectKey(3, 3, PAUSE); selectKey(3, 3, PAUSE); end(); fin(); break; case 1056: send(); selectKey(3, 5, PAUSE); selectKey(3, 3, PAUSE); selectKey(4, 3, PAUSE); end(); fin(); break; case 1057: send(); selectKey(3, 5, PAUSE); selectKey(3, 3, PAUSE); selectKey(2, 4, PAUSE); end(); fin(); break; case 1058: send(); selectKey(3, 5, PAUSE); selectKey(3, 3, PAUSE); selectKey(3, 3, PAUSE); end(); fin(); break; case 1059: send(); selectKey(3, 5, PAUSE); selectKey(3, 3, PAUSE); selectKey(4, 3, PAUSE); end(); fin(); break; case 1060: send(); selectKey(3, 5, PAUSE); selectKey(4, 3, PAUSE); selectKey(3, 5, PAUSE); end(); fin(); break; case 1061: send(); selectKey(3, 5, PAUSE); selectKey(4, 3, PAUSE); selectKey(2, 2, PAUSE); end(); fin(); break; case 1062: send(); selectKey(3, 5, PAUSE); selectKey(4, 3, PAUSE); selectKey(3, 2, PAUSE); end(); fin(); break; case 1063: send(); selectKey(3, 5, PAUSE); selectKey(4, 3, PAUSE); selectKey(4, 2, PAUSE); end(); fin(); break; case 1064: send(); selectKey(3, 5, PAUSE); selectKey(4, 3, PAUSE); selectKey(2, 3, PAUSE); end(); fin(); break; case 1065: send(); selectKey(3, 5, PAUSE); selectKey(4, 3, PAUSE); selectKey(3, 3, PAUSE); end(); fin(); break; case 1066: send(); selectKey(3, 5, PAUSE); selectKey(4, 3, PAUSE); selectKey(4, 3, PAUSE); end(); fin(); break; case 1067: send(); selectKey(3, 5, PAUSE); selectKey(4, 3, PAUSE); selectKey(2, 4, PAUSE); end(); fin(); break; case 1068: send(); selectKey(3, 5, PAUSE); selectKey(4, 3, PAUSE); selectKey(3, 4, PAUSE); end(); fin(); break; case 1069: send(); selectKey(3, 5, PAUSE); selectKey(4, 3, PAUSE); selectKey(4, 4, PAUSE); end(); fin(); break; case 1070: send(); selectKey(3, 5, PAUSE); selectKey(2, 4, PAUSE); selectKey(3, 5, PAUSE); end(); fin(); break; case 1071: send(); selectKey(3, 5, PAUSE); selectKey(2, 4, PAUSE); selectKey(2, 2, PAUSE); end(); fin(); break; case 1072: send(); selectKey(3, 5, PAUSE); selectKey(2, 4, PAUSE); selectKey(3, 2, PAUSE); end(); fin(); break; case 1073: send(); selectKey(3, 5, PAUSE); selectKey(2, 4, PAUSE); selectKey(4, 2, PAUSE); end(); fin(); break; case 1074: send(); selectKey(3, 5, PAUSE); selectKey(2, 4, PAUSE); selectKey(2, 3, PAUSE); end(); fin(); break; case 1075: send(); selectKey(3, 5, PAUSE); selectKey(2, 4, PAUSE); selectKey(3, 3, PAUSE); end(); fin(); break; case 1076: send(); selectKey(3, 5, PAUSE); selectKey(2, 4, PAUSE); selectKey(4, 3, PAUSE); end(); fin(); break; case 1077: send(); selectKey(3, 5, PAUSE); selectKey(2, 4, PAUSE); selectKey(2, 4, PAUSE); end(); fin(); break; case 1078: send(); selectKey(3, 5, PAUSE); selectKey(2, 4, PAUSE); selectKey(3, 4, PAUSE); end(); fin(); break; case 1079: send(); selectKey(3, 5, PAUSE); selectKey(2, 4, PAUSE); selectKey(4, 4, PAUSE); end(); fin(); break; case 1080: send(); selectKey(3, 5, PAUSE); selectKey(3, 4, PAUSE); selectKey(3, 5, PAUSE); end(); fin(); break; case 1081: send(); selectKey(3, 5, PAUSE); selectKey(3, 4, PAUSE); selectKey(2, 2, PAUSE); end(); fin(); break; case 1082: send(); selectKey(3, 5, PAUSE); selectKey(3, 4, PAUSE); selectKey(3, 2, PAUSE); end(); fin(); break; case 1083: send(); selectKey(3, 5, PAUSE); selectKey(3, 4, PAUSE); selectKey(4, 2, PAUSE); end(); fin(); break; case 1084: send(); selectKey(3, 5, PAUSE); selectKey(3, 4, PAUSE); selectKey(2, 3, PAUSE); end(); fin(); break; case 1085: send(); selectKey(3, 5, PAUSE); selectKey(3, 4, PAUSE); selectKey(3, 3, PAUSE); end(); fin(); break; case 1086: send(); selectKey(3, 5, PAUSE); selectKey(3, 4, PAUSE); selectKey(4, 3, PAUSE); end(); fin(); break; case 1087: send(); selectKey(3, 5, PAUSE); selectKey(3, 4, PAUSE); selectKey(2, 4, PAUSE); end(); fin(); break; case 1088: send(); selectKey(3, 5, PAUSE); selectKey(3, 4, PAUSE); selectKey(3, 4, PAUSE); end(); fin(); break; case 1089: send(); selectKey(3, 5, PAUSE); selectKey(3, 4, PAUSE); selectKey(4, 4, PAUSE); end(); fin(); break; case 1090: send(); selectKey(3, 5, PAUSE); selectKey(4, 4, PAUSE); selectKey(3, 5, PAUSE); end(); fin(); break; case 1091: send(); selectKey(3, 5, PAUSE); selectKey(4, 4, PAUSE); selectKey(2, 2, PAUSE); end(); fin(); break; case 1092: send(); selectKey(3, 5, PAUSE); selectKey(4, 4, PAUSE); selectKey(3, 2, PAUSE); end(); fin(); break; case 1093: send(); selectKey(3, 5, PAUSE); selectKey(4, 4, PAUSE); selectKey(4, 2, PAUSE); end(); fin(); break; case 1094: send(); selectKey(3, 5, PAUSE); selectKey(4, 4, PAUSE); selectKey(2, 3, PAUSE); end(); fin(); break; case 1095: send(); selectKey(3, 5, PAUSE); selectKey(4, 4, PAUSE); selectKey(3, 3, PAUSE); end(); fin(); break; case 1096: send(); selectKey(3, 5, PAUSE); selectKey(4, 4, PAUSE); selectKey(4, 3, PAUSE); end(); fin(); break; case 1097: send(); selectKey(3, 5, PAUSE); selectKey(4, 4, PAUSE); selectKey(2, 4, PAUSE); end(); fin(); break; case 1098: send(); selectKey(3, 5, PAUSE); selectKey(4, 4, PAUSE); selectKey(3, 4, PAUSE); end(); fin(); break; case 1099: send(); selectKey(3, 5, PAUSE); selectKey(4, 4, PAUSE); selectKey(4, 4, PAUSE); end(); fin(); break; case 1100: send(); selectKey(2, 2, PAUSE); selectKey(3, 5, PAUSE); selectKey(3, 5, PAUSE); end(); fin(); break; } } if (b > 0&&buf == '#') { LCD_Clear(); LCD_Goto(0, 4); LCD_SendString("Ochistka"); _delay_ms(2000); LCD_Clear(); b = 0; k = 0; for (i = 0; i < 4; i++) { buttons[i] = 0; } } } } return 0; }

ну хотя бы send(); end(); fin(); выкинуть из каждого case. И какой смысл передавать одну и ту же PAUSE в selectKey?
но без алгоритма такую портянку оптимизировать - только за деньги.
Но думаю, что cont явно просится передаться в selectKey без всяких switch-case...

Martian писал(а):

...но без алгоритма такую портянку оптимизировать - только за деньги.
..

В этом коде при наборе комбинации от 1001 до 1100 происходит последовательное включение светодиодов в определенном порядке. С начало идёт включение по функции send(); затем набранная комбинация после чего команды из функции end(); после чего вывод на ЖК дисплей через функцию fin(); хотя думаю end(); и fin(); можно было объединить в одну.

эта комбинация как формируется? могу я изменить правило формирования, например, не 1001...1100, а 1002...1202, то есть, те же сто, только все чётные? или эти числа где-то ещё используются - отображаются?

Martian писал(а):

эта комбинация как формируется?...

Прикрепил схему.
Функция send(); поочерёдно включает комбинацию светодиодов RES-PRI-TOT-CALL-0-OK-1.
Затем поочередно включает последовательность набранной комбинации с клавиатуры. Оно может быть от 1001, 1002, 1003 и тд. до 1100.
К примеру, комбинация 1012 будет выглядеть так:

Код:: selectKey(3, 5, PAUSE); // 0 selectKey(2, 2, PAUSE); //1 selectKey(3, 2, PAUSE); //2

Начальная единица уже прописана внутри функции send(); и в коде проверяется на её присутствие, если начальный символ другой то код выдаст ошибку и остановиться.

1100 конечная максимальная комбинация, далее нельзя.
Затем срабатывает комбинация из функции end(); При этом поочерёдно включаются и выключаются светодиоды PRI и RES.

Подправил функцию

Код:: void selectKey(int r, int c) { PORTD = col[r]; PORTC = row[c]; _delay_ms(PAUSE); PORTC = row[0]; PORTD = col[0]; _delay_ms(PAUSE); }

и

Код:: void end(void) { selectKey(1, 4); // PRI selectKey(1, 5); // RES _delay_ms(2); LCD_Clear(); LCD_Goto(0, 0); LCD_SendString("Kod otpravlen! "); _delay_ms(1000); LCD_Clear(); b = 0; k = 0; for (i = 0; i < 4; i++) { buttons[i] = 0; } cont = 0; }

Теперь switch выглядит так:

Код:: case 1001: send(); selectKey(3, 5); selectKey(3, 5); selectKey(2, 2); end(); break;

ЗЫ: PAUSE устанавливает время через которое светодиоды должны отключиться.

Массивы можно выбросить.
из аргументов selectKey убрать PAUSE - она нигде не отличается
тогда получится:

Код:: void selectKey(unsigned char r, unsigned char c) // чтобы не вычитать единицу, номера столбов-строк должны начинаться с 0 { PORTD = (unsigned char)(1 << r); PORTC = (0x1f & (unsigned char)(~(1 << c))); _delay_ms(PAUSE); PORTC = 0x1f; PORTD = 0; _delay_ms(PAUSE); }

ну и это:

Код:: switch(cont) { case 1001: send(); selectKey(3, 5, PAUSE); selectKey(3, 5, PAUSE); selectKey(3, 4, PAUSE); end(); fin(); break;

переписать как

Код:: send(); switch(cont) { case 1001: selectKey(2, 4); selectKey(2, 4); selectKey(2, 3); break; } end(); fin();

299 строк минус...

Добавлено after 12 minutes 44 seconds:
но если я правильно понял алгоритм, то весь этот громадный switch сокращается до перебора буфера и отправки каждой цифры:

Код:: send(); for (i = 0; i < 4; i++) { switch(buttons[i] - '0') { case 0: selectKey(3, 5, PAUSE); break; case 1: selectKey(3, 5, PAUSE); break; case 2: selectKey(3, 5, PAUSE); break; case 3: selectKey(3, 5, PAUSE); break; case 4: selectKey(3, 5, PAUSE); break; case 5: selectKey(3, 5, PAUSE); break; case 6: selectKey(3, 5, PAUSE); break; case 7: selectKey(3, 5, PAUSE); break; case 8: selectKey(3, 5, PAUSE); break; case 9: selectKey(3, 5, PAUSE); break; } end(); fin();

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

Код:: send(); for (i = 0; i < 4; i++) { selectKey(arr1[buttons[i] - '0'], arr2[buttons[i] - '0'], PAUSE); } end(); fin();

в общем, есть куда ещё подумать.

Martian писал(а):

если я правильно понял алгоритм, то весь это громадный switch сокращается

Почти да, но в Вашем коде комбинация от 0 до 9. А в моём от 1001 до 1100.
Спасибо за подсказки и советы.

ЗЫ: а вот над этим

Код:: { selectKey(arr1[buttons[i] - '0'], arr2[buttons[i] - '0'], PAUSE); }

мне надо поэкспериментировать )

Последний раз редактировалось grachevvlad2023 Вс ноя 05, 2023 22:05:25, всего редактировалось 1 раз.

Вы перебрали все 100 вариантов для комбинаций буфера
я же перебрал 10 вариантов для каждой ячейки в буфере.
если буфер станет 100 ячеек, в моём варианте достаточно изменить число итераций в цикле. а в Вашем? дописать ещё десяток тыщ строк...

Бы попробовал заменить "длинный" switch(count) на напр. 6 массивов возможных состояний. Написанно так будет "короче" как код (будет произведена только замена с данными (один раз в осн. цикле), заданными индексом массива).

(Не гарантирую, что будет занимать меньше ресурса (памяти). Должно быть проверено).

Пример:

Код:: const uint8_t r0[] = {3, 3, 3 ... }; const uint8_t c0[] = {5, 5, 5 ... }; const uint8_t r1[] = {3, 3, 3 ... }; const uint8_t c1[] = {5, 5, 5 ... }; const uint8_t r2[] = {2, 3, 4 ... }; const uint8_t c2[] = {2, 2, 2 ... }; ... uint8_t x = count - 1001; send(); selectKey(r0[x], c0[x]); selectKey(r1[x], c1[x]); selectKey(r2[x], c2[x]); end(); fin();

(Если выбранный компилятор сам не помещает массивы во flash (const), добавьте необходимые функции/команды для этого).

по мне так самая очевидная оптимизация
switch(cont)
это просто использовать массив из 2-3-4 байтных константных слов, упаковать можно множеством способов, хоть побитно, хоть используя 2й индекс , после получения указателя - разбирать их и грузить куда нужно.
исходный это какойто кошмар с которым никакой O2 не справится, это и по размеру кода в разы и по нагрузке на ядро вдвое неэффективно. на научном языке такое называется классический ГК :)))

Добавлено after 18 minutes 16 seconds:
забавно veso74 почти тожесамое написал, не даром говорят у программистов мысли совпадают

1. В оригинальной функции selectKey два раза используется функция _delay_ms, причем при первом вызове всегда передается константа PAUSE, а при втором вызове - аргумент delay. С учетом того, что далее по коду в функцию selectKey в качестве аргумента delay всегда передается константа PAUSE, то это выглядит крайне странно. Если вам нужно выдерживать 2 разных по длительности интервала, то используйте и 2 разных константы (или макросы, как у вас). Иначе это выглядит как опечатка, и оба вызова _delay_ms должны быть с аргументом delay. Как заметил Martian, можно просто убрать аргумент delay и сразу прописать PAUSE, т.к. как я понял этот интервал зависит от аппаратной реализации вашей схемы, и не меняется в runtime.

2. При вводе числового кода, у вас есть проверка на то, что первые 2 цифры должны быть "10" или "11". Проверка разделена на 2 условия, и в обоих условиях продублирован практически один и тот же код. Можно объединить в одну проверку:

Спойлер

Код:: // Пропускаем ввод, если первый адрес массива не равен единице char isKeycodeInvalid_First = (b == 0 && buf != '1'); // Пропускаем ввод, если второй адрес массива не равен нулю или единице char isKeycodeInvalid_Second = (b == 1 && (buf != '0' && buf != '1')); if (isKeycodeInvalid_First || isKeycodeInvalid_Second) { if (isKeycodeInvalid_Second) LCD_Clear(); LCD_Goto(0, 0); LCD_SendString("Oshibka"); _delay_ms(2000); LCD_Clear(); b = 0; k = 0; for (i = 0; i < 4; i++) buttons[i] = 0; continue; }

Кроме того, часть алгоритма (очистка переменных b, k и buttons, очистка LCD) также продублирована в методе fin. Это тоже можно оптимизировать, либо а) вынести общую часть алгоритма очитки в отдельную функцию, и вызывать эту функцию из функции fin и из условия проверки первых двух цифр, либо б) сделать одну общую функцию "завершения ввода комбинации", в которую через аргументы передавать причину очистки (строки "Код отправлен!", "Ошибка", "Очистка" через аргумент типа const char*).

3. Как уже отметили veso74 и AlexS4, можно использовать массив значений для функции selectKey. Я бы использовал массив структур, т.к. не уловил закономерности (зависимость вводимого кода 1001-1100 от порядка и номеров установки Row и Col при вызове selectKey):

Спойлер

Код:: typedef struct { char Row; char Col; } TSelectedKey; typedef struct { TSelectedKey Key[3]; } TSelectedKeys; // все возможные комбинации const TSelectedKeys SelectedKeys[100] = { { {3, 5}, {3, 5}, {2, 2} }, // 1001 { {3, 5}, {3, 5}, {3, 2} }, // 1002 { {3, 5}, {3, 5}, {4, 2} }, // 1003 { {3, 5}, {3, 5}, {2, 3} }, // 1004 { {3, 5}, {3, 5}, {3, 3} }, // 1005 ... // и т.д. }; ... // работаем с оригинальной глобальной переменной 'cont', в которой уже посчитано введённое значение { int index = cont - (1001); // т.к. массивы индексируются с нуля const TSelectedKey* key = SelectedKeys[index].Key; // далее из оригинала: send(); selectKey(key[0].Row, key[0].Col, PAUSE); selectKey(key[1].Row, key[1].Col, PAUSE); selectKey(key[2].Row, key[2].Col, PAUSE); end(); fin(); }

Однако тут есть важное замечание: для работы с массивами вам необходимо организовать проверку на выход за границы этого самого массива (при использовании "динамического" индекса)!
Например, в оригинальном коде не было проверки на вводимые числа 1101-1199, есть только отдельные проверки на *0**-*1** и **00-**99, что не эквивалентно проверке *001-*100. При этом в оригинальном коде использовался switch-case, и все неописанные комбинации не выполнялись (не изменялось состояние портов). Т.е. при нажатии 'звездочки' при неправильном числовом коде ничего не происходило: ни пины не дергались, ни дисплей не реагировал, но можно было это стереть через 'решетку'.
При использовании же массивов всегда будет что-то выполняться, и если входные данные некорректные, то программа просто поломается (лечится перезагрузкой МК).

P.S. Пишите понятные имена объектов. Функции вы нормально назвали (по имени ясно, что она делает), а вот переменные обделили (b, k, ...)

Да не надо там такой громадный массив. Там приблизительно такая логика: есть некий буфер, размером 4. Его цифровое десятичное содержимое (или часть) надо последовательно высветить на экране. Вот и всё. Так как каждое значение имеет лишь десять вариантов, то просо необходим цикл с числом итераций не больше размера этого буфера, где либо проверяется на совпадение одному из 10 состояний, либо адресуется к массиву[10]

Используя Ваши подсказки и советы, за что Вам премного благодарен, немного переписал код и теперь он состоит только из 651 строк против начальных ~2900.:

Спойлер

Код:: #define F_CPU 32000000UL // Определение тактовой частоты для функций задержки #include <inttypes.h> // Библиотека для работы с типами int #include <avr/io.h> // Библиотека для ввода-вывода #include <avr/interrupt.h> // Библиотека для работы с прерываниями #include <avr/sleep.h> // Библиотека для управления режимами сна #include <util/delay.h> // Библиотека для работы с задержками #include "keyboard.h" // Библиотека для работы с клавиатурой #include "lcd_lib.h" // Библиотека для работы с LCD #define PAUSE 150 // Определение времени задержки int k = 0; int b = 0; int i = 0; int c = 0; int cont = 0; // Определение столбцов и строк int col[5] = {0b00000000, 0b00000001, 0b00000010, 0b00000100, 0b00001000}; int row[6] = {0b00011111, 0b00011110, 0b00011101, 0b00011011, 0b00010111, 0b00001111}; unsigned char buf = 0; unsigned char buttons[4]; // Функция выбора ключа void selectKey(int r, int c) { PORTD = col[r]; // Выбор столбца PORTC = row[c]; // Выбор строки _delay_ms(PAUSE); // Ждем PORTC = row[0]; // Выбираем первую строку PORTD = col[0]; // Выбираем первый столбец _delay_ms(PAUSE); // Ждем } // Функция отправки void send(void) { selectKey(1, 5); // RES selectKey(1, 4); // PRI selectKey(1, 1); // TOT selectKey(1, 2); // CAL int i; for ( i = 0; i < 4; i++) { selectKey(3, 5); // 0 } selectKey(4, 5); // OK selectKey(2, 2); // 1 } // Функция завершения void end(void) { selectKey(1, 4); // PRI selectKey(1, 5); // RES _delay_ms(2); // задержка LCD_Clear(); // очищаем экран LCD_Goto(0, 0); // выбираем позицию на экране LCD_SendString("Kod otpravlen! "); // выводим сообщение _delay_ms(1000); // задержка LCD_Clear(); // очищаем экран b = 0; k = 0; for (i = 0; i < 4; i++) // обнуляем массив { buttons[i] = 0; } cont = 0; // обнуляем вход } int main(void) { // Установить все выводы порта D и C как выводы DDRD=0xFF; DDRC=0xFF; // Инициализировать LCD, установить курсор в начало и отобразить сообщение. LCD_Init(); LCD_Goto(0, 0); LCD_SendString("Vvedite "); LCD_Goto(4, 1); LCD_SendString("Koefitsent: "); _delay_ms(2000); // задержка LCD_Clear(); // очищаем экран KEYB_Init(); // инициализируем клавиатуру //начинаем основной цикл while (1) { // сканируем клавиатуру и получаем значение введенной клавиши KEYB_ScanKeyboard(); buf = KEYB_GetKey(); // если была нажата клавиша if (buf) { // условие ввода символов и исключения некоторых клавиш if (b < 4 && buf != '*'&& buf != '#') { // проверяем правильность кода char isKeycodeInvalid_First = (b == 0 && buf != '1'); char isKeycodeInvalid_Second = (b == 1 && (buf != '0' && buf != '1')); if (isKeycodeInvalid_First || isKeycodeInvalid_Second) { if (isKeycodeInvalid_Second) LCD_Clear(); LCD_Goto(0, 0); LCD_SendString("Oshibka"); _delay_ms(2000); LCD_Clear(); b = 0; k = 0; for (i = 0; i < 4; i++) buttons[i] = 0; continue; } buttons[b++] = buf; LCD_Goto(k++, 1); LCD_WriteData(buf); } // по нажатию '*' выполняем обработку и отправку введенных данных if (buf == '*') { for (c = 0; c < 4; c++) { if (buttons[c] >= '0' && buttons[c] <= '9') { int digit = buttons[c] - '0'; cont = cont * 10 + digit; } } send(); switch(cont) // здесь указана привязка введенных значений к функциям (это нужно прокомментировать подробнее) { case 1000: selectKey(3, 5); selectKey(3, 5); selectKey(3, 5); break; // и так далее... для каждого действия } end(); } // Клавиша '#' очищает все введенные значения if (b > 0&&buf == '#') { LCD_Clear(); LCD_Goto(0, 4); LCD_SendString("Ochistka"); _delay_ms(2000); LCD_Clear(); b = 0; k = 0; for (i = 0; i < 4; i++) { buttons[i] = 0; } } } } return 0; }

что-то странное и очень много.

Рискну утверждать, что ниже приведенный код из 154 строк эквивалентен начальному на 99%

Код:: #define F_CPU 32000000UL #include <inttypes.h> #include <avr/io.h> #include <avr/interrupt.h> #include <avr/sleep.h> #include <util/delay.h> #include "keyboard.h" #include "lcd_lib.h" #define PAUSE 150 unsigned char k = 0; unsigned char b = 0; unsigned char i = 0; unsigned char buf = 0; unsigned char col[5] = {0b00000000, 0b00000001, 0b00000010, 0b00000100, 0b00001000}; unsigned char row[6] = {0b00011111, 0b00011110, 0b00011101, 0b00011011, 0b00010111, 0b00001111}; unsigned char buttons[4]; void selectKey(unsigned char r, unsigned char c) { PORTD = col[r]; PORTC = row[c]; _delay_ms(PAUSE); PORTC = row[0]; PORTD = col[0]; _delay_ms(PAUSE); } void clear(void) { LCD_Clear(); b = 0; k = 0; for (i = 0; i < 4; i++) { buttons[i] = 0; } } void send(void) { selectKey(1, 5); // RES selectKey(1, 4); // PRI selectKey(1, 1); // TOT selectKey(1, 2); // CAL for (i = 0; i < 4; i++) { selectKey(3, 5); // 0 } selectKey(4, 5); // OK selectKey(2, 2); // 1 } void end(void) { selectKey(1, 4); // PRI selectKey(1, 5); // RES LCD_Clear(); LCD_Goto(0, 0); LCD_SendString("Kod otpravlen! "); _delay_ms(1000); clear(); } int main(void) { DDRD=0xFF; DDRC=0xFF; LCD_Init(); LCD_Goto(0, 0); LCD_SendString("Vvedite "); LCD_Goto(4, 1); LCD_SendString("Koefitsent: "); _delay_ms(2000); LCD_Clear(); KEYB_Init(); while (1) { KEYB_ScanKeyboard(); buf = KEYB_GetKey(); if (buf) { if (b < 4 && buf != '*'&& buf != '#') { // Проверяем, есть ли еще место в массиве и выполняем ограничения на ввод if ((b == 0 && buf != '1') || (b == 1 && (buf != '0' && buf != '1'))) { LCD_Goto(0, 0); LCD_SendString("Oshibka"); _delay_ms(2000); clear(); continue; // Пропускаем ввод, если первый адрес массива не равен единице и если если второй адрес массива не равен нулю или единице } buttons[b++] = buf; LCD_Goto(k++, 1); LCD_WriteData(buf); } if ((buf == '*') && (buttons[0] = '1')) { send(); for (i = 1; i < 4; i++) { switch(buttons[i]) { case '0': selectKey(3, 5); break; case '1': selectKey(2, 2); break; case '2': selectKey(3, 2); break; case '3': selectKey(4, 2); break; case '4': selectKey(2, 3); break; case '5': selectKey(3, 3); break; case '6': selectKey(4, 3); break; case '7': selectKey(2, 4); break; case '8': selectKey(3, 4); break; case '9': selectKey(4, 4); break; } end(); } if (b > 0&&buf == '#') { LCD_Clear(); LCD_Goto(0, 4); LCD_SendString("Ochistka"); _delay_ms(2000); LCD_Clear(); clear(); } } } }

Добавлено after 5 minutes 26 seconds:
и думаю, что можно ещё оптимизировать. Особенно в плане проверки этого:

Код:: if (buf) { if (b < 4 && buf != '*'&& buf != '#') { // Проверяем, есть ли еще место в массиве и выполняем ограничения на ввод if ((b == 0 && buf != '1') || (b == 1 && (buf != '0' && buf != '1')))

но мне неохота разбираться, верно ли сиё алгоритму. Но чувствуется некое противоречие

Martian писал(а):

что-то странное и очень много.
Рискну утверждать, что ниже приведенный код из 154 строк эквивалентен начальному на 99%

Проверил Ваш код. При вводе 1012, или 1058 отрабатывает 1 и 0, при вводе 1100 отрабатывает 1 и 1. Два конечных символов не отрабатываются в switch.

ну, это и есть 1% - я ж досконально не знаю алгоритма. Но он всё же отрабатывает, значит, осталось откорректировать в каком порядке и сколько должно браться, и с каким элементом в буфере сравнивается '1'

то есть, всего лишь это:

Код:: if ((buf == '*') && (buttons[0] = '1')) { send(); for (i = 1; i < 4; i++)

и условие заполнения буфера тоже проверить:

Код:: if (b < 4 && buf != '*'&& buf != '#') { // Проверяем, есть ли еще место в массиве и выполняем ограничения на ввод if ((b == 0 && buf != '1') || (b == 1 && (buf != '0' && buf != '1'))) {

Добавлено after 24 minutes 47 seconds:
Поскольку в некоторых функциях уже нет нужды (send и end), можно их исключить, и оптимизировать вывод на дисплей. Итого 134 строк.

Код:: #define F_CPU 32000000UL #include <inttypes.h> #include <avr/io.h> #include <avr/interrupt.h> #include <avr/sleep.h> #include <util/delay.h> #include "keyboard.h" #include "lcd_lib.h" #define PAUSE 150 unsigned char k = 0; unsigned char b = 0; unsigned char i = 0; unsigned char buf = 0; unsigned char col[5] = {0b00000000, 0b00000001, 0b00000010, 0b00000100, 0b00001000}; unsigned char row[6] = {0b00011111, 0b00011110, 0b00011101, 0b00011011, 0b00010111, 0b00001111}; unsigned char buttons[4]; void selectKey(unsigned char r, unsigned char c) { PORTD = col[r]; PORTC = row[c]; _delay_ms(PAUSE); PORTC = row[0]; PORTD = col[0]; _delay_ms(PAUSE); } void Message(string message, unsigned short delay) { LCD_Clear(); LCD_Goto(0, 0); LCD_SendString(message); _delay_ms(delay); LCD_Clear(); b = 0; k = 0; for (i = 0; i < 4; i++) { buttons[i] = 0; } } int main(void) { DDRD=0xFF; DDRC=0xFF; LCD_Init(); LCD_Goto(0, 0); LCD_SendString("Vvedite "); LCD_Goto(4, 1); LCD_SendString("Koefitsent: "); _delay_ms(2000); LCD_Clear(); KEYB_Init(); while (1) { KEYB_ScanKeyboard(); buf = KEYB_GetKey(); if (buf) { if ((b < 4) && (buf != '*') && (buf != '#')) { // Проверяем, есть ли еще место в массиве и выполняем ограничения на ввод if (((b == 0) && (buf != '1')) || ((b == 1) && ((buf != '0') && (buf != '1')))) { Message("Oshibka", 2000); continue; // Пропускаем ввод, если первый адрес массива не равен единице } buttons[b++] = buf; LCD_Goto(k++, 1); LCD_WriteData(buf); } if (buf == '*') { selectKey(1, 5); // RES selectKey(1, 4); // PRI selectKey(1, 1); // TOT selectKey(1, 2); // CAL for (i = 0; i < 4; i++) { selectKey(3, 5); // 0 } selectKey(4, 5); // OK for (i = 0; i < 4; i++) { switch(buttons[i]) { case '0': selectKey(3, 5); break; case '1': selectKey(2, 2); break; case '2': selectKey(3, 2); break; case '3': selectKey(4, 2); break; case '4': selectKey(2, 3); break; case '5': selectKey(3, 3); break; case '6': selectKey(4, 3); break; case '7': selectKey(2, 4); break; case '8': selectKey(3, 4); break; case '9': selectKey(4, 4); break; } selectKey(1, 4); // PRI selectKey(1, 5); // RES Message("Kod otpravlen!", 1000); } if ((b > 0) && (buf == '#')) { Message("Ochistka", 2000); } } } }

Martian писал(а):

ну, это и есть 1% - я ж досконально не знаю алгоритма...

Как отправить Вм видео файл работы устройства? Вы бы смогли понять алгоритм..

А зачем? Вам осталось совсем немного, код получился простой, довести до ума совершенно не сложно. Да и не люблю алгоритмы на видео, я люблю их в виде блок-схем или хотя бы подробного текстового описания....

Прикрепил видео работы устройства.

Спойлер

Здесь функция send(); отрабатывает до однократного мигания светодиода под лейблом "1" после того как отработает "#", затем идут три символа из конструкции switch, после чего работает функция end();
Так же прикрепил немного оптимизированный код.

Последний раз редактировалось grachevvlad2023 Ср ноя 08, 2023 11:06:36, всего редактировалось 2 раз(а).

grachevvlad2023 писал(а):Так же прикрепил немного оптимизированный код.

Поглядел. Осознал, что я зря потратил время...

Оптимизация кода на С, возможно ли для этого кода?

Оптимизация кода на С, возможно ли для этого кода?

Re: Оптимизация кода на С, возможно ли для этого кода?

Re: Оптимизация кода на С, возможно ли для этого кода?

Re: Оптимизация кода на С, возможно ли для этого кода?

Re: Оптимизация кода на С, возможно ли для этого кода?

Re: Оптимизация кода на С, возможно ли для этого кода?

Re: Оптимизация кода на С, возможно ли для этого кода?

Re: Оптимизация кода на С, возможно ли для этого кода?

Re: Оптимизация кода на С, возможно ли для этого кода?

Re: Оптимизация кода на С, возможно ли для этого кода?

Re: Оптимизация кода на С, возможно ли для этого кода?

Re: Оптимизация кода на С, возможно ли для этого кода?

Re: Оптимизация кода на С, возможно ли для этого кода?

Re: Оптимизация кода на С, возможно ли для этого кода?

Re: Оптимизация кода на С, возможно ли для этого кода?

Re: Оптимизация кода на С, возможно ли для этого кода?

Re: Оптимизация кода на С, возможно ли для этого кода?

Re: Оптимизация кода на С, возможно ли для этого кода?

Re: Оптимизация кода на С, возможно ли для этого кода?

Re: Оптимизация кода на С, возможно ли для этого кода?