1

コマンドパターンを実装したい。私は次のものを持っています：

public class State
{
    public int Number { get; set; }

    public void Execute(IAction action)
    {
        if (action.IsValid(this))
            action.Apply(this);
    }            
}

public interface IAction
{
    bool IsValid(State state);
    void Apply(State state);
}       

public class ActionSet5IfZero : IAction
{

    public bool IsValid(State state)
    {
        if (state.Number == 0)
            return true;
        else
            return false;
    }

    public void Apply(State state)
    {
        state.Number = 5;
    }
}

そしてプログラム：

static void Main(string[] args)
{
    State s = new State();
    s.Execute(new ActionSet5IfZero());
}

それは期待どおりに機能します。私の問題は、State クラスを拡張したいときに始まります。

public class ExtendedState : State
{
    public int Number2 { get; set; }
}

ここで、アクションは に変更を適用する必要がありますExtendedState。ExtendedStateそこで、パラメーターとして受け取る 2 つの追加関数を持つ拡張アクションを作成すると考えました。

public class ExtendedActionSet5IfZero : IAction
{

    public bool IsValid(State state)
    {
        throw new NotImplementedException();
    }

    public void Apply(State state)
    {
        throw new NotImplementedException();
    }

    public bool IsValid(ExtendedState state)
    {
        if (state.Number == 0 && state.Number2 == 0)
            return true;
        else
            return false;
    }

    public void Apply(ExtendedState state)
    {
        state.Number = 5;
        state.Number2 = 5;
    }
}

インターフェイスを実装する関数が冗長になるため、これは私がすでに気に入っていないことです。さらに、IAction ではなく、新しい型を使用する新しいExecute関数を myに作成する必要があります(そうしないと、実装されていない関数が呼び出されます)。ExtendedState

私はそれが素敵なオブジェクト指向の方法でできると確信しています。あなたは私を助けることができます？目的は、拡張可能な State クラスと IAction インターフェイスを作成することです (ジェネリックかもしれませんが、私にはわかりません) State。

4

3 に答える 3

1

状態に仮想SetNumberメソッドを追加できます

public class State 
{ 
    public int Number { get; set; } 

    public virtual void SetNumber(int n)
    { 
        Number = n;
    }

    public void Execute(IAction action) 
    { 
        if (action.IsValid(this)) 
            action.Apply(this); 
    }             
}

拡張状態では、それをオーバーライドします

public class ExtendedState : State  {
    public int Number2 { get; set; }

    public orverride void SetNumber(int n)
    { 
        base.SetNumber(n);
        Number2 = n;
    }
}

アクションは次のように実装されます

public void Apply(State state)        
{
    state.SetNumber(5);        
}

編集：

Number を配列として宣言するのはどうですか?

public class State  
{
    public int[] Numbers { get; private set; }

    public State()
    {
        Numbers = new int[1];
    }

   ...
}

次に、アクションはこれを行います

public void Apply(State state)         
{
    for (int i = 0; i < state.Numbers.Length; i++) {
        state.Numbers[i] = 5;
    }
}

ExtendedStatewouldのコンストラクターはで初期化Numbersします

Numbers = new int[2];

さらに、単一の数値のプロパティを持つことができます

public int Number { 
    get { return Numbers[0]; }
    set { Numbers[0] = value; }
}

と

public int Number2 { 
    get { return Numbers[1]; }
    set { Numbers[1] = value; }
}
于 2012-02-09T16:19:24.077 に答える
0

ジェネリックを使用できます：

interface IAction<TState> where TState: State
{
    bool IsValid(TState state);
    void Apply(TState state);
}
于 2012-02-09T16:09:50.127 に答える
0

StateContainer を State と Action に追加するのはどうですか:

public interface IStateContainer<TState, TAction> where TState : IState where TAction : IAction<TState> { 
    public TState State;
    public void Execute(TAction action);
}

public interface IState { }

public interface IAction<TState> where TState : IState {
    bool IsValid(TState state);
    void Apply(TState state);
}

次に、元のクラスを次のように置き換えることができます。

public class ValidatingStateContainer<TState, TAction> : IStateContainer<TState, TAction> {

    public ValidatingStateContainer(TState state) {
        State = state;
    }

    public TState State { get; private set; }

    public void Execute(TAction action)
    {
        if (action.IsValid(this))
            action.Apply(State);
    }
}

public class ActionSet5IfZero : IAction<NumberState>
{
    public boolean IsValid(NumberState state)
    {
        if (state.Number == 0)
            return true;
        else
            return false;
    }

    public void Apply(NumberState state)
    {
        state.Number = 5;
    }
}

public class ExtendedActionSet5IfZero : ActionSet5IfZero, IAction<TwoNumberState>
{   
    public boolean IsValid(TwoNumberState state)
    {
        if (base.IsValid(state) && state.Number2 == 0)
            return true;
        else
            return false;
    }

    public void Apply(TwoNumberState state)
    {
        base.Apply(state);
        state.Number2 = 5;
    }
}

public class NumberState : IState {
    public int Number { get; set; }
}

public class TwoNumberState : NumberState {
    public int Number2 { get; set; }
}
于 2012-02-09T16:16:07.317 に答える