C - 循环不会中断

Question

我希望在按下“Enter”时循环中断。有什么建议吗？

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

#define len 20
#define limit 100

//Prototypes for functions
int read_word(char str[], int n);



int main(void)
{
  char *p;
  char word[len+1];
  int i=0, nwords = 0;

//Loop for reading in words and allocating an array 
  for (;;)
   {
      if (nwords == limit)
       {
          printf("Insufficient Space\n");
          break;
       }
      printf("Enter word: ");
      scanf("%c", &word);
      p = (char*) malloc(nwords*sizeof(char));
      p[i]= read_word(word, len);
      i++;

      if (p == NULL)
      {
          printf("Insufficient Space\n");
          break;
      }
  }

  for(i=0; i<nwords; i++)
      printf(" %s\n", p[i]);

  return 0;

  } 
int read_word(char str[], int n)
{
  char ch; 
  int i = 0;

  while((ch = getchar()) != '\n')
      if (i<n)
          str[i++] = ch; 
  str[i] = '\0';
  return i;
}

Answer 1

您的scanf 调用读取第一个字符，然后您的read_word 函数将其覆盖。如果scanf 调用读取了换行符，它将被忽略。

线条：

  p = (char*) malloc(nwords*sizeof(char));
  p[i]= read_word(word, len);

... 也出现错误。 read_word 返回一个整数（读取的字符串的长度），但您将存储到 char 数组中。此外，您每次通过循环都为p 重新分配内存，因此之前存储的值将丢失。

修复：

• 将p更改为int *，并将其初始化为null
• 将malloc 调用更改为合适的realloc
• 完全删除对scanf的调用
• 移动检查p == null在分配`p = (char*) malloc(nwords*sizeof(char));'之前

或者：p 实际上是一个字符串数组（单词本身）而不是单词长度？在这种情况下，您必须：

• 将p 更改为char **
• 将分配大小（用于realloc 调用）更改为nwords * sizeof(*p)
• 为每个单词分配（使用malloc）存储空间，而不是让word 成为堆栈分配的数组
• 设置p[i] = word; 而不是当前分配。