Question

作为练习的一部分，我在C中开发一个类似grep的小程序，它应该接收一个字符串和一些文件名，以便在引用的文件中查找该字符串。

我通过硬编码文件的名称和字符串来实现它：

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

int searchString(char *str1, char *str2) {
    if (strstr(str1, str2) == NULL) {
        return 0;
    } else {
        return 1;
    }
}

void searchInFile(char *filename, char *str) {
    FILE *f;
    int lineno =0;
    size_t len;
    char *buffer;
    f = fopen(filename, "r");
    if (f != NULL) {
        while (getline(&buffer, &len, f) != -1) {
            lineno++;
            if (searchString(buffer, str)) {
                printf("[%s:%d] %s", filename, lineno, buffer);
            } 
        }
    }
    fclose(f);
}

int main() {
    searchInFile("loremipsum.txt", "dolor");
    return 0;
}

但是一旦我修改代码以转换int main（）行以添加 argc 和 argv ，编译它然后运行它，脚本返回一个错误信息。修改是：

int main(int argc, char *argv[]) {
    searchInFile("loremipsum.txt", "dolor");
    return 0;
}

收到的错误：

*`./a.out'出错：realloc（）：无效指针：0x000056424999394d *

任何人都知道我为什么会收到此错误？

Answer 1

getline()的主要问题是在len = 0;作为buffer = NULL;参数传递buffer之前未能初始化lineptr和len n参数。 man 3 getline，解释说：

ssize_t getline(char **lineptr, size_t *n, FILE *stream); 
...
If *lineptr is set to NULL and *n is set 0 before the call, then 
getline() will allocate a buffer for storing the line."

除非您初始化lineptr的指针和n的值，否则这些将包含不确定（垃圾）值，这将阻止getline自动正确处理分配。

修复这些问题，提供一些小的清理，并启用文件名和searchterm作为程序的第一个和第二个参数，你可以做类似以下的事情：

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

/* consider adding parameter for case-insensitive search */
int searchstring (const char *str1, const char *str2)
{
    if (strstr (str1, str2) == NULL)
        return 0;
    else
        return 1;
}

int searchinfile (const char *filename, char *str)
{
    int lineno = 0;
    size_t len = 0;
    char *buffer = NULL;
    FILE *f = fopen (filename, "r");

    if (!f) {   /* validate file open for reading */
        fprintf (stderr, "error: file open failed '%s'.\n", filename);
        return 1;
    }

    /* read each line into buffer, search for searchstring */
    while (getline (&buffer, &len, f) != -1) {
        lineno++;
        if (searchstring (buffer, str))
            printf("[%s:%d] %s", filename, lineno, buffer);
    }
    fclose (f);     /* close file */

    return 0;
}

int main (int argc, char **argv) {

    if (argc < 3) { /* validate at least 2 arguments given */
        fprintf (stderr, "error: insufficient input.\n"
                "usage: %s <file> <searchstring>\n", argv[0]);
        return 1;
    }

    /* search file argv[1] for string argv[2] */
    if (searchinfile (argv[1], argv[2]))
        fprintf (stderr, "unable to search '%s'.\n", argv[1]);

    return 0;
}

示例输入文件

$ cat dat/captnjack.txt
This is a tale
Of Captain Jack Sparrow
A Pirate So Brave
On the Seven Seas.

示例使用/输出

$ /tmp/tmp-david/greplite dat/captnjack.txt Sea
[dat/captnjack.txt:4] On the Seven Seas.

注意代码中的注释 - 传递一个额外的参数来指示是否应该执行不区分大小写的搜索（然后转换行和搜索 - 这将是一个巧妙的技巧 - 在搜索之前使用小写的术语）。

仔细看看，如果您有其他问题，请告诉我。

realloc（）：使用argc和argv时指针错误

1 个答案: