.NETのインターフェイス継承/階層 - より良い方法がありますか？

Question

私は、私が意味するものの例として、私のコードで何かの事例研究を使用します。

今、コンポーネントベースのゲームシステムの動作/アクションシステムに取り組んでいます。ゲームオブジェクトにのみ関連する詳細情報を示し、IBehaviorComponent及びそれらに添付IActionComponentを持つことができ、以下の公開：

public interface IBehaviorComponent : IBaseComponent 
{ 
    IBehavior Behavior { get; } 
} 

public interface IActionComponent : IBaseComponent 
{ 
    IAction Action { get; } 
    void ExecuteAction(IGameObject target = null); 
} 

は今、これはこれまでのところ、すべての罰金です（少なくとも私には！）。しかし、IActionComponentの実装を見ると、問題が発生し始めます。例えば

、シンプルIActionComponent実装：

public class SimpleActionComponent : IActionComponent 
{ 
    public IAction Action { get; protected set; } 

    public void ExecuteAction(IGameObject target = null) 
    { 
     Action.Execute(target); 
    } 

    public void Update(float deltaTime) { } //from IBaseComponent 
} 

しかし、のは、私はアクションは、時限スケジュール通りに実行されるようにすることができます、より複雑なIActionComponentの実装を紹介したいとしましょう：

public class TimedActionComponent : IActionComponent 
{ 
    public IAction Action { get; protected set; } 

    public float IdleTime { get; set; } 

    public float IdleTimeRemaining { get; protected set; } 

    public void ExecuteAction(IGameObject target = null) 
    { 
     IdleTimeRemaining = IdleTime; 
    } 

    public void Update(float deltaTime) 
    { 
     if (IdleTimeRemaining > 0f) 
     { 
     IdleTimeRemaining -= deltaTime; 
     } 
     else 
     { 
     Action.Execute(target); 
     } 
    } 
} 

今、IdleTimeを外部の影響によって変更できるように公開したいとします。

public interface ITimedActionComponent : IActionComponent 
{ 
    float IdleTime { get; set; } 

    float IdleTimeRemaining { get; set; } 
} 

しかし、ここでの問題は私のコンポーネントシステムはIBaseComponentから1レベルアップ時にすべてを保存することです：最初に私の考えは、新しいインターフェイスを作成していました。そのため、GameObjectのアクションコンポーネントは、ITimedActionComponent、IRandomizedActionComponent、ICrashTheProgramActionComponentなど、IActionComponentとして取得されます。基本的なタイプのコンポーネント（IActionComponent、IRenderableComponent、IPhysicsComponentなど）を超えて、必要なものを正確に知らなくても、そのコンポーネントの1つに対してGameObjectを照会することができるようにするために、理由は明らかです。

これを処理するよりクリーンな方法があります。これにより、取得したIActionComponentを興味のあるタイプにキャストすることなく、子クラスで定義されたこれらのプロパティを公開することができます。それとも、これを達成するための唯一の/最良の方法です。ような何か：

public void IWantYouToAttackSuperSlow(IGameObject target) 
{ 
    //Let's get the action component for the gameobject and test if it's an ITimedActionComponent... 
    ITimedActionComponent actionComponent = target.GetComponent<IActionComponent>() as ITimedActionComponent; 

    if (actionComponent != null) //We're okay! 
    { 
     actionComponent.IdleTime = int.MaxValue; //Mwhaha 
    } 
} 

は、今私はそれが唯一の方法だ考えていますが、私は無知だ木工品の中に隠れたパターンがあります場合、私は見たい、または誰でも提案することができた場合に考え出しこれを始めるにはもっと良い方法です。

ありがとうございます！

Answer 1

サブタイトルにキャストすることで特定の実装をテストするのは良い考えではありません。あなたが提供している例から、何かアクションを実行したいと思っているIGameObjectがあるようです。 IActionComponentには、IGameObjectを引数とするメソッドが公開されているのではなく、別の方法で並べ替えることができます。すなわち、IGameObjectを実行するか、実行可能なアクションを好むことができます。あなたがIActionComponentの実装方法に応じて

public void IWantYouToAttackSuperSlow(IGameObject target) 
{ 
    target.ActionComponent.ExecuteAction(); 
} 

を行う

interface IActionComponent 
{ 
    ExecuteAction(); 
} 

interface IGameObject 
{ 
    IActionComponent ActionComponent { get; } 
} 

は、実行中のメソッドでは、多型は、右のコードが実行されていることを確認します。

Command Patternというパターンがあり、必要に応じているようです。

Answer 2

あなたはそれがキャストダウンIActionComponentをITimedActionComponentに示したコードは右、あなたがそれらを使用するためにIdleTimeの特性を認識する必要がありunavoidable-（私の知る限り）ですか？

あなたのIActionComponentを使用するクラスではそれを処理する必要がないところで、あまり見栄えのないコードを隠すことがここのトリックだと思います。これを行う方法の
私が最初に考えたのは使用することですFactory：

public void IWantYouToAttackSuperSlow(IGameObject target) 
{ 
    //Let's get the action component for the gameobject 
    IActionComponent actionComponent = ActionComponentsFactory.GetTimedActionComponentIfAvailable(int.MaxValue); 
} 

とあなたの工場の方法：

public IActionComponent GetTimedActionComponentIfAvailable(IGameObject target, int idleTime) 
{ 
var actionComponent = target.GetComponent<IActionComponent>() as ITimedActionComponent; 

    if (actionComponent != null) //We're okay! 
    { 
     actionComponent.IdleTime = int.MaxValue; //Mwhaha 
    } 
return (actionComponent != null)? actionComponent : target.GetComponent<IActionComponent>(); 
}