Question

我尝试编写文件管理器的c ++程序。我写了一个“CommandEngine”类，它处理命令，一个“Command”类，它是抽象类，包含“执行”函数等。我尝试在指定的路径上创建一个文件（没有在该级别的开发中给出响应）我的节目）。我编写的代码编译成功，但是当我尝试执行它时，我遇到了错误

“FileManager2.exe中0x0022DC96处的未处理异常：0xC0000005：访问冲突读取地址0x00000014。”

我将非常感谢任何帮助。感谢大家。

// FileManager.cpp



#include <stdio.h>
#include <tchar.h>
#include "CommandEngine.h"
#include "Command.h"
#include "CreateFile.h"
using namespace std;

int _tmain(int argc, _TCHAR* argv[])
{
    CommandEngine C;
    C.CommandHandler();
    return 0;


}



// CommandEngine.h


#ifndef COMMANDENGINE_H
#define COMMANDENGINE_H

#include "Command.h"
#include "CreateFile.h"
#include "Input.h"
#include <map> 
#include <string>
using namespace std; 



class CommandEngine
{




public:

    typedef map< string , Command * >  MapOfHandlers;
    MapOfHandlers CommandHandlers;

    Input * input;

    Command * GetCommand(const string & commandName)
    {
        map< string , Command * >::iterator iter;
        iter = CommandHandlers.find(commandName);
        return iter->second;
    };

    void CommandHandler();

    CommandEngine();
    ~CommandEngine();



};

CommandEngine::CommandEngine()
{
    CreateFileCl * Cr;
    string s = "create";
    CommandHandlers.insert(pair<string, Command *>(s,  Cr));



}

CommandEngine::~CommandEngine()
{
}



void CommandEngine::CommandHandler()
{


    Response response;

    Command * command = GetCommand( ( input->ReadInput() ) -> GetCommandName());
    command->Execute(input, &response);

    WriteResponse(&response);
}


#endif // COMMANDENGINE_H



//Command.h

#ifndef COMMAND_H
#define COMMAND_H 






#include "Input.h"
#include "Response.h"
using namespace std;

class Command
{   public:
    virtual void Execute(Input * input, Response * response ) = 0;
};

#endif // Command_H



/*void StrToChar(string s)
{
    string s;
string writable = new char[str.size() + 1];
std::copy(str.begin(), str.end(), writable);
writable[str.size()] = '\0'; // don't forget the terminating 0

// don't forget to free the string after finished using it
delete[] writable;

} */


////////////////////////////////////////////////////////////////////////////////////////////////////

// Input.h


#ifndef INPUT_H
#define INPUT_H


#include <string>
#include <iostream>
using namespace std;

class Input
{

public:
    string CommandName;
    string FileName;
    string DestinationPath; // For " copy " command


    Input * ReadInput();

    const string  GetCommandName()// can be no useful
    {
        return CommandName;

    };

    Input();
    ~Input();



};

Input::Input()
{
    CommandName = FileName = DestinationPath = " ";
}

Input::~Input()
{

}


Input * Input::ReadInput()
{
    cout<<"Enter command";


    getline(cin,CommandName);
    getline(cin, FileName);
    getline(cin,DestinationPath );
    return this;

}

#endif // INPUT_H



// CreateFile.h


#ifndef CREATEFILE_H
#define CREATEFILE_H

#include <windows.h>
#include "Command.h"


using namespace std;

class CreateFileCl : public Command
{

public:


    virtual void Execute(Input * input, Response * response );


};

void CreateFileCl::Execute(Input * input, Response * response )
{
    /*const string text = (input->FileName).c_str();
 wchar_t wtext[20];
 mbstowcs(wtext, text, strlen(text)+1);//Plus null
 LPWSTR ptr = wtext; */


    CreateFileA( (input->FileName).c_str(), 0, 0, 0, 0, 0, 0);

}


#endif // CREATEFILE_H

Answer 1

在CommandEngine构造函数中，您在CreateFileCl地图中存储了未初始化的CommandHandlers指针。当您稍后尝试通过该指针调用Command::Execute时，会发生可怕的事情（未定义的行为）。

Answer 2

您有许多未初始化的对象可能会导致代码崩溃。

首先，您不要在CommandEngine类中初始化 input ，因此当您调用input->ReadInput()时，您正在使用未初始化的指针。

其次，正如Casey所说，你没有初始化你在 CommandHandlers 列表中插入的 CreateFileCl 对象，所以当你尝试执行命令时也会失败。

您可以通过更新 CommandEngine 构造函数来初始化这两个对象来解决此问题。

CommandEngine::CommandEngine()
{
    input = new Input();
    CreateFileCl * Cr = new CreateFileCl();
    string s = "create";
    CommandHandlers.insert(pair<string, Command *>(s,  Cr));
}

请注意，您还需要删除 CommandEngine 析构函数中的输入对象。

CommandEngine::~CommandEngine()
{
    delete input;
}

释放 CreateFileCl 对象的内存更复杂 - 假设可能有多个命令处理程序，因此您需要浏览列表并删除所有这些内容。但实际上你不应该像这样分配内存。理想情况下，您应该使用smart pointers来处理内存管理。

文件管理器的c ++程序编译，但不起作用

2 个答案: