2

嗨,我正在尝试通过将整个文件加载到char[]using中来标记字符串fread。由于某些奇怪的原因,它并不总是有效,并且 valgrind 在这个非常小的示例程序中抱怨。

给定一个像这样的输入test.txt

first
second

以及以下程序

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <sys/stat.h>


//returns the filesize in bytes
size_t fsize(const char* fname){
  struct stat st ;
  stat(fname,&st);
  return st.st_size;
}

int main(int argc, char *argv[]){
  FILE *fp = NULL;
  if(NULL==(fp=fopen(argv[1],"r"))){
    fprintf(stderr,"\t-> Error reading file:%s\n",argv[1]);
    return 0;
  }
  char buffer[fsize(argv[1])];
  fread(buffer,sizeof(char),fsize(argv[1]),fp);
  char *str = strtok(buffer," \t\n");

  while(NULL!=str){
    fprintf(stderr,"token is:%s with strlen:%lu\n",str,strlen(str));
    str = strtok(NULL," \t\n");
  }
  return 0;
}

像这样编译

gcc test.c -std=c99 -ggdb

像跑步一样

./a.out test.txt

谢谢

4

5 回答 5

6

你的buffer尺寸应该是filesize + 1. +1用于null字符。

filesize = fsize(argv[1]);
char buffer[filesize + 1];

也不fread将 a\0放在字符串的末尾。所以你必须自己做:

fread(buffer,sizeof(char),filesize,fp);
buffer[filesize] = 0;
于 2010-04-02T07:34:45.700 回答
5

从这个网站

int main(int argc, char* argv[])
{
  std::string str = "The quick brown fox";

  // construct a stream from the string
  std::istringstream stream(str);

  // use stream iterators to copy the stream to the vector
  // as whitespace separated strings
  std::istream_iterator<std::string> it(stream), end;

  std::vector<std::string> results(it, end);

  // results = ["The", "quick", "brown", "fox"]
}

比处理那些不断敲打你头的讨厌的 C 弦要容易得多。

你知道使用高阶方法有什么好处吗?它占用更少的屏幕空间并且更容易理解。

于 2010-04-02T08:53:45.887 回答
2

buffer不是空终止的。您需要使其比文件大小大一个字节,并且需要将最后一个字节设置为\0.

于 2010-04-02T07:35:01.990 回答
2

您的缓冲区必须是filesize + 1并且您还需要设置终止 0:

int size = fsize(argv[1]);
char buffer[size + 1];
buffer[size] ='\0';

此外,您可能应该在堆而不是堆栈上分配缓冲区......

于 2010-04-02T07:38:37.920 回答
1

你的缓冲区太小了。试试这个:

int fileSize = fsize(argv[1]);
char buffer[fileSize + 1]; 
buffer[fileSize] = 0;

就在您致电之前fread

于 2010-04-02T07:37:44.213 回答