0

基本クラスが次のメッセージ ハンドラーを作成しようとしています。

1ハンドラー基本クラス

適切なメッセージ タイプの適切なハンドラーを生成する 2-Handler ファクトリ

3-メッセージの基本ジェネリック クラス

彼らのコードは次のようなものです:

#include <map>
#include<iostream>
//Base Handler
template<class MSG>
class Handler
{
    MSG message;
public:
    Handler(MSG message):message(message){
    }

    virtual void handle() = 0;
    MSG getMessage()
    {
        return message;
    }
};

//Base Handler Factory
template<class MSG>
class HandlerFactory {
public:
    virtual Handler<MSG> * create(MSG & message) = 0;
};

//Base message
template<class T>
class Message
{
    T messageType;
public:
    T getMessageType()
    {
        return messageType;
    }

    void setMessageType(T messageType_)
    {
        messageType = messageType_;
    }
};

//次に、メッセージ タイプに基づいて、すべての基本クラスのサブクラスを記述します。

//my custom types
enum MessageType
{
    ANNOUNCE,
    KEY_SEND,
    KEY_REQUEST
};

//my first custom message format
class MyMessage_1 : public Message<MessageType>
{
    //...
};

//my first custom handler
class MyMessageHandler_1 : public Handler<MyMessage_1>
{

public:
    MyMessageHandler_1(MyMessage_1 &message_): Handler<MyMessage_1>(message_)
    {
    }
    void handle(){}
};

//my custom handler factory
class MyHandlerFactory : public HandlerFactory<Message<MessageType> > {
    Handler<Message<MessageType> > *value;
public:
    MyHandlerFactory(){};
    Handler<Message<MessageType>  > * create(Message<MessageType> & message){
        switch (message.getMessageType())
        {
        case ANNOUNCE:
            MyMessage_1 t1;
            value = new MyMessageHandler_1(t1);//error here
            break;
            //etc. etc.
        default:
            value = 0;
            break;
        };
        return value;
    };

};

//簡単にコンパイルできるように main を入れましょう

int main()
{

}

問題は、switch-case 句で、カスタム メッセージ クラスの 1 つのハンドラーのインスタンスを作成しようとすると、次のエラーが発生することです。

templateArgList.cpp: In member function ‘virtual Handler<Message<MessageType> >* MyHandlerFactory::create(Message<MessageType>&)’:
templateArgList.cpp:86:37: error: cannot convert ‘MyMessageHandler_1*’ to ‘Handler<Message<MessageType> >*’ in assignment

Handler<Message<MessageType> > * 私は次のようにキャストできるという印象を受けました :

MyMessageHandler_1-> Handler<MyMessage_1>
                             ^
                             |
                     `Message<MessageType>`  which finally gives me:
                                                `Handler<Message<MessageType>  >`  :P

私が間違っている？もちろん、私はなぜ上記のエラーが発生するのでしょうか :)) 理由と修正方法がわかりません。そのため、親切に私を助けていただければ幸いです。

あなたの親切な助けをどうもありがとう

4

1 に答える 1

1

これは、あなたが行ったことに類似した非常に単純なプログラムですが、理解できるようにネストされたテンプレートはありません。

#include <vector>

class A {
};

class B : public A {
};

int main() {

   std::vector<A>* va;
   va = new std::vector<B>;

}

そして実際、g ++はエラーを出します:

error: cannot convert ‘std::vector<B>*’ to ‘std::vector<A>*’ in assignment

これは明らかです。B は A から継承されますが、A のベクトルは B のベクトルと同じではありません。継承を利用するには、関連するオブジェクトへのポインタが必要です。例えば：

int main() {
   std::vector<A*> va(3);
   for (int i=0; i<3; ++i) {
      va[i] = new B;
   }
}

ここでのアナロジーは次のとおりです。

std::vector< > ----> Handler< >
A              ----> Message<MessageType>
B              ----> MyMessage_1

ところで、messageMyMessageHandler_1 と Handler<> の両方で名前付き変数を定義していることに気付きましたか? これにより、MyMessageHandler_1::message が Handler<>::message を非表示にします。これがあなたの望むものかどうかわかりません。

また、構築しているアプリケーションに適している可能性があるため、 Python の Twisted パッケージを調べることもできます。(Python を使用してもかまわない場合。)

質問: 「私のコードを変更するための提案はありますか?」

応答：

テンプレートを削除して、継承の力を楽しんでみます。MessageHandler クラスは、どちらも基本クラスであるため、オブジェクト (または参照またはポインター) を受け入れることができます。のHandlerFactorycreate もMessageオブジェクトを受け入れます。次に、型メンバー変数Messageを持つクラスに進み、内部でスイッチを使用して、作成する正しい派生クラスを決定できます。enum MessageTypeHandlerFactorHandler

NewHandler()または、列挙型の代わりに、" " 関数をに追加することで、継承をさらに活用できますMessage。これは、純粋な仮想でMessageあり、派生クラスで定義されます。この方法では、スイッチは必要ありません。各タイプのメッセージはHandler必要なものを認識しており、factor は単純に を呼び出しますmessage->NewHandler()。

...プロジェクトの方向性がわからないので、テンプレートを使用する必要があるかどうかを判断するのは少し難しいです。ただし、大まかな経験則として、(a) 異なる型に対して同等のコード ブロックを使用したい場合、および (b) 継承を使用してそれを達成できない場合は、テンプレートを使用することをお勧めします。std::vector<>は良い例です。 と の動作は同じですが、std::vector<int>とは共通ベースで関連付けられていないため、VectorI と VectorF のコードを書き直すのではなく、コンパイラはコードを書き直すように求められます。代わりは。std::vector<float>intfloat

これまでのところ、継承を利用してやりたいことを実行できるようです。他の人があなたのコードを読みやすくするという追加のボーナスもあります. :)

于 2013-05-08T04:13:29.467 に答える