Работа с командами GSM под stm32f4 (массивы)

[supercelt]

Всем привет) Укажите пожалуйста на ошибки. Код работает, но может что-то я делаю слишком не оптимизированно.
Суть в следующем:

Есть массив

char gsm_cusd_command[30] = "AT+CUSD=1,,15\r";
char *gsm_point_cusd_command = gsm_cusd_command;

Есть массив, который заполняется тем, что напечатают на матричной клаве

char gsm_ussd_balance_command[16] = "";
char *gsm_point_ussd_balance_command = gsm_ussd_balance_command;

когда напечатали команду проверки баланса, в массиве будет "*100#"

Далее по нажатию определённой кнопки, надо всё что в массиве поместить в исходный массив между запятыми.
Сначала я писал под конкретную задачу, но потом подумал, а вдруг мне надо будет где-нибудь ещё это сделать
с другим массивом. И решил универсалить ф-ию.
Вобщем в ф-ию передаются параметры: указатель на массив в который дописывать, размер этого массива, указатель на массив
в котором хранится напечатанная команда, 2 символа между которыми надо всё вырезать если что-то было и вставить новое.
То есть если там раньше было ,*2005#, к примеру, то это вырезаем и вставляем новое.

system_cut_and_insert_array(gsm_point_cusd_command  , 30, gsm_point_ussd_balance_command, ',', ',');

Алгоритм такой:
Сначала надо посчитать не превышает ли размер массива (30) новый массив после вставки новой строки.
Поэтому я прогоняю массив, слежу за символами и считаю. После этого проверяю, если превышает, то return
Далее всё что после 2 символа надо потом дописать в хвост. Поэтому я это копирую во временный массив tmp
Далее я стираю всё что после первого символа.
Далее я после первого символа вставляю новую подстроку.
А далее дописываю в хвост тот кусочек ,15\r

void system_cut_and_insert_array(char *source, const uint8_t size, char *insert, char s1, char s2){ 
	uint8_t count = 0, c = 0;
	char *point_insert = 0, *point_delete = 0;
	char *s = source;
	char tmp[size];
	char *t = tmp;
	while(*s){
		if(c){
			if(*s == s2) break;				
			count++;
		}
		if(*s == s1 && !c){
			c = 1;
			point_insert = s + 1;
			point_delete = point_insert;
		}
		s++;
	}
	if((strlen(source) - count + strlen(insert)) › size) return;
	while(*s){
		*t = *s;
		t++;
		s++;
	}
	while(*point_delete){
		*point_delete = 0;
		point_delete++;
	}
	while(*insert){
		*point_insert = *insert;
		point_insert++;
		insert++;
	}
	t = tmp;
	while(*t){
		*point_insert = *t;
		point_insert++;
		t++;
	}
}

Скажите? Это конечно сделано по прямолинейной логике. Но может есть какие-нибудь программерские финты, что бы делать это не так размажисто?

[akegor]

На мой взгляд, проще иметь строковые константы начала и конца строки, между которыми вставляется переменная часть. Выходная строка в этом случае формируется в буфере простой конкатенацией констант и переменной части.
Этот подход более универсальный, чем Ваш. Особенно, если учесть, что есть достаточно много АТ команд, в которых возможно несколько пустых полей, разделенных запятыми.

[mike-y-k]

А передача последовательно префикса, команды и суффикса не будет сильно проще?
Три строки в подпрограмму…
Если очень нужно в одном поле, то sprintf 'префикс%sсуффикс'.
Распределение буфера можно и динамически - главное не забыть его освободить после использования .

[Easyrider83]

Вся эта белиберда заменяется одной командой sprintf. А для буферов лучше использовать динамическое выделение памяти. Будет более гибко.
И команду strcpy почитайте из stdlib. Зачем городить огород, когда он уже есть?

[Bill]

Вся эта белиберда заменяется одной командой sprintf. А для буферов лучше использовать динамическое выделение памяти. Будет более гибко.
И команду strcpy почитайте из stdlib. Зачем городить огород, когда он уже есть?

Вообще-то, есть стандартные библиотечные функции для работы со строкам: strlen, strstr, strcat, strcpy... Или нет?

[dgrishin]

Если исходить из концепции "чем проще - тем лучше" я бы сделал так...

char gsm_cusd_command[30] = "AT+CUSD=1,";

char* pGsm_point_ussd_balance_command = &char gsm_cusd_command[10];

// то есть место в памяти, где располагается '\0' строки "AT+CUSD=1," я присвоил
//новой переменной, куда будет ввод данных.
// а после окончания ввода в конец надо дописать ",15\r"
// такой алгоритм требует меньше манипуляций со строками

[supercelt]

Спасибо за ответы! Захотел проверить ещё одну идею,
Вот я создаю массив указателей

char *gsm_cusd_command[2] = {"AT+CUSD=1,",",15\r"};

Получается, что указатель gsm_cusd_command[0] будет указывать на строку AT+CUSD=1,
а gsm_cusd_command[1] на ,15\r
Как передать всё это в функцию одним параметром?
Если так: my_func(gsm_cusd_command){} на работает

void my_func(char *source){}

User\src\gsm.c(149): error: #167: argument of type "char **" is incompatible with parameter of type "char *"

Или только каждый передавать

my_func(gsm_cusd_command[0], gsm_cusd_command[1]){}

UPD

Сделал указатель на указатель

char *gsm_cusd_part_command[2] = {"AT+CUSD=1,",",15\r"};
char **gsm_point_cusd_command = gsm_cusd_part_command;

Для доступа к значениям делаю так:

void gsm_merge_command(char **source){
char s
s = *source[0]; - это например доступ к значению первого указателя и его значение 'A'
s = *source[1]; - это например доступ к значению второго указателя и его значение ','

Но почему существуют ещё какие-то лишние элементы?
*source[2] будет '@'
*source[3] будет 'e'

Хотя их же нет. Мне просто надо каким-то образом из char **source в ф-ии сосчитать сколько там массивов с указателями. В данном случае должно получиться 2.

[dgrishin]

Но почему существуют ещё какие-то лишние элементы?
*source[2] будет '@'
*source[3] будет 'e'

Они совсем даже не лишние - вам ничего не мешает обратится по любому offset в массиве.
Если вам хочется красивое и универсальное решение - это можно сделать через классы (CPP). Создаётся класс STRING, в нём перегружается левая и правая операция '+' и вы сможете складывать элементы классов а также классов и строк как простые числа (для строк под '+' понимается конечно конкатенация).
Тогда весь ваш код будет выглядеть приблизительно так:

STRING Tail("AT+CUSD=1,");
STRING End(",15\r");
char gsm_ussd_balance_command; // здесь ввод пользователя
STRING Result();
Result = Tail + char gsm_ussd_balance_command;
Result = Result + End;
char* Output = Result.Get(); //Get() - ещё один метод класса STRING.

Такой пример со строками есть в любом учебнике по CPP

[supercelt]

Они совсем даже не лишние

Как не лишние, если этих индексов я не создавал?)

char *gsm_cusd_command[2] = {"AT+CUSD=1,",",15\r"};

Вот, их же только два, с индексами 0 и 1.

[H4LF]

supercelt, Вы на чём пишете? Как думаете - там есть проверка границ массивов? В C об этом должен заботиться программист.