c - K&R 将多个空格替换为单个空格

Question

#include <stdio.h>
#define MAXLENGTH 1000

void get_input(char ch[], int maxline);
void replace_blanks(char ch[], int z);

int main(void)
{

  char ch[MAXLENGTH];
  int z;

  z = 0;

  printf("Please enter some input. ");
  get_input(ch,MAXLENGTH);
  replace_blanks(ch,MAXLENGTH);

  while(putchar(ch[z]) != EOF)
    {
        ++z;
    }

return 0;

}

void get_input(char ch[], int lim)
{

  int z, i;

  z = 0;

  while((i = getchar()) !=  EOF && z < lim-1 && i != '\n')
      {
          ch[z] = i;
          z++;
      }

  if(i == '\n')
      {
          ch[z] = i;
          z++;
      }

  ch[z] = '\0';

}

void replace_blanks(char ch[], int lim)
{
  char ch_copy[lim];
  int i, arrayindex, copy_counter;

  copy_counter = 0;
  arrayindex = 0;
  i = arrayindex;

  while(ch[arrayindex] != EOF && arrayindex < lim-1){ 
    if(ch[arrayindex] != ' '){
      ch_copy[copy_counter] = ch[arrayindex];
      i = arrayindex;
      arrayindex++;
      copy_counter++;
    }
    else if(ch[arrayindex] == ' ' && ch[i] != ' '){
      ch_copy[copy_counter] = ch[arrayindex];
      i = arrayindex;
      arrayindex++;
      copy_counter++;
    } else {
      i = arrayindex;
      arrayindex++;
    }   
 }

  ch_copy[arrayindex] = '\0';

  for(arrayindex = 0; ch_copy[arrayindex] != EOF && arrayindex < lim-1; arrayindex++){
    printf("%d", arrayindex);
    ch[arrayindex] = ch_copy[arrayindex];
  }

  ch[arrayindex] = '\0';
}

printf() 中该函数的输出为

Please enter some input. hello   hi
0123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821822823824825826827828829830831832833834835836837838839840841842843844845846847848849850851852853854855856857858859860861862863864865866867868869870871872873874875876877878879880881882883884885886887888889890891892893894895896897898899900901902903904905906907908909910911912913914915916917918919920921922923924925926927928929930931932933934935936937938939940941942943944945946947948949950951952953954955956957958959960961962963964965966967968969970971972973974975976977978979980981982983984985986987988989990991992993994995996997998hello hi
Segmentation fault: 11

Answer 1

不不，声明ch如char[] :

char ch[MAXLENGTH];

虽然可能，但绝对可以将字符串存储在 int 中大批，你根本不应该这样做。

我也会写你的 get_input像这样:

int get_input(char *ch, size_t lim)
{
    if(!fgets(ch, lim, stdin))
        return 0;

    // removing possible \n
    ch[strcspn(ch, "\n")] = 0;
    return 1;
}

算法背后的逻辑似乎没问题，但是你的while和for条件有点奇怪，在单步执行时会出现问题调试器。

while(ch[arrayindex] != EOF && arrayindex < lim-1)

不检查 EOF ，你不应该写 EOF在一个字符串，因为 EOF是 signed int并且不适合 char 。如果你读取一行 fgets ，永远不会有 EOF在字符串中。还因为你做了while((i = getchar()) != EOF && z < lim-1 && i != '\n')在读取功能，你不会有 EOF在字符串中。你应该检查'\0'这是终止字符串的唯一正确方法。

正确的条件是:

while(ch[arrayindex] != '\0' && arrayindex < lim-1)

如果您要检查边界，那么最好先检查边界然后检查字符串的结尾。

while(arrayindex < lim-1 && ch[arrayindex])

这里我省略了!= '\0'因为'\0' == 0 .

循环之后你应该做

ch_copy[copy_counter] = '\0';

使用copy_counter ，不是arrayindex ，因为arrayindex是索引原始的，其中存储了多个空白。 copy_counter是索引复制。

但如果先于 while你所做的循环:

memset(ch_copy, 0, lim);

然后整个缓冲区将被初始化为 0，你不必担心关于 0 终止，因为副本永远不会大于原始副本。

最后，for中的条件应使用相同的逻辑纠正循环至于while环形。在这里你不需要检查边界，因为你知道副本永远不会比原始版本大:

for(arrayindex = 0; ch_copy[arrayindex]; arrayindex++){
    ch[arrayindex] = ch_copy[arrayindex];
}

ch[arrayindex] = 0;

或者您可以使用 strcpy :

strcpy(ch, ch_copy);

所以函数应该是这样的:

void replace_blanks(char ch[], int lim)
{
  char ch_copy[lim];
  int i, arrayindex, copy_counter;

  memset(ch_copy, 0, lim);

  copy_counter = 0;
  arrayindex = 0;
  i = arrayindex;

  while(arrayindex < lim -1  && ch[arrayindex]){
    if(ch[arrayindex] != ' '){
      ch_copy[copy_counter] = ch[arrayindex];
      i = arrayindex;
      arrayindex++;
      copy_counter++;
    }
    else if(ch[arrayindex] == ' ' && ch[i] != ' '){
      ch_copy[copy_counter] = ch[arrayindex];
      i = arrayindex;
      arrayindex++;
      copy_counter++;
    } else {
      i = arrayindex;
      arrayindex++;
    }   
 }

  strcpy(ch, ch_copy);
}