コマンドパターンの醜い経験：OOPの任意の点？

Question

私は自分のシステムにコマンドパターンを実装しました。なぜなら、私はいくつかの層を持っており、ロジックを遠隔から呼び出す必要があるからです。

class DoWorkCommandMessage { int param; } 

class DoWorkCommandHandler : Handler<DoWorkCommandMessage> 
{ 
    Execute(MyObject object) { 
     object.DoWork(message.param); 
    } 
} 

class MyObject 
{ 
    void DoWork(int param) { 
     _proxy.SendMessage(new DoWorkBinaryMessage(param)); 
    } 
} 

あなたは、私は基本的に、別の階層に送信されるメッセージに戻ってそれを変換されたメソッド呼び出し、に変換、メッセージを取得しています見ることができるように。

ここに何か問題があるように感じます。

私はすべてのメソッドを削除するためにMyObjectをリファクタリングし、単純なProcessMessageメソッドに置き換えました。このメソッドは、メッセージを受け取り、それを翻訳し、ディスパッチします。

これは）私がProcessMessageを（呼び出して維持する必要がありますユニットテストのために... MyObjectには、主にちょうど変換コードではなく、「オブジェクト」は終わった以外

OKだった代わりに簡単なメソッド呼び出しを作ります。

「メッセージ&の変換」と「メッセージ - >方法 - >メッセージ」の対戦のこの戦いについて、私は考えています。明らかにメッセージとメソッドは非常に密接に関連しています。

Answer 1

私は、実際にコマンドパターンを使用していないという混乱や困難があると思います。あなたはそれをMessagingと混合します。あなたはあなたのコマンドの名前でこれを見ることさえできます... DoWorkCommandMessage。古典的なコマンドパターンでは、実際にはCommandオブジェクトによって処理が行われます。メッセージング手法では、メッセージはDTOとして渡され、ハンドラによって実行されます。

をさらに説明するために

は、私が使用するアプリケーションがある

AbstractCommand 
+ Execute() 
+ CanUndo() 
+ Undo() 
+ CanChain(cmd) 
+ Chain(cmd) 

そのクラスの具体的な例は次のとおりです。

MoveUnitCommand // a unit is a graphic piece on a game board 
+ UnitId 
+ Destination 
+ Execute() 
    { 
     units = Units.Find(UnitId) 
     prevCoords = units.Position 
     units.Position = Destination 
    } 
+ CanUndo() { return true; } 
+ Undo() 
    { 
     units = Units.Find(UnitId) 
     units.Position = prevCoords 
    } 
+ etc, etc 

コマンドサブクラスはただ、それをどうするかについては説明しません。なおシステム状態に実際に作用します。