0

ファイルマネージャの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
4

2 に答える 2

1

コードがクラッシュする可能性がある初期化されていないオブジェクトが多数あります。

まず、 CommandEngine クラスで入力を初期化しないため、呼び出すときに初期化されinput->ReadInput()ていないポインターを使用しています。

第二に、Casey が既に述べたように、 CommandHandlersリストに挿入するCreateFileClオブジェクトを初期化しないため、コマンドを実行しようとすると、それも失敗します。

CommandEngineコンストラクターを更新してこれら 2 つのオブジェクトを初期化することで、これを修正できます。

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オブジェクトのメモリを解放するのはもっと複雑です。おそらく複数のコマンド ハンドラが存在する可能性があるため、リストを調べてそれらをすべて削除する必要があります。しかし、実際には、このようにメモリを割り当てるべきではありません。理想的には、メモリ管理を処理するスマート ポインターを使用する必要があります。

于 2013-07-19T19:05:07.823 に答える
1

コンストラクターでは、初期化されていないポインターをマップ内にCommandEngine格納しています。後でそのポインターを介して呼び出そうとすると、ひどいこと (未定義の動作) が発生します。CreateFileClCommandHandlersCommand::Execute

于 2013-07-19T18:45:07.353 に答える