访问冲突读取位置 fprintf答案

【问题标题】：Access violation reading location fprintf访问冲突读取位置 fprintf
【发布时间】：2017-01-09 21:32:13
【问题描述】：

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

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

#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;
}

【问题讨论】：

你是通过命令行运行的吗？节目刚刚结束？你确定有是错误吗？如果 if (GetAsyncKeyState(i) == -32767) 永远不会评估为 true，那么您的程序将很快退出，而不会向控制台打印任何内容。
我没试过，但是你可以fopen同一个文件两次吗？看起来您的 Save 函数在开始时打开文件，然后在每个分支中再次打开。
@dckuehn 是的，通过 Visual Studio 中的调试器。 Visual Studio 让我休息一下。在我尝试实现 CapsLock 功能之前，它工作得很好。
@AndrewPiliser 不应该是因为我的变量刚刚被覆盖。但它的冗余确实我需要解决这个问题。

标签： c++ keyboard char int printf

【解决方案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

【讨论】：