Как поставить пробел после запятой в строке?

Question

[Илья]

Нужно вставить 1 пробел после каждой запятой, если он отсутствует перед следующим словом.

#include <stdio.h>
#include <string.h>
void Insert(char sentence[256], int pos) {
  for (int i = strlen(sentence); i >= pos; i--) {
    sentence[i + 1] = sentence[i];
    sentence[pos] = ' ';
  }
}
int main() {
  char sentence[256];
  printf("Insert text: ");
  fgets(sentence, sizeof(sentence), stdin);
  for (int i = 0; i < strlen(sentence); i++)
    if (sentence[i] == ',' && sentence[i + 1] != ' ')
      Insert(sentence, i);
  printf("\nYour edited text: %s\n", sentence);
  return 0;
}

В результате выполнения, программа всё же добавляет пробел, но удаляет запятую


В чём же проблема?

Comments

[res2001]

Есть же memmove, зачем руками копировать байты? Это скучное занятие.
Искать запятые можно с помощью strchr.

В Insert вы затираете завершающий ноль и не восстанавливаете его. По идее последний printf вам может выдать вашу строку с мусорным продолжением, так же может быть и SEGFAULT если не повезет.
Вы не контролируете выход за границы массива в Insert. Например, что будет, если введенная строка будет длиной в 255 байт и там надо будет вставить 2 пробела (хотя достаточно и одного, чтоб выход за границы массива случился в printf)?

[jcmvbkbc]

В Insert вы затираете завершающий ноль и не восстанавливаете его.

res2001, Нет, не затирает. Первая итерация цикла как раз его копирует.

[res2001]

jcmvbkbc, Да, согласен. Завершающий ноль не затирается.
Но выход за границу массива все равно возможен, когда итоговая строка будет больше 255 символов.
Т.к. массив sentence лежит на стеке, то это может проявляться по разному.

Answer 1

[Илья]

for (int i = strlen(sentence); i > pos; i--) {
sentence[i + 1] = sentence[i];
}
sentence[pos] = ',';
sentence[pos + 1] = ' ';

Comments

[jcmvbkbc]

sentence[pos] = ','; не нужно, потому что функция вызывается только когда sentence[pos] == ',';

Answer 2

[jcmvbkbc]

В чём же проблема?

for (int i = strlen(sentence); i >= pos; i--) {
    sentence[i + 1] = sentence[i];
    sentence[pos] = ' ';
  }

Должно быть

for (int i = strlen(sentence); i > pos; i--) {
    sentence[i + 1] = sentence[i];
  }
  sentence[pos + 1] = ' ';