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 =（char *）malloc（nwords * sizeof（char））之前移动p == null 的检查;'

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

将p更改为char **

将分配大小（realloc调用）更改为nwords * sizeof(*p)

为每个单词分配（使用malloc）存储，而不是让word成为堆栈分配的数组

设置p[i] = word;而非当前分配。

C - Loop不会破坏

1 个答案: