将字符串文字读入指针时出现令人尴尬的错误

Question

我觉得这个帖子有点迟钝，但我不知道为什么我的程序在从键盘读取字符串文字时会爆炸（即，然后将其分配给指针）。

调试了一个多小时，程序从键盘读取时一直在爆炸。

我已尽一切努力解决此问题。将字符串初始化为字符串文字（即编译器说它有 nullptr 问题）。这几乎就像我在某个地方有一个看不见的角色。如果有人能告诉我我做错了什么，我将不胜感激。

ma​​in.cpp

#define _CRTDBG_MAP_ALLOC
#include <crtdbg.h>

#include <iostream>
#include <fstream>
#include "protocol.h"

int main()
{
_CrtSetDbgFlag(_CRTDBG_ALLOC_MEM_DF | _CRTDBG_LEAK_CHECK_DF);

int menuChoice = 0;

char * fileName = nullptr;
char * byteArray = nullptr;
char * hexArray = nullptr;
int numberOfBytes = 0;

PrintMenu();
GetMenuChoice(menuChoice);
ExecuteMenuChoice(menuChoice, fileName, byteArray, hexArray, numberOfBytes);

return 0;
}

protocol.cpp

void GetFile(char * fileName)
{

//Prompt user for binary file
std::cout << "\nEnter filename: " << std::endl;

//Read in location of binary file
std::cin.ignore(std::cin.rdbuf()->in_avail());
std::cin.getline(fileName, 256);
std::cin.clear();
std::cin.ignore(std::cin.rdbuf()->in_avail());
}

protocol.h

#ifndef PROTOCOL_H
#define PROTOCOL_H

//Function declarations
void PrintMenu();
void GetMenuChoice(int &menuChoice);
void ExecuteMenuChoice(int menuChoice, char *& fileName, char *& byteArray,
                   char *& hexArray, int numberOfBytes);
void NewLine();
void ThankUser();
void ErrorMessage();

#endif

Answer 1

您发布的代码不完整，但我没有看到您在调用 std::cin.getline(fileName, 256) 之前分配文件名。

Answer 2

我没有看到您在任何地方调用 GetFile，但它看起来需要分配的缓冲区或字符数组。在main() 中，您声明了char *fileName，但不为其分配任何内存。如果您致电（来自main()）GetFile(fileName)，那么我预计它会崩溃。您需要在main() 或GetFile() 中分配空间来读取数据。

Answer 3

传递要填充的字符串文字没有任何意义。你需要传递一个缓冲区：

#include <iostream>
#include <fstream>
#include <stdlib.h> // for _MAX_PATH

int main()
{
    char filename[_MAX_PATH];

    GetFile(filename);

    std::cout << filename << std::endl;
}

_MAX_PATH 比硬编码像256 这样的任意值更有意义。

最好还是使用std::string 并完全删除这些缓冲区大小！

#include <iostream>

void GetFile(std::string& fileName)
{
    std::cout << "\nEnter filename: " << std::endl;

    std::getline(std::cin, fileName);
}

int main()
{
    std::string filename;

    GetFile(filename);

    std::cout << filename << std::endl;
}

Answer 4

cin::getline() 不会为您分配内存来将字符串文字存储到fileName 中。你必须为它提供内存空间，通过分配内存给fileName，就像

fileName = new char[_MAX_PATH];

最后别忘了释放内存

delete[] fileName;