如何在给定指针数组的情况下创建字符串数组?

时间:2015-03-10 23:24:00

标签: c arrays string pointers

我得到了一个文本文件,我需要将它放在一个缓冲区中,并在将每一行转换为字符串后使用get_lines创建一个指针数组。我在使用get_lines函数时遇到了麻烦,因为我在运行时遇到了seg错误。

这是我的代码:

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

int readfile(FILE *fp, char **cbuf);       
char **get_lines(char *cbuf, int bufsize, int word);
int readword(FILE*tmp);

int main(int argc, char *argv[])
{
  int i,bufsize, num_word;
  char *cbuf;
  char **lines;
  FILE *fp;
  FILE *tmp;

  if( (fp=fopen(argv[1],"r")) == NULL)
    {
      perror("ERROR: bad/no filename");
      exit(0);
    }
  tmp = fopen(argv[1],"r");
  bufsize = readfile(fp,&cbuf);
  num_word = readword(tmp);
  lines = get_lines(cbuf, bufsize, num_word) ; 
  i=0;
  while( lines[i] != NULL) { 
    printf("%i\t%s\n",i,lines[i]); 
    i++;
    }
  return 0;
}

int readfile(FILE *fp,char**cbuf)
{
  int i;
  char c;

  fseek(fp, 0L, SEEK_END);
  int bufsize = ftell(fp);
  fseek(fp, 0L, SEEK_SET);

  *cbuf = (char *)malloc(sizeof(char) * bufsize);

  for (i = 0; i < bufsize; i++)
    {
      c = fgetc(fp);
      (*cbuf)[i] = c;
    }
  return bufsize;
}

int readword(FILE*tmp)
{
  int word = 0;
  char c;

  while((c = fgetc(tmp)) != EOF )
    {
      if (c == '\n')
      word++;
    }
  return word;
}

char **get_lines(char *cbuf, int bufsize, int word)
{
  int i = 0, j = 0, counter = 0;
  char (*lines)[bufsize];
  lines = (char**)malloc(sizeof(char*)*bufsize);

  counter = cbuf;  
  for (i = 0; i < bufsize; i++)
    {
      if(cbuf[i] == '\n')
    {
      cbuf[i] == '\0';
      counter = cbuf[i + 1];
      j++;
    }else
    {
      *lines[j] = &counter;
    }
    }

  lines[word] == NULL;
  return lines;
}

导致错误的违规行为对我来说并不是很明显,有人可以告诉我get_lines()中可能有什么问题吗?

1 个答案:

答案 0 :(得分:1)

此代码错误:

char (*lines)[bufsize];
lines = (char**)malloc(sizeof(char*)*bufsize);

它分配一个指向char数组的指针。然后你malloc错误的空间量,转换为错误的类型,并写*lines[j] = &counter;试图将指针存储在char

您应该为get_lines函数获取许多编译器错误/警告。重要的是要注意这些消息,因为它们告诉您代码出了问题。在修复所有错误和警告之前,甚至没有必要开始调查段错误。

See here有关如何调试代码的精彩指南;我怀疑你会对get_lines函数的橡皮鸭测试失败。


这是一个固定版本(未经测试):

// Precondition: cbuf[bufsize] == '\0'
//
char **get_lines(char *cbuf, size_t bufsize, size_t num_lines)
{
    // +1 for the NULL termination of the list
    char **lines = malloc((num_lines + 1) * sizeof *lines);

    size_t line = 0;
    while ( line < num_lines )
    {
        lines[line++] = cbuf;
        cbuf = strchr(cbuf, '\n');

        if ( !cbuf )
            break;

        *cbuf++ = '\0';
    }

    lines[line] = NULL;
    return lines;
}

在现有代码中,没有空间为最后一行写空终止符;我的建议是让readfile实际上malloc增加一个字节,并确保将其设置为0