Question

老师要求我们这样做：

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

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

其他问题

fopen()内的switch调用是多余的：该文件之前已经打开过。没有关闭之前打开文件是没有用的。

函数文档说明了最低和最高有效位，但没有关于中间位的信息。所以检查应该是：

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

访问冲突读取位置fprintf

1 个答案: