java - 在无法实现多态性时使用 instaceof

Question

编辑:这不是重复的，因为我正在解决无法重载类以实现多态性的问题。我不是要比较多态性和 instanceof。我在问一个非常具体的基于场景的问题。

我已经在网上阅读了 n 次，应该始终尽量避免使用 instanceof 运算符，除非我们使用它来实现 .equals () 一个类的方法。

有人说，只要有可能，我们就应该尝试实现多态性，而不是使用instanceof 来确定对象的类型来决定接下来的 Action 。

例如:http://www.javapractices.com/topic/TopicAction.do?Id=31

但是，在这个特定场景中，我在父类(super class)和子类中没有任何可用的方法(toString、equals、accessor 和 mutator 方法除外)。使用 instanceof 是否有效？

class Warrior
{

    int damage;
    int defense;
    Item handSlot;

    public Warrior(){
        damage = 0;
        defense = 0;
        handSlot = null;
    }

    private void equipHand(Item item)
    {
        //Determine class of object to decide further actions
        if (item instanceof Weapon){
            this.handSlot= item;
            this.damage += ((Weapon)item).getDamage();  
        }
        else if (item instanceof Shield){
            this.handSlot = item;
            this.defense += ((Shield)item).getProtection();
        }
    }
}


class Item
{
    private String name;
    public Item(String name)
    {
        this.name = name;
    }       
}

class Shield extends Item
{
    private int protection;

    public Shield(String name, int protection){
        super(name);
        this.protection = protection;
    }

    public int getProtection(){
        return protection;
    }
}

class Weapon extends Item
{
    private int damage;

    public Weapon(String name, int damage){
        super(name);
        this.damage = damage;   
    }

    public int getDamage(){
        return damage;
    }   
}

请注意，如果Weapon, Item & Shield class 有重载方法(例如:equip())，我可以简单地使用多态性。但是在这种情况下，equip 方法在另一个类中。 在这种特定情况下，是否可以使用 instanceof operator？或者它仍然是一个糟糕的设计？

Answer 1

您的设计的替代方案是将 equipHand 方法从 Warrior 移动到 Item(或许也可以根据此重命名它):

/* in Item */
public abstract void giveToWarrior(Warrior w);

/* in Weapon */
@Override
public void giveToWarrior(Warrior w) {
    w.setHandSlot(this);
    w.increaseDamageBy(getDamage());
}

/* in Shield */
@Override
public void giveToWarrior(Warrior w) {
    w.setHandSlot(this);
    w.increaseProtectionBy(getProtection());
}

(我使用了一些 Warrior 中不存在的方法，但它们的功能应该是不言自明的。进行指定的更改需要实现这些方法。)

此设计解决了您当前设计的主要问题，即添加新的 Item 子类会变得很麻烦，因为这样做还需要更新 equipHand。