サブクラスの型チェックを避けるには？

Question

私はこのコードを持っている：

public abstract class Character { 
    public abstract void attack(Character victim); 
    public abstract void receiveAttack(Attack attack); 
} 

public class CharacterA extends Character { 
    public void attack(Character victim) { 
     if (victim.getClass().equals(this.getClass()) 
      return; 

     victim.receiveAttack(new Attack(BASE_DMG)); 
    } 
} 

アイデアは文字が他の文字から/への攻撃とreceiveAttacksことができるということですが、それはそれ自身のクラスの性格から来て攻撃を受けることができない（CharacterAは別のものを攻撃することはできませんがキャラクターBを攻撃したり、キャラクターBからの攻撃を受けることができます。

私は、オブジェクトのタイプのチェックが悪い匂いであることを知っています。通常、デザインが悪いために発生します。そのため、デザインを変更してオブジェクトタイプを確認する必要はありません？

EDIT

クラスは、実際には名前だけの例を簡素化することで、Character命名されていません。

チームはありません。 CharacterAは、CharacterAではない他のCharacterを攻撃することができます。

FINAL EDIT

みんなありがとう、私はVisitorパターンを使用して、それを解決しました。

Answer 1

非常に単純な解決法/回避策は1つのクラスのみを持つことができ、Characterオブジェクトはすべてという名前のフィールド、つまりcharacterTeamなどを持ちます。 Aチームのキャラクタの場合は、そのフィールドを"A"または"ATeam"に設定します。 Bチームの場合は、そのフィールドを"B"または"BTeam"に設定します。次にattackメソッドのcharacterTeamStringを比較します。

また、私はこのようCharacterという名前のクラスもCharacterという名前charのラッパークラス、と干渉する可能性があるに対して助言します。

Answer 2

これは、特定のサブクラスのインスタンスを取得し、このメソッドをノーオペレーションにするか、警告を出力すると宣言されている各サブクラスにオーバーロードされた攻撃メソッドを追加することで実現できます。

CharacterAの署名は、あなたのサブクラスのインスタンスが特定のタイプではなく、基本文字型で宣言されている場合ただし、これはのみ動作します

public void attack(CharacterA victim) 

だろう。

class Character 
{ 
    public void attack(Character victim) 
    { 
     System.out.println(getClass() + " attacking " + victim.getClass()); 
     victim.receiveAttack(new Attack()); 
    } 

    public void receiveAttack(Attack attack) {} 
} 

class CharacterA extends Character 
{ 
    public void attack(CharacterA victim) 
    { 
     System.out.println(getClass() + " cannot attack " + victim.getClass()); 
    } 
} 

class CharacterB extends Character 
{ 
    public void attack(CharacterB victim) 
    { 
     System.out.println(getClass() + " cannot attack " + victim.getClass()); 
    } 
} 

class Attack 
{ 
} 

public class Test 
{ 
    public static void main(String[] args) 
    { 
     CharacterA a = new CharacterA(); 
     CharacterB b = new CharacterB(); 

     a.attack(a); 
     a.attack(b); 
     b.attack(a); 
     b.attack(b); 

     // redeclare a & b to have the base type 
     Character c = a; 
     Character d = b; 

     c.attack(c); 
     c.attack(d); 
     d.attack(c); 
     d.attack(d); 
    } 
} 

出力：

class CharacterA cannot attack class CharacterA 
    class CharacterA attacking class CharacterB 
    class CharacterB attacking class CharacterA 
    class CharacterB cannot attack class CharacterB 
    class CharacterA attacking class CharacterA 
    class CharacterA attacking class CharacterB 
    class CharacterB attacking class CharacterA 
    class CharacterB attacking class CharacterB