как удалить символ из абзаца в с?

Question

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

#include <stdio.h>

int main()
{
    char inputString[1000], ch, outputString[1000];

    int i,j=0;

    printf ("Enter a multi line string:\n");
    scanf ("%[^;]s", inputString);
    gets(inputString);

    printf ("Enter a letter to be removed:\n");
    scanf ("%c", &ch);
    for (i=0; inputString[i]!='\0';i++)
    {
        if(inputString[i]==ch)
        {
            outputString[j]=inputString[i+1];
            i++;
        }
        else
        {
            outputString[j]=inputString[i];
        }
        j++;
    }
    for (i=0;i<j;i++)
    {
        printf ("%c", outputString[i]);
    }
    return 0;
}

Answer 1

В вашей программе есть несколько проблем

1) С

scanf ("%[^;]s", inputString);
gets(inputString);

вы переписываете inputString с получением после scanf ,например, если вход foo bar loop;, вы устанавливаете inputString с foo bar loop, тогда вы устанавливаете его снова с ;, поэтому вы потеряли все, что получили с scanf

2) Оба scanf и получают могут записывать из inputString , у вас нет никакой защиты, максимальный размер для чтения может быть задан scanf и вы можете заменить использование gets на fgets (забыв, что вы переписываете в inputString )

3) выникогда не принимайте во внимание случай EOF, всегда предполагая, что что-то вводится

4) В

    if(inputString[i]==ch)
    {
        outputString[j]=inputString[i+1];
        i++;
    }
    else
    {
        outputString[j]=inputString[i];
    }
    j++;

outputString[j]=inputString[i+1]; предполагается, что вам никогда не придется два раза подряд удалять символ, таким образом, сделатьнеправильно.

5) Чтобы использовать фиксированный размер массива не обязательно, выделите массив, тогда realloc при необходимости

6) В

for (i=0;i<j;i++)
{
    printf ("%c", outputString[i]);
}

лучше заканчивать outputString нулевым символом, который просто печатается с помощью fputs или , ставит , чтобы добавитьокончательный \ n, если он отсутствует

---

Вот предложение:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

int main()
{
  char * inputString = NULL;
  size_t sz = 0, used = 0;
  int v;

  printf ("Enter a multi line string (ended by ';'):\n");

  while ((v = getchar()) != ';') {
    if (v == EOF) {
      puts("premature EOF, abort");
      return -1;
    }

    if (used == sz) {
      sz += 100;
      inputString = realloc(inputString, sz);
      if (inputString == NULL) {
        puts("not enough memory");
        return -1;
      }
    }
    inputString[used++] = v;
  }

  /* read up to end of line */
  char * outputString = 0;

  if (getline(&outputString, &sz, stdin) == -1) {
    puts("premature EOF or not enough memory");
    return -1;
  }
  free(outputString);

  printf ("Enter a letter to be removed:\n");

  char ch;

  if (scanf ("%c", &ch) != 1) {
     puts("premature EOF, abort");
     return -1;
  }

  size_t i, j = 0;

  outputString = malloc(used + 1); /* worst case, +1 to have place for null character */

  for (i = 0; i != used; ++i)
  {
    if (inputString[i] != ch)
      outputString[j++] = inputString[i];
  }
  outputString[j] = 0;

  puts(outputString);

  free(inputString);
  free(outputString);

  return 0;
}

Компиляция и выполнение:

pi@raspberrypi:/tmp $ gcc -pedantic -Wextra -g l.c
pi@raspberrypi:/tmp $ ./a.out
Enter a multi line string (ended by ';'):
this is a paragraph
  terminated by
a ;!!
Enter a letter to be removed:
i
ths s a paragraph
  termnated by
a 

Выполнение в valgrind :

pi@raspberrypi:/tmp $ valgrind ./a.out
==3900== Memcheck, a memory error detector
==3900== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==3900== Using Valgrind-3.13.0 and LibVEX; rerun with -h for copyright info
==3900== Command: ./a.out
==3900== 
Enter a multi line string (ended by ';'):
this is a paragraph
  terminated by
a ;!!
Enter a letter to be removed:
i
ths s a paragraph
  termnated by
a 
==3900== 
==3900== HEAP SUMMARY:
==3900==     in use at exit: 0 bytes in 0 blocks
==3900==   total heap usage: 5 allocs, 5 frees, 2,307 bytes allocated
==3900== 
==3900== All heap blocks were freed -- no leaks are possible
==3900== 
==3900== For counts of detected and suppressed errors, rerun with: -v
==3900== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 6 from 3)