c++ - 访问冲突读取位置 fprintf

Question

所以这是老师要求我们做的:

创建一个文件来存储用户输入的内容。我遇到的麻烦是抛出异常并且没有给我任何错误。第一个问题是文本文件中的字符默认为大写(没有大写验证)。第二个错误在我按下一个键时发生(如果有上限验证)。

#include <direct.h>
#include <fstream>
#include <stdlib.h>


using namespace std;

//function that write to the file
int Save(int _key, char *file, bool caps);
int Backspace();
bool caps = false;

int main() {
    char i;

    while (true) {
        for (i = 8; i <= 255; i++) {
            // Checks if a key is currently up or down
            if (GetAsyncKeyState(i) == -32767) {
                //Get caps lock state
                if ((GetKeyState(VK_CAPITAL) & 0x0001) != 0) {
                    //Caps lock is on
                    caps = true;
                    Save(i, "C:/users/myComputer/documents/school/c++/input.txt", caps);
                }
                else {
                    //Caps lock is off
                    caps = false;
                    Save(i, "C:/users/myComputer/documents/school/c++/input.txt", caps);
                }
            }
        }
    }
    return 0;
}

int Save(int _key, char *file, bool caps) {
    Sleep(10);
    FILE *OUTPUT_FILE;
    OUTPUT_FILE = fopen(file, "a+");

 switch (_key) {

     case VK_RETURN:
         OUTPUT_FILE = fopen(file, "a+");
         fprintf(OUTPUT_FILE, "%s", "\n");
         break;

     case VK_LBUTTON || VK_RBUTTON:
         OUTPUT_FILE = fopen(file, "a+");
         fprintf(OUTPUT_FILE, "%s", "");
         break;

     default:
         if (caps == true) {
             char i = tolower(putchar(_key));
             OUTPUT_FILE = fopen(file, "a+");
             fprintf(OUTPUT_FILE, "%s", i);
         }
         else {
             char i = toupper(putchar(_key));
             OUTPUT_FILE = fopen(file, "a+");
             fprintf(OUTPUT_FILE, "%s", i);
         }
 }
 fclose(OUTPUT_FILE);

 return 0;
}

Answer 1

段错误

这段代码有几个问题。但导致分割错误的主要问题是

        fprintf(OUTPUT_FILE, "%s", i);   // Ouch !!!

"%s" 告诉 fprintf() 它将找到一个指向字符数组的指针作为参数。交叉检查您的格式字符串 here .不幸的是，您提供了一个 8 到 255 之间的整数。这会导致未定义的行为。

要更正错误，请将其更改为:

        fprintf(OUTPUT_FILE, "%c", i);

其他问题

switch 中的 fopen() 调用是多余的:文件之前已经打开过。之前没有关闭文件就打开它是没有用的。

函数文档说明了最低和最高有效位，但没有说明中间位。所以检查应该是:

 if (GetAsyncKeyState(i) & 0x8000 )   // but not strict equality