尷尬的錯誤讀取字符串文字時進入一個指針

Question

我覺得半推遲發佈，但是我不知道爲什麼我的程序是吹起來時，它讀取一個字符串從鍵盤文字（即然後將其分配給一個指針）。

經過一個多小時的調試，程序在從鍵盤讀取數據時不斷爆炸。

我試過一切來解決這個問題。將字符串初始化爲字符串文字（即編譯器說它對nullptr有問題）。這幾乎就像我在某個地方有一個無形的角色。如果有人能告訴我我做錯了什麼，我將不勝感激。

的main.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)之前我沒有看到您分配fileName。

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;