c - C-使用fgets直到换行符/-1

Question

Closed. This question is off-topic. It is not currently accepting answers. Learn more。
想改进这个问题吗？ Update the question所以堆栈溢出的值小于aa>。
去年关门了。
所以我试着把它写下来，这样你就可以把文本写进一个文件，直到你换行或者输入-1。我的问题是，当你写的时候，它会一直运行，直到崩溃并给出错误“Stack around the variable”inputChoice“was corrupted”。
我相信问题在于当你想停止输入时程序不会停止接受stdin（-1，newline），这会导致错误。我试过用一个简单的扫描法，但你只能写一个字。没有空格，也不支持多行。所以我要用fgets

最佳答案

从你的评论来看，我认为C语言中有一些基本概念
你还没有完全理解。
C字符串
C字符串是字节序列。此序列必须以值0结尾。
序列中的每个值都表示基于
ASCII编码，例如
字符'a'是97，'b'是98，等等
值0是决定字符串结尾的字符。
这就是为什么你经常听到C字符串以“\0”结尾的原因。
在C语言中，使用一个字符数组（'\0'，char string[]）来
保存字符串。对于长度为n的字符串，需要维度为n+1的数组，因为
您还需要一个空格作为结束字符。
在处理字符串时，总是要考虑合适的类型，
无论您使用的是数组还是指针。指针
to char并不一定意味着你在处理一个C字符串！
我为什么要告诉你这些？因为：

char inputChoice = 0;

printf("Do you wish to save the Input? (Y/N)\n");
scanf("%s", &inputChoice);

我没有太大的变化，在尝试了一段时间后变得非常沮丧。
我把 char string[SOME VALUE]改成 '\0'的
似乎已经阻止了程序崩溃。
这表明他们还没有理解 %s和 %c之间的区别。
scanf(" %c, &inputChoice)转换说明符告诉 %s它必须匹配单个字符，并且它需要指向 %c的指针。
人少
c类
匹配由最大字段指定长度的字符序列。
宽度（默认值1）；下一个指针必须是
指向char的指针，必须有足够的空间容纳所有字符
（不添加终止空字节）。通常的跳跃
前导空白被抑制。要先跳过空白，请在格式中使用显式空格。
别提长度了，现在不重要了。
重要的部分是粗体的。对于格式 %c，函数
需要一个指向 scanf的指针，并且它不会写入终止 char
字符，它不是C字符串。如果你想读一封又一封的信
仅限信函：

char c;
scanf("%c", &c);

// also possible, but only the first char
// will have a defined value
char c[10];
scanf("%c", c);

第一个很容易理解。第二个更有趣：这里
你有一个10维的 scanf("%c"数组（即它包含10 chars）。 '\0'
将匹配一个字母并将其写在 char上。但是结果不会是
一个C字符串，不能传递给 char或其他期望的函数
C字符串（如 scanf）。
c[0]转换说明符字符告诉 puts它必须匹配非空白字符序列
人少
s公司
匹配一系列非空白字符；下一个指针必须是
指向字符数组的初始元素的指针，该元素的长度足以
保留输入序列和终止空字节（ strcpy），添加
自动。
这里的结果是保存了一个C字符串。你也要有足够的
保存字符串的空间：

char string[10];
scanf("%s", string);

如果字符串匹配9个或更少的字符，一切都会好的，因为
对于长度为9的字符串，需要10个空格（不要忘记终止
%s）。如果字符串与9个以上的字符匹配，您将没有足够的
缓冲区中出现空间和缓冲区溢出（访问超出大小）。
这是一种未定义的行为，任何事情都可能发生：您的程序可能
崩溃，程序可能不会崩溃，但会重写另一个变量，因此
你的程序流，它甚至可能在某个地方杀死一只小猫，做
你真的想杀小猫？
那么，你明白为什么你的代码错了吗？

char inputChoice = 0;
scanf("%s", &inputChoice);

scanf是一个 '\0'变量，它只能保存一个值。
'\0'为您提供了 inputChoice变量的地址，但是
之后的字符超出了限制，如果您读/写它，您将有一个
溢出，所以你杀了一只小猫。即使你只输入一个字符，它也会
写至少2个字节，因为你只有一个字符的空间，小猫就会死。
所以，让我们谈谈你的代码。
从用户的角度来看：我为什么要输入几行文本，可能是很多行文本
然后回答“不，我不想保存这些行”。没有道理
我。
在我看来，您应该首先询问用户是否要保存
先输入，然后请求输入。如果用户不想保存
任何东西，那么要求用户在
全部但这只是我的看法。
如果你真的想坚持你的计划，那么你必须把每一行都存起来
当用户结束输入数据时，您询问并保存文件。

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

#define BUFFERLEN 1024


void printFile () {
    int i;
    char openFile[BUFFERLEN];
    FILE *file;

    printf("What file do you wish to write in?\n");
    scanf("%s", openFile);
    getchar();

    file = fopen(openFile, "w");
    if (file == NULL) {
        printf("Could not open file.\n");
        return;
    }

    // we save here all lines to be saved
    char **lines = NULL;
    int num_of_lines = 0;

    char buffer[BUFFERLEN];
    printf("Enter an empty line of -1 to end input\n");

    // for simplicity, we assume that no line will be
    // larger than BUFFERLEN - 1 chars
    while(fgets(buffer, sizeof buffer, stdin))
    {
        // we should check if the last character is \n,
        // if not, buffer was not large enough for the line
        // or the stream closed. For simplicity, I will ignore
        // these cases
        int len = strlen(buffer);
        if(buffer[len - 1] == '\n')
            buffer[len - 1] = '\0';


        if(strcmp(buffer, "") == 0 || strcmp(buffer, "-1") == 0)
            break; // either an empty line or user entered "-1"

        char *line = strdup(buffer);

        if(line == NULL)
            break; // if no more memory
                   // process all lines that already have been entered


        char **tmp = realloc(lines, (num_of_lines+1) * sizeof *tmp);

        if(tmp == NULL)
        {
            free(line);
            break; // same reason as for strdup failing
        }

        lines = tmp;

        lines[num_of_lines++] = line;  // save the line and increase num_of_lines
    }

    char inputChoice = 0;

    printf("Do you wish to save the Input? (Y/N)\n");
    scanf("%c", &inputChoice);
    getchar();

    if (inputChoice == 'Y' || inputChoice == 'y') {

        for(i = 0; i < num_of_lines; ++i)
            fprintf(file, "%s\n", lines[i]); // writing every line

        printf("Your file has been saved\n");
        printf("Please press any key to continue");
        getchar();
    }

    // closing FILE buffer
    fclose(file);

    // free memory
    if(num_of_lines)
    {
        for(i = 0; i < num_of_lines; ++i)
            free(lines[i]);
        free(lines);
    }

}

int main(void)
{
    printFile();
    return 0;
}

关于守则的评论
我用了和你相同的代码作为我的基础，这样你就可以发现
差异要快得多。
我使用宏 char来声明缓冲区的长度。那是
我的风格。
查看 &inputChoice行：

fgets(buffer, sizeof buffer, stdin)

我在这里使用 inputChoice而不是1024或 BUFFERLEN。再说一次，那是我的
风格，但我认为这样做更好，因为即使你改变了尺寸
通过更改宏或使用另一个显式大小， fgets
将始终返回正确的大小。请注意，只有当
sizeof buffer是一个数组。
函数返回指向新字符串的指针
重复参数。它用于创建字符串的新副本。什么时候？
使用此函数时，不要忘记必须使用
BUFFERLEN。 sizeof buffer不是标准库的一部分，它符合
至SVr4，4.3BSD，POSIX.1-2001。如果你使用Windows（我不使用Windows，
我不熟悉Windows生态系统），这个功能可能不是
现在。在这种情况下，你可以自己写：

char *strdup(const char *s)
{
    char *str = malloc(strlen(s) + 1);
    if(str == NULL)
        return NULL;
    strcpy(str, s);
    return str;
}

关于c - C-使用fgets直到换行符/-1 ，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/48102728/

上一篇：c - 重新定义 C 错误的不兼容类型

下一篇：循环头依赖

相关文章：

c++ - 将库添加到 C/c++ eclipse indigo 项目

java - 使用 While 循环进行加法

java - 多文件输入，用stdin//Argparse4j，Java

haskell - 在haskell中从stdin懒惰地读取和操作 float

c - 使用 fgetc(stdin) 时未定义的行为

c - 为什么我的 C 程序跳过 if 语句？

c - Scanf 用于多个用户输入

c - 父进程在无限循环中杀死子进程

c++ - 我尝试用 C++ 编写自己的简单移动平均线

c - while 循环中的 scanf 函数

Answer 1

从你的评论来看，我认为C语言中有一些基本概念
你还没有完全理解。
C字符串
C字符串是字节序列。此序列必须以值0结尾。
序列中的每个值都表示基于
ASCII编码，例如
字符'a'是97，'b'是98，等等
值0是决定字符串结尾的字符。
这就是为什么你经常听到C字符串以“\0”结尾的原因。
在C语言中，使用一个字符数组（'\0'，char string[]）来
保存字符串。对于长度为n的字符串，需要维度为n+1的数组，因为
您还需要一个空格作为结束字符。
在处理字符串时，总是要考虑合适的类型，
无论您使用的是数组还是指针。指针
to char并不一定意味着你在处理一个C字符串！
我为什么要告诉你这些？因为：

char inputChoice = 0;

printf("Do you wish to save the Input? (Y/N)\n");
scanf("%s", &inputChoice);

我没有太大的变化，在尝试了一段时间后变得非常沮丧。
我把 char string[SOME VALUE]改成 '\0'的
似乎已经阻止了程序崩溃。
这表明他们还没有理解 %s和 %c之间的区别。
scanf(" %c, &inputChoice)转换说明符告诉 %s它必须匹配单个字符，并且它需要指向 %c的指针。
人少
c类
匹配由最大字段指定长度的字符序列。
宽度（默认值1）；下一个指针必须是
指向char的指针，必须有足够的空间容纳所有字符
（不添加终止空字节）。通常的跳跃
前导空白被抑制。要先跳过空白，请在格式中使用显式空格。
别提长度了，现在不重要了。
重要的部分是粗体的。对于格式 %c，函数
需要一个指向 scanf的指针，并且它不会写入终止 char
字符，它不是C字符串。如果你想读一封又一封的信
仅限信函：

char c;
scanf("%c", &c);

// also possible, but only the first char
// will have a defined value
char c[10];
scanf("%c", c);

第一个很容易理解。第二个更有趣：这里
你有一个10维的 scanf("%c"数组（即它包含10 chars）。 '\0'
将匹配一个字母并将其写在 char上。但是结果不会是
一个C字符串，不能传递给 char或其他期望的函数
C字符串（如 scanf）。
c[0]转换说明符字符告诉 puts它必须匹配非空白字符序列
人少
s公司
匹配一系列非空白字符；下一个指针必须是
指向字符数组的初始元素的指针，该元素的长度足以
保留输入序列和终止空字节（ strcpy），添加
自动。
这里的结果是保存了一个C字符串。你也要有足够的
保存字符串的空间：

char string[10];
scanf("%s", string);

如果字符串匹配9个或更少的字符，一切都会好的，因为
对于长度为9的字符串，需要10个空格（不要忘记终止
%s）。如果字符串与9个以上的字符匹配，您将没有足够的
缓冲区中出现空间和缓冲区溢出（访问超出大小）。
这是一种未定义的行为，任何事情都可能发生：您的程序可能
崩溃，程序可能不会崩溃，但会重写另一个变量，因此
你的程序流，它甚至可能在某个地方杀死一只小猫，做
你真的想杀小猫？
那么，你明白为什么你的代码错了吗？

char inputChoice = 0;
scanf("%s", &inputChoice);

scanf是一个 '\0'变量，它只能保存一个值。
'\0'为您提供了 inputChoice变量的地址，但是
之后的字符超出了限制，如果您读/写它，您将有一个
溢出，所以你杀了一只小猫。即使你只输入一个字符，它也会
写至少2个字节，因为你只有一个字符的空间，小猫就会死。
所以，让我们谈谈你的代码。
从用户的角度来看：我为什么要输入几行文本，可能是很多行文本
然后回答“不，我不想保存这些行”。没有道理
我。
在我看来，您应该首先询问用户是否要保存
先输入，然后请求输入。如果用户不想保存
任何东西，那么要求用户在
全部但这只是我的看法。
如果你真的想坚持你的计划，那么你必须把每一行都存起来
当用户结束输入数据时，您询问并保存文件。

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

#define BUFFERLEN 1024


void printFile () {
    int i;
    char openFile[BUFFERLEN];
    FILE *file;

    printf("What file do you wish to write in?\n");
    scanf("%s", openFile);
    getchar();

    file = fopen(openFile, "w");
    if (file == NULL) {
        printf("Could not open file.\n");
        return;
    }

    // we save here all lines to be saved
    char **lines = NULL;
    int num_of_lines = 0;

    char buffer[BUFFERLEN];
    printf("Enter an empty line of -1 to end input\n");

    // for simplicity, we assume that no line will be
    // larger than BUFFERLEN - 1 chars
    while(fgets(buffer, sizeof buffer, stdin))
    {
        // we should check if the last character is \n,
        // if not, buffer was not large enough for the line
        // or the stream closed. For simplicity, I will ignore
        // these cases
        int len = strlen(buffer);
        if(buffer[len - 1] == '\n')
            buffer[len - 1] = '\0';


        if(strcmp(buffer, "") == 0 || strcmp(buffer, "-1") == 0)
            break; // either an empty line or user entered "-1"

        char *line = strdup(buffer);

        if(line == NULL)
            break; // if no more memory
                   // process all lines that already have been entered


        char **tmp = realloc(lines, (num_of_lines+1) * sizeof *tmp);

        if(tmp == NULL)
        {
            free(line);
            break; // same reason as for strdup failing
        }

        lines = tmp;

        lines[num_of_lines++] = line;  // save the line and increase num_of_lines
    }

    char inputChoice = 0;

    printf("Do you wish to save the Input? (Y/N)\n");
    scanf("%c", &inputChoice);
    getchar();

    if (inputChoice == 'Y' || inputChoice == 'y') {

        for(i = 0; i < num_of_lines; ++i)
            fprintf(file, "%s\n", lines[i]); // writing every line

        printf("Your file has been saved\n");
        printf("Please press any key to continue");
        getchar();
    }

    // closing FILE buffer
    fclose(file);

    // free memory
    if(num_of_lines)
    {
        for(i = 0; i < num_of_lines; ++i)
            free(lines[i]);
        free(lines);
    }

}

int main(void)
{
    printFile();
    return 0;
}

关于守则的评论
我用了和你相同的代码作为我的基础，这样你就可以发现
差异要快得多。
我使用宏 char来声明缓冲区的长度。那是
我的风格。
查看 &inputChoice行：

fgets(buffer, sizeof buffer, stdin)

我在这里使用 inputChoice而不是1024或 BUFFERLEN。再说一次，那是我的
风格，但我认为这样做更好，因为即使你改变了尺寸
通过更改宏或使用另一个显式大小， fgets
将始终返回正确的大小。请注意，只有当
sizeof buffer是一个数组。
函数返回指向新字符串的指针
重复参数。它用于创建字符串的新副本。什么时候？
使用此函数时，不要忘记必须使用
BUFFERLEN。 sizeof buffer不是标准库的一部分，它符合
至SVr4，4.3BSD，POSIX.1-2001。如果你使用Windows（我不使用Windows，
我不熟悉Windows生态系统），这个功能可能不是
现在。在这种情况下，你可以自己写：

char *strdup(const char *s)
{
    char *str = malloc(strlen(s) + 1);
    if(str == NULL)
        return NULL;
    strcpy(str, s);
    return str;
}