自定义get_next_line函数连续换行符处理异常问题求助

问题现象 #

• 连续换行场景：文件包含hello there\n\n\n\n时，预期返回5行，实际仅返回2行
• EOF处理异常：文件末尾无换行的剩余内容会一次性输出，而非按逐行逻辑返回

问题代码分析 #

1. get_next_line初始检查逻辑缺失 #

函数调用时未先检查现有静态buffer中是否存在换行符，直接进入读取循环。导致buffer中已有的连续换行符无法被及时处理，需等到下一次读取才会被检测（但此时已无新内容可读）。

2. update_remainder_buffer中realloc参数错误 #

原代码错误传递3个参数给realloc（标准realloc仅需2个参数：指针、新大小），导致内存重分配逻辑异常，剩余内容无法正确保留：

*buffer = realloc(*buffer, strlen(*buffer), remaining_len + 1); // 错误调用

3. extract_line冗余逻辑导致处理不统一 #

开头单独处理\n的分支与后续逻辑重复，且未覆盖所有连续换行场景，导致部分空行被跳过。

修复方案 #

1. 修正get_next_line的初始检查逻辑 #

先检查现有buffer中的换行符，再决定是否读取新内容，同时优化EOF处理逻辑：

char    *get_next_line(int fd)
{
    static char *buffer = NULL;
    char        *line;
    int         newline_index;

    if (fd < 0 || BUFFER_SIZE <= 0)
        return (NULL);
    if (!buffer)
    {
        buffer = malloc(1);
        if (!buffer)
            return (NULL);
        buffer[0] = '\0';
    }
    // 先检查现有buffer中的换行符
    newline_index = find_newline(buffer);
    // 仅当无换行符时才读取新内容
    while (newline_index == -1 && read_newpiece(fd, &buffer) > 0)
        newline_index = find_newline(buffer);
    
    if (newline_index == -1)
    {
        // EOF场景：buffer为空则释放返回NULL，否则提取剩余内容
        if (!buffer || buffer[0] == '\0')
        {
            free(buffer);
            buffer = NULL;
            return (NULL);
        }
        // 提取剩余内容后清空buffer
        line = extract_line(&buffer, newline_index);
        free(buffer);
        buffer = NULL;
        return (line);
    }
    line = extract_line(&buffer, newline_index);
    return (line);
}

2. 修复update_remainder_buffer的realloc调用 #

修正参数数量与逻辑，确保剩余内容正确保留：

static int  update_remainder_buffer(char **buffer, int newline_pos) 
{
    int remaining_len;

    remaining_len = strlen(*buffer + newline_pos);
    if (remaining_len > 0)
        memmove(*buffer, *buffer + newline_pos, remaining_len + 1);
    else
        (*buffer)[0] = '\0';
    // 修正realloc参数：仅传递新大小
    *buffer = realloc(*buffer, remaining_len + 1);
    if (!*buffer)
        return (0);
    return (1);
}

3. 简化extract_line逻辑 #

移除冗余的开头\n处理分支，统一用通用逻辑处理所有场景：

char    *extract_line(char **buffer, int newline_pos)
{
    char    *line;
    int     line_len;

    if (newline_pos == -1)
        line_len = strlen(*buffer);
    else
        line_len = newline_pos + 1; // 包含换行符本身
    
    line = malloc(line_len + 1);
    if (!line)
        return (NULL);
    memcpy(line, *buffer, line_len);
    line[line_len] = '\0';
    
    if (!update_remainder_buffer(buffer, line_len))
    {
        free(line);
        return (NULL);
    }
    return (line);
}

4. 统一find_newline函数调用 #

确保get_next_line中调用的是正确的find_newline函数（修正原代码中find_newline_eof的笔误）：

int find_newline(char *buffer)
{
    int i;

    i = 0;
    if (!buffer)
        return (-1);
    while (buffer[i])
    {
        if (buffer[i] == '\n')
            return (i);
        i++;
    }
    return (-1);
}

修复验证 #

• 连续换行文件：输出5个()，每个空行被正确返回
• 文本+连续换行文件：输出10个()，所有行（包括空行）被逐个返回
• 长文件末尾：剩余无换行内容被作为单独一行返回，符合逐行逻辑

内容的提问来源于stack exchange，提问作者Rache Bartmoss