更改顺序修复了我的错误。为什么？

Question

我花了一段时间才弄清楚这个错误，并且想知道为什么第二个代码块不起作用。

工作:

FILE *readFile;
FILE *saveFile;

char readFileName;
char saveFileName;

printf("read file name:\n");
scanf("%s", &readFileName);
readFile = fopen(&readFileName, "r");

printf("save file name:\n");
scanf("%s", &saveFileName);
saveFile = fopen(&saveFileName, "w");

没用:

FILE *readFile;
FILE *saveFile;

char readFileName;
char saveFileName;

printf("read file name:\n");
scanf("%s", &readFileName);

printf("save file name:\n");
scanf("%s", &saveFileName);

readFile = fopen(&readFileName, "r");
saveFile = fopen(&saveFileName, "w");

Answer 1

您正在获取对分配在堆栈上的单个 char 的引用。该指针无效，不能用作对字符数组的引用。

尝试用用作缓冲区的真实字符数组替换您的字符串:

char readFileName[128];
scanf("%127s", readFileName);

否则会发生的是 scanf，它不检查任何东西，将用从 stdio 获取的字符覆盖堆栈上的数据，从而导致 stack buffer overflow ，这意味着未定义的行为。

您可以在格式说明符中指定最多获取多少个字符，但请记住减去一个，因为 null terminator由 scanf 本身自动附加。