Question

我需要一个为另一个程序重现getline函数基本功能的函数，这个函数应该能够正确读取FP并且没有巨大的缓冲区（256应该就够了）我在下面的代码中有2个问题 - 例如http://pastebin.com/BXj3SH92（./a.out＆lt; file - 当我调用我的函数而不给它一个文件时，它应该像猫一样工作但是在我按下第一个回车键（./a.out）之后停止

这是代码：

#define MOAR_BUF 256

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

char            *my_getline(const int fd)
{
  static char   save[MOAR_BUF];
  static int    i = MOAR_BUF;
  int           g;
  int           f;
  char          *save2;

  save2 = NULL;
  g = 0;
  f = 0;
  if (g == 0 && i >= (MOAR_BUF -1))
    {
      i = 0;
      g = read(fd, save, MOAR_BUF + 1);
      save[g] = '\0';
    }
  if (i <= MOAR_BUF && save[i] != '\0')
    {
      save2 = malloc((sizeof(*save2) * MOAR_BUF));
      while(save[i] != '\n' && save[i] != '\0')
        save2[f++] = save[i++];
      save2[f] = '\0';
      if (save[i] != '\0')
        i = i + 1;
    }
  return(save2);
}

int             main()
{
  int           i;
  char          **converted_map;
  char          *map;

  i = 0;
  converted_map = malloc(sizeof(*converted_map) * 30);
  while(map = my_getline(0))
    {
      converted_map[i] = strdup(map);
      printf("%s\n", converted_map[i]);
      i++;
    }
}

Answer 1

static char   save[MOAR_BUF];

g = read(fd, save, MOAR_BUF);
save[g] = '\0';

您告诉read读取MOAR_BUF个字节，当它成功读取多个时，0终止写入数组的末尾，调用未定义的行为。

您需要告诉read读取更少的字节，例如MOAR_BUF - 1，或者让缓冲区更大，例如MOAR_BUF + 1。

然后，在main，

while(map = my_getline(0))
{
  printf("%s\n", converted_map[i]);
  converted_map[i] = strdup(map);
  i++;
}

在分配之前打印出converted_map[i]，这是更多未定义的行为。您需要首先 strdup(map)到converted_map[i]，然后就可以打印出来。

在C中记录getline函数

1 个答案: