增加函数中字符串的大小时,realloc()无效的指针错误

时间:2018-11-03 10:07:08

标签: c dynamic-memory-allocation c89

当我运行代码时,它显示realloc()无效的指针错误。

input()函数有什么问题吗?

#include<stdio.h>
#include<string.h>
#include<stdlib.h>
#include<time.h>
char *input(void)
{
    int n = 1;
    char *str = malloc(sizeof(char));
    *str = '\0';
    while((*str=getchar())!='\n')
    {
        n++;
        str = realloc(str,sizeof(char)*n);
        str++;
    }
    return str;
}

int main(int argc, char const *argv[])
{
    char *str = input();
    printf("%s",str);
    free(str);
    return 0;
}

3 个答案:

答案 0 :(得分:4)

您犯了一些错误:

  • 您将返回字符串的结尾,而不是开头。

  • realloc需要原始地址(请参阅托马斯的回答)

  • realloc可能会返回一个新地址

  • 您不终止字符串。

以下内容解决了这些错误,并提供了一些建议:

char *input(void)
{
    size_t i=0;
    int c;
    char *str = malloc(1);
    if (!str) return 0;
    while((c=getchar())!=EOF && c!='\n')
    {
        str[i]= c;
        if ((newstr = realloc(str,i+1))==0)
            break;          // out of memory: return what we have
        str= newstr;
        i++;
    }
    str[i]= '\0';
    return str;
}

答案 1 :(得分:2)

完成str++之后,指针不再指向分配的字符串的开头。 realloc需要原始指针,而不是指向已分配数据内部某处的指针。

答案 2 :(得分:0)

一种没有冗余调用且可以完成错误检查的方法:

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

char *input(void)
{
  size_t n = 0;
  char * str = NULL;

  do
  {
    ++n;

    {
      void * pv = realloc(str, (n + 1) * sizeof *str);
      if (NULL == pv)
      {
        perror("realloc() failed");

        break;
      }

      str = pv;
    }

    {
      int result = getchar();

      if (EOF == result)
      {
        if (ferror(stdin))
        {
          fprintf(stderr, "getchar() failed\n");
        }

        --n;

        break;
      }

      str[n - 1] = result;
    }
  } while ('\n' != str[n - 1]);

  if (NULL != str)
  {
    str[n] = '\0';
  }

  return str;
}

int main(int argc, char const *argv[])
{
  int result = EXIT_SUCCESS; /* Be optimistic. */

  char * str = input();
  if (NULL == str)
  {
    result = EXIT_FAILURE;

    fprintf(stderr, "input() failed\n");
  }
  else
  {
    printf("input is: '%s'\n", str);
  }

  free(str);

  return result;
}