Question

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

    int main()

    {

        char file_name[100];
        char insert[100];

        printf("Enter the file name:");
        scanf("%s", file_name);
        FILE *f = fopen((strcat(file_name, ".txt")), "a");
        printf("Insert:");
        scanf ("%[^\n]%*s", insert);
        fprintf(f, "%s", insert);

    }

上述代码应该写入/插入数据到文件，但是，程序在打印后立即终止＆＃39;插入：＆＃39;。我似乎无法在这里找到问题。

Answer 1

scanf：

scanf("%s", file_name);

不会消耗输入文件名后按下的换行符。它保留在标准输入流（stdin）中。 scanf：

scanf ("%[^\n]%*s", insert);

扫描所有内容直到换行符，然后扫描并丢弃字符串。如果下一个字符为%[^\n] ， \n将失败。因此，scanf，在您的情况下失败并返回，因为下一个字符是\n。 insert仍然未初始化并且包含垃圾。

fprintf(f, "%s", insert);

将垃圾写入f。
<小时/> 如何解决问题？解决问题的两种方法：

在第一个getchar();之后使用scanf摆脱第一个scanf遗留下来的换行符。

使用

scanf (" %[^\n]%*s", insert);

而不是

scanf ("%[^\n]%*s", insert);

%[^\n]之前的空格扫描并丢弃所有whitespace characters（如\n，空格等），如果有的话，直到第一个非空格字符，如C11的引用中所示标准如下：


7.21.6.2 fscanf功能

[...]


由空白字符组成的指令通过读取第一个非空白字符（仍然未读取）的输入来执行，或者直到不再能够读取字符为止。该指令永远不会失败


我更喜欢第二种解决方案，因为它处理任何数量的空白字符。
<小时/> 这两种解决方案都有效，但是你有另一个问题。在为第二个scanf提供数据后，您的程序不会停止扫描！解决这个问题很容易。只需从第二个%*s中删除scanf。

想知道为什么？引用C11标准，


7.21.6.2 fscanf功能

[...]


转换说明符及其含义如下：
  [...]
  s匹配一系列非空白字符。 ²⁸⁶
  [...]


因此，%s将跳过前导空格字符并在遇到空格字符时停止扫描，或者，如果存在长度字段，则直到指定长度或直到空白字符 ，以先发生者为准。粗体部分解释了它。
<小时/> 为了提高安全性，请限制scanf为insert和file_name读取的字符数，以便您可以避免buffer overflows。另外，请检查scanf和fopen的返回值，看看它们是否成功。在您的情况下，scanf s如果成功则返回1。如果不是1，则发生错误。 fopen，失败时返回NULL。如果不是NULL，则fopen成功。

最终节目：

#include <stdio.h> #include <stdlib.h> #include <string.h> int main() { char file_name[100]; char insert[100]; printf("Enter the file name:"); if(scanf("%94s", file_name) != 1) //Scans a maximum of 94 chars,+1 for \0 at the end,+4 for ".txt" { printf("scanf failed\n"); exit(-1); //Exit the program with a negative return value } strcat(file_name, ".txt"); //Improves readability,do one thing in one line FILE *f = fopen(filename, "a"); if(f == NULL) //if fopen failed to open the filename, { printf("fopen failed\n"); exit(-1); } printf("Insert:"); if(scanf (" %99[^\n]", insert) != 1) // Note the space before %[^\n],99 scans a maximum of 99 chars,+1 for the \0 at the end { printf("scanf failed\n"); fclose(f); //Free resources before exiting exit(-1); } fprintf(f, "%s", insert); fclose(f); //Close the file after use }

<小时/> 如果您要丢弃第二个scanf留下的换行符，请使用

scanf (" %99[^\n]%*c", insert)

而不是

scanf (" %99[^\n]", insert)

但是这个程序并没有什么不同。
<小时/> 如果你想知道为什么我在“ 7.21.6.2 fscanf函数”章节中引用引号，那是因为fscanf与第一个参数的scanf完全相同它是stdin。再次引用C11标准，


7.21.6.4 scanf函数

[...]


scanf函数相当于fscanf，参数stdin插入scanf的参数之前。

Answer 2

麻烦在于你的第一个scanf。回车没有被处理，这意味着下一个scanf在被叫时会立即处理它，因为它仍在缓冲区中。

也许试试，scanf("%s\n", filename);

在我的“编写文件”中找不到错误。程序

2 个答案: