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

[grachevvlad2023]

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

Спойлер

Код: Выделить всё

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

[Martian]

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

[grachevvlad2023]

[uquote="Martian",url="/forum/viewtopic.php?p=4498426#p4498426"]...но без алгоритма такую портянку оптимизировать - только за деньги.
..[/uquote]

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

[Martian]

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

[grachevvlad2023]

[uquote="Martian",url="/forum/viewtopic.php?p=4498440#p4498440"]эта комбинация как формируется?...[/uquote]
Прикрепил схему.
Функция 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 устанавливает время через которое светодиоды должны отключиться.

[Martian]

Массивы можно выбросить.
из аргументов 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();

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

[grachevvlad2023]

[uquote="Martian",url="/forum/viewtopic.php?p=4498458#p4498458"]если я правильно понял алгоритм, то весь это громадный switch сокращается[/uquote]
Почти да, но в Вашем коде комбинация от 0 до 9. А в моём от 1001 до 1100.
Спасибо за подсказки и советы.

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

Код: Выделить всё

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

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

[Martian]

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

[veso74]

Бы попробовал заменить "длинный" 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), добавьте необходимые функции/команды для этого).

[AlexS4]

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

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

[Azeront]

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, ...)

[Martian]

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

[grachevvlad2023]

Используя Ваши подсказки и советы, за что Вам премного благодарен, немного переписал код и теперь он состоит только из 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;
} 

[Martian]

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

Рискну утверждать, что ниже приведенный код из 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')))

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

[grachevvlad2023]

[uquote="Martian",url="/forum/viewtopic.php?p=4499315#p4499315"]что-то странное и очень много.
Рискну утверждать, что ниже приведенный код из 154 строк эквивалентен начальному на 99%[/uquote]
Проверил Ваш код. При вводе 1012, или 1058 отрабатывает 1 и 0, при вводе 1100 отрабатывает 1 и 1. Два конечных символов не отрабатываются в switch.

[Martian]

ну, это и есть 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); 
            }
        }
    }
}

[grachevvlad2023]

[uquote="Martian",url="/forum/viewtopic.php?p=4499336#p4499336"]ну, это и есть 1% - я ж досконально не знаю алгоритма...[/uquote]
Как отправить Вм видео файл работы устройства? Вы бы смогли понять алгоритм..

[Martian]

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

[grachevvlad2023]

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

Спойлер

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

[Martian]

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

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