时间戳

首页 > 解决方案 > 使用 C 打印指向字符串的指针数组会忽略最后一个字符串

c - 使用 C 打印指向字符串的指针数组会忽略最后一个字符串

问题描述

我希望创建一个指向从 C 中的文件读取的字符串的指针数组。但是，当我尝试打印复制到的字符串时stdout，文件的最后一行总是被遗漏。该程序有时也会遇到segmentation fault我无法完全消除的问题。它发生大约五分之二。

这是我的input.c代码：

#define _GNU_SOURCE
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include "input.h"

#define MAXLINES 5000


void writelines(char *arr[], int l);

char *read_lines[MAXLINES];

void get_input(const char *fp) {
    FILE *contents;
    char *line;
    char *temp;
    size_t len;
    ssize_t read;
    int i;
    i = 0;
    contents = fopen(fp, "r");
    if (contents == NULL)
        exit(EXIT_FAILURE);
    while ((read = getline(&line, &len, contents)) != -1) {
        if ((temp = (char *) malloc(strlen(line) + 1)) == NULL) {
            printf("Could not allocate required memory.");
            exit(EXIT_FAILURE);
        }
        else {
            line[strlen(line) - 1] = '\0';
            strcpy(temp, line);
            read_lines[i++] = temp;
        }
    }
    fclose(contents);
    free(line);
    free(temp);
    writelines(read_lines, i);
    exit(EXIT_SUCCESS);
}

void writelines(char *arr[], int l) {
    int i;

    for (i = 0; i < l; i++) {
        printf("%s\n", arr[i]);
    }
}

我的main.c文件是：

#include <stdio.h>
#include "input.h"

int main(int argc, char *argv[]) {
    if (argc == 1)
        printf("Please provide a valid source code file.\n");
    else
        get_input(*(++argv));

    return 0;
}

我编译使用gcc main.c input.c -Wall没有警告或错误。

使用gdb我可以确认该过程正常运行。
当它遇到分段错误时，回溯显示对它的调用strlen显然失败了。

标签： carrayspointersstdout

解决方案

从文档：

如果 *lineptr 为 NULL，则 getline() 将分配一个缓冲区来存储该行，该缓冲区应由用户程序释放。（在这种情况下，*n 中的值被忽略。）

但在您的情况下，您第一次传递了一个未初始化的值，因此认为它可以写入该非法位置，这是未定义的行为（这解释了“它发生了大约 5 次中的 2 次”的事情）getlinegetline

第一个修复应该是初始化line：

char *line = NULL;

那么，line为什么要line创建NULL. 所以下次getline重用之前的缓冲区，它可能不够长，无法容纳下一行。

修复只是存储该行：

read_lines[i++] = line;

然后设置line = NULLsogetline为下一行分配适当的 len。并删除malloc代码，它没有用。

固定部分（你不需要传递指针len被忽略）：

line = NULL;
while ((read = getline(&line, NULL, contents)) != -1) {
         read_lines[i++] = line;
         line[strcspn(line, "\n")] = 0;  // strip off linefeed if there's one
         line = NULL;
 }

（换行条改编自从fgets() 输入中删除尾随换行符）

推荐阅读