我试图从一个文件中读取内容,将其存储在一个字符串中,获取该字符串的长度,写入该字符串的长度,然后将该文件的内容写入另一个文件。实现存档
#内容,其中#是内容的长度。
到目前为止,这写了一些奇怪的字符和写出来的内容行额外的时间或太多。
如果内容超过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在缓冲区之间复制数据。

07-24 09:46
查看更多