-1

我想添加一个新行来分隔文件中的每个条目。

我试过:

show_dump(buff + "\n", len, dump_fd);

但有一个错误。

我从其他地方得到了这个函数,它将客户端发送的缓冲区格式化为十六进制和 ASCII 数据。

/*
Usage:
    show_dump(buffer, buffer_length, stdout);
    show_dump(buffer, buffer_length, fd);
*/

#include <string.h>



void show_dump(unsigned char *data, unsigned int len, FILE *stream) {
    const static char       hex[] = "0123456789abcdef";
    static unsigned char    buff[67];   /* HEX  CHAR\n */
    unsigned char           chr,
                            *bytes,
                            *p,
                            *limit,
                            *glimit = data + len;

    memset(buff + 2, ' ', 48);

    while(data < glimit) {
        limit = data + 16;
        if(limit > glimit) {
           limit = glimit;
           memset(buff, ' ', 48);
        }

        p     = buff;
        bytes = p + 50;
        while(data < limit) {
            chr = *data;
            *p++ = hex[chr >> 4];
            *p++ = hex[chr & 15];
            p++;
            *bytes++ = ((chr < ' ') || (chr >= 0x7f)) ? '.' : chr;
            data++;
        }
        *bytes++ = '\n';

        fwrite(buff, bytes - buff, 1, stream);
     }
}

这就是我创建文件的方式:

if(dump) {
    add = dumpfile;

    for(i = 1; ; i++) {
        sprintf(add, "%d.log", i);
        dump_fd = fopen(dumpfile, "rb");
        if(!dump_fd) break;
        fclose(dump_fd);
    }
    dump_fd = fopen(dumpfile, "wb");
    if(!dump_fd) std_err();

}

这就是我写入文件的方式:

if(dump_fd) show_dump(buff, len, dump_fd);

问题:代码有什么问题?为什么文件都写在一起而不是逐行写?

4

1 回答 1

1

我想您可以将 '\n' 添加到缓冲区中:

buff[strlen(buff)]='\n';
buff[strlen(buff)+1]='\0';

应该工作,我想。

-编辑:

或者像评论中指出的那样(可能是更好的解决方案..),使用 strcat:

strcat(buff,"\n");

如果需要,用 if 语句封装它以防止溢出:

if (sizeof(buff) > strlen(buff)+1)
于 2013-04-22T17:16:28.687 回答