我试图从一个文件中读取内容,将其存储在一个字符串中,获取该字符串的长度,写入该字符串的长度,然后将该文件的内容写入另一个文件。实现存档
#内容,其中#是内容的长度。
到目前为止,这写了一些奇怪的字符和写出来的内容行额外的时间或太多。
如果内容超过1024,我需要继续读取,所以我假设我在len==1024时写入,然后再次读取并将其转换为数据。我目前还没有实现,但我想让这个工作之前,我开始使我的文件更大。
int file2p = open(curFilePath, O_RDONLY, 0);
if(file2p == -1){
printf("File open error.");
exit(1);
}
char buffer[1024];
int len;
int dataLen;
char data[1024];
while((len = read(file2p, buffer, 1024)) != 0){
if(len == -1){
printf("File open error.\n");
exit(1);
}
strcat(data, strdup(buffer));
printf("data: %s", data);
}
dataLen = strlen(data);
int lenLen = strlen(&dataLen);
write(filep, &dataLen, lenLen);
write(filep, ">", 1);
write(filep, data, dataLen);
//free(data);
close(file2p);
最佳答案
内存泄漏,缓冲区溢出。这不酷。您的data
数组是固定大小:strcat
不会使其变大。而且不能保证缓冲区以空结尾,因此strdup
是不可能的。
你想要这样的东西:
size_t dataLen = 0;
char *data = NULL;
while( (len = read(file2p, buffer, 1024)) != 0 ){
if( len == -1 ) {
perror( "Read failed" );
exit(1);
}
data = realloc( data, dataLen + len );
if( !data ) {
printf( "Not enough contiguous memory\n" );
exit(1);
}
memcpy( &data[dataLen], buffer, len );
dataLen += len;
}
write(filep, &dataLen, sizeof(dataLen));
write(filep, ">", 1);
if( data ) {
write(filep, data, dataLen);
free(data);
}
上面的代码不是最有效的方法,只是基于现有代码的说明。它根据需要动态分配和调整
data
缓冲区的大小,并使用memcpy
在缓冲区之间复制数据。