在C中使用指针和malloc时出现多个错误消息

Question

我对C还是很陌生，并且在我的代码中收到了多个错误消息，如果有人可以向我解释这些错误消息，那就太好了。

我的代码：

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

int main()
{
  char newINPUT;

  char* INPUT = (char*)malloc(1 * sizeof(char));



  while((INPUT = getchar()) =! EOF)
    char* newINPUT = (char*)realloc(INPUT, 2* sizeof(INPUT));
    if(newINPUT =! NULL)
    {
      INPUT = newINPUT;
    }
    else
    {
      free(INPUT);
      printf("Out of memory!");
    }

    return 0;

}

这应该做的是：它应该分配一些内存并读取您键入的内容。 如果没有分配的内存，则应重新分配两倍于以前分配的内存。

错误消息：警告行13（带有while的行）：赋值使指针从整数开始而不进行强制转换（默认启用）错误行13：要求左值作为赋值的左操作数错误行14：'char'警告行​​之前的期望表达式17：与第13行。

如果有人可以向我解释这一点会很好。 谢谢！

---

编辑：感谢到目前为止的所有答案！ 他们确实帮助我了解了malloc和realloc。 我昨天尝试了另一个自己的版本，但是由于不能用足够大的输入来测试它，所以我不确定分配的内存是否正常工作。 在重新分配和加倍的条件下（在语法上）可行吗？

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

int main()
{

  int *READ; //input
  READ = malloc(sizeof(int));


  while ((*READ = getchar()) != EOF)
  {
    if (READ) //if read not null so memory is enough
    {
      putchar(*READ);
    }
    else
    {
      int x;
      int* extendedREAD = realloc(READ, (x*2));//expression for "double as much as before"????

      if (extendedREAD = NULL) // looking if there s enough memory
      {
        printf("Error, not enough memory!");
      }
      x = x * 2;
    }
  }



  return 0;
}




/*
Zeichen einlesen - checken ob eof, wenn nein, dann speicher ausreichend? nein dann vergräßern, sonst zeichen dazuschreiben, dann wieder vonvorne)
*/

谢谢！

Answer 1

这里有很多问题...

int main()
{
  char newINPUT;

  char* INPUT = (char*)malloc(1 * sizeof(char));

不要将调用的结果malloc()为malloc() ； 这是不必要的，并且可能导致意外的问题。

  while((INPUT = getchar()) =! EOF)

这里有两个问题。 首先， a =! b a =! b表示“将a设置为b的逻辑补” 。 您可能想使用a != b形式a != b东西（ “ a不等于b ” ）。 其次， getchar()返回一个int值。 如果尝试将返回值存储在char容器中，则可能会发现它永远不会等于EOF 。

    char* newINPUT = (char*)realloc(INPUT, 2* sizeof(INPUT));

在这里，您将newINPUT重新声明为char*指针。 您之前char newINPUT其声明为char newINPUT 。 这将导致编译错误。

    if(newINPUT =! NULL)

您在此处将newINPUT设置为NULL的逻辑补码。 我想你是说if (newINPUT != NULL) 。 您可以简单地使用if (newINPUT)代替。

（关于内存泄漏的胡言乱语已删除-我的错！）

总体而言，您的程序逻辑存在缺陷，因为您试图为输入的每个单个字符将分配的内存加倍。 因此，即使您解决了上述问题，在输入大约32个字符后，内存也将耗尽。

---

建议改写：

下面的代码存储在输入缓冲器的当前大小size ，并且在所述输入字符串的长度nchars 。 两者都初始化为零。 缓冲区位置本身（ char *input ）可以初始化为NULL ，因为使用空指针调用realloc()等同于调用malloc() 。 这意味着我们可以从代码中消除一个系统调用。

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

int main() {
  char *input=NULL, *new_input;
  int c, size=0, nchars=0;

  while (1) {
    if (size==nchars) {
      size = size*2 + (size==0);  /* (size==0) is 1 if size==0, zero othewise */
      if (!(new_input = realloc(input,size))) {
        puts("Out of memory");
        free(input);
        return 1;
      }
      input = new_input;
      printf("Buffer reallocated; new size is %d byte(s)\n",size);
    }
    if ((c=getchar())==EOF) {
      input[nchars] = '\0';  /* Terminate the input string */
      break;
    }
    input[nchars++] = c;
  }

  printf("Buffer size:   %d\n"
         "String length: %d\n"
         "Input was:     %s\n",size,nchars,input);
  free(input);
  return 0;
}

Answer 2

修改示例

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

int main(void){
    int ch;
    char* INPUT = (char*)malloc(1 * sizeof(char));
    int input_buf_size = 1;
    int input_count = 0;

    while((ch = getchar()) != EOF){
        INPUT[input_count] = ch;
        input_count += 1;
        if(input_count == input_buf_size){
            char *newINPUT = (char*)realloc(INPUT, input_buf_size *= 2);
            if(newINPUT != NULL){
                INPUT = newINPUT;
            } else {
                free(INPUT);
                printf("Out of memory!");
                exit(EXIT_FAILURE);
            }
        }
    }
    INPUT[input_count] = '\0';
    puts(INPUT);
    free(INPUT);
    return 0;
}