c - 我该如何解决这个有趣的getdelim/getline(动态内存分配)错误？

我有这个C分配，在这个特定点上我有点挣扎。我有一定的C语言背景，但是指针和动态内存管理仍然让我很头疼。

作业要求我们编写一个程序，该程序将模拟UNIX中“ uniq”命令/过滤器的行为。

但是我遇到的问题是C库函数getline或getdelim（我们需要根据实现规范来使用这些函数）。

根据规范，用户输入可能包含任意数量的行，并且每行可能具有任意长度（在编译时未知）。

问题是，while循环的以下行
    while（cap = getdelim（stream.linesArray，size，'\ n'，stdin））

当我这样离开时，它会以某种方式编译和“工作”。我的意思是，当我执行程序时，我每行输入任意数量的任意长度的行，并且该程序不会崩溃-但它会一直循环，除非我停止程序执行（是否正确存储了这些行）在“ char ** linesArray;”中，我不确定另外一个故事。

我想能够做的是
    同时（（cap = getdelim（stream.linesArray，size，'\ n'，stdin））&&（cap！= -1））

因此，当getdelim在某一行（除了EOF或\ n）不读取任何字符时（又是用户输入空行时的第一次），该程序将停止从stdin中获取更多行。
（然后通过getdelim打印存储在stream.linesArray中的行）。

问题是，当我执行上述更改时执行程序时，程序显示“ Segmentation Fault”错误，坦率地说，我不知道为什么以及如何解决此问题（我已尝试对此做一些事情，很多次都无济于事）。

以供参考：

https://pubs.opengroup.org/onlinepubs/9699919799/functions/getdelim.html

https://en.cppreference.com/w/c/experimental/dynamic/getline

http://man7.org/linux/man-pages/man3/getline.3.html

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define DEFAULT_SIZE 20

typedef unsigned long long int ull_int;

typedef struct uniqStream
{
    char **linesArray;
    ull_int lineIndex;

}   uniq;

int main()
{
    uniq stream = { malloc(DEFAULT_SIZE * sizeof(char)), 0 };

    ull_int cap, i = 0;
    size_t *size = 0;

    while ((cap = getdelim(stream.linesArray, size, '\n', stdin))) //&& (cap != -1))
    {
        stream.lineIndex = i;

        //if (cap == -1) { break; }
        //print("%s", stream.linesArray[i]);

        ++i;
        if (i == sizeof(stream.linesArray))
        {
            stream.linesArray = realloc(stream.linesArray, (2 * sizeof(stream.linesArray)));
        }
    }

    ull_int j;
    for (j = 0; j < i; ++j)
    {
        printf("%s\n", stream.linesArray[j]);
    }

    free(stream.linesArray);
    return 0;
}

最佳答案

好的，目的很明确-使用getdelim将行存储在数组中。 getline本身使用动态分配。 manual非常清楚：


  getline（）从流中读取整行，并存储
  包含文本到* lineptr的缓冲区。缓冲区是
  终止于null并包括换行符（如果已找到）。


getline()“将缓冲区的地址存储到* lineptr中”。因此，lineptr必须是指向char *变量的有效指针（读取两次）。


  * lineptr和* n将被更新
  分别反映缓冲区地址和分配的大小。


同样，n必须是指向size_t变量的有效指针（！），以便函数可以对其进行更新。

另请注意，lineptr缓冲区：


  即使getline（）失败，该缓冲区也应由用户程序释放。


那么我们该怎么办？我们需要有一个指向字符串数组的指针数组。因为我不喜欢成为三星级的程序员，所以我使用结构。我对您的代码做了一些修改，添加了一些检查。您好，我不喜欢typedef，所以我不使用它们。将uniq重命名为struct lines_s：

#define _GNU_SOURCE
#include <stdio.h>
#include <unistd.h>
#include <string.h>
#include <stdlib.h>

struct line_s {
    char *line;
    size_t len;
};

struct lines_s {
    struct line_s *lines;
    size_t cnt;
};


int main() {
    struct lines_s lines = { NULL, 0 };

    // loop breaks on error of feof(stdin)
    while (1) {

        char *line = NULL;
        size_t size = 0;
        // we pass a pointer to a `char*` variable
        // and a pointer to `size_t` variable
        // `getdelim` will update the variables inside it
        // the initial values are NULL and 0
        ssize_t ret = getdelim(&line, &size, '\n', stdin);
        if (ret < 0) {
            // check for EOF
            if (feof(stdin)) {
                // EOF found - break
                break;
            }
            fprintf(stderr, "getdelim error %zd!\n", ret);
            abort();
        }

        // new line was read - add it to out container "lines"
        // always handle realloc separately
        void *ptr = realloc(lines.lines, sizeof(*lines.lines) * (lines.cnt + 1));
        if (ptr == NULL) {
            // note that lines.lines is still a valid pointer here
            fprintf(stderr, "Out of memory\n");
            abort();
        }
        lines.lines = ptr;
        lines.lines[lines.cnt].line = line;
        lines.lines[lines.cnt].len = size;
        lines.cnt += 1;


        // break if the line is "stop"
        if (strcmp("stop\n", lines.lines[lines.cnt - 1].line) == 0) {
            break;
        }
    }

    // iterate over lines
    for (size_t i = 0; i < lines.cnt; ++i) {
        // note that the line has a newline in it
        // so no additional is needed in this printf
        printf("line %zu is %s", i, lines.lines[i].line);
    }

    // getdelim returns dynamically allocated strings
    // we need to free them
    for (size_t i = 0; i < lines.cnt; ++i) {
         free(lines.lines[i].line);
    }
    free(lines.lines);
}

对于此类输入：

 line1 line1
 line2 line2
 stop

将输出：

 line 0 is line1 line1
 line 1 is line2 line2
 line 2 is stop

在onlinegdb上测试。

笔记：


if (i == sizeof(stream.linesArray)) sizeof不会神奇地存储数组的大小。 sizeof(stream.linesArray)只是sizeof(char**)只是一个指针的sizeof。通常为4或8个字节，具体取决于32位还是64位体系结构。
uniq stream = { malloc(DEFAULT_SIZE * sizeof(char)),-stream.linesArray是char**变量。因此，如果要具有指向char的指针数组，则应为指针malloc(DEFAULT_SIZE * sizeof(char*))分配内存。
typedef unsigned long long int ull_int; size_t类型，如果该类型表示数组大小或sizeof（变量）。有时在posix api中使用ssize_t返回大小和错误状态。使用这些变量，无需键入unsigned long long。
ull_int cap cap = getdelim-上限是无符号的，永远不会是cap != 1。

关于c - 我该如何解决这个有趣的getdelim/getline(动态内存分配)错误？，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/55505716/