Проектирование системы инвентаризации предметов для игры и с проблемами

Question

Я работал над небольшой системой инвентаризации, чтобы практиковать мое программирование OO. Я использую Java 7 в Netbeans.

У игрока будет инвентарь (ArrayList базового класса предметов), все созданные объекты будут дочерними элементами этого базового класса предметов.

Нет элементов, которые должны быть типа BaseItem, все они будут простираться от этого, так что, насколько мне известно, класс BaseItem должен быть абстрактным?

public abstract class BaseItem { 
    protected GoldValue itemValue; 
    protected String itemName; 

    //accessors and mutators down here 
} 

типов элементов для детей также будут иметь разные свойства, которые я собирался реализовать, как интерфейсы, такие как стекируемые, расходная и т.д.

Интерфейс выглядит что-то вроде этого

public interface Stackable { 

    public void stack (StackableItem bi1); 

} 

Как вы можете видеть, я ссылаюсь на StackableItem здесь, это один из дочерних элементов BaseItem, который также является абстрактным, а конкретные элементы будут построены.

public abstract class StackableItem extends BaseItem implements Stackable{ 

    protected int quantity; 
    protected int maxStacks; 

    @Override 
    public void stack (StackableItem si) 
    { 
     if(this.getQuantity() + si.getQuantity() < maxStacks){ 
      this.setQuantity(this.getQuantity()+si.getQuantity()); 
      si.setQuantity(0); 
     } 
     else if(this.getQuantity() + si.getQuantity() > maxStacks){ 
      int diff = maxStacks - this.getQuantity(); 
      this.setQuantity(this.getQuantity() + diff); 
      si.setQuantity(si.getQuantity() - diff); 
     } 
    } 
} 

и вот пример конкретного пункта:

public class StackableItemExample extends StackableItem{ 

    public StackableItemExample()throws GoldException 
    { 
     this(new GoldValue(0,0,0), "unnamed", 1); 
    } 
    public StackableItemExample(GoldValue gv, String name, int quan) throws GoldException 
    { 
     this.itemValue = gv; 
     this.itemName = name; 
     this.quantity = quan; 
     this.maxStacks = 10; 
     this.itemValue.setGoldValue(gv.getGold()*quan, gv.getSilver()*quan, gv.getCopper()*quan); 
    } 


} 

Мои Inventory, будучи ArrayList из BaseItem объектов, разрешено иметь StackableItemExample объект не вставлен в него без проблем. Проблема в том, что моя система инвентаризации не может сказать BaseItem.stack(), поскольку родитель не знает об этом методе.

Я хочу сделать очень легко расширяемую систему элементов, что новый элемент можно создать с предельной простотой, внедряя и расширяя предварительно созданные классы и интерфейсы, но это мешает мне.

Answer 1

Ни один элемент не должен быть типа BaseItem, все они будут простираться от этого, так что, насколько мне известно, класс BaseItem должен быть абстрактным?

Звучит хорошо для меня

---

Если вы хотите BaseItem.stack() работать, вы могли бы сделать, по крайней мере, две вещи:

• Move stack() в свой базовый класс, и дать ему реализацию по умолчанию ничего не делать. Затем переопределите это в своем классе StackableItem, или
• Используйте instanceof, чтобы узнать, является ли BaseItem StackableItem, затем отбросит его на StackableItem и оттуда.

Второй подход будет выглядеть так:

BaseItem item = YourList.get(i); 
if(item instanceof StackableItem){ 
    StackableItem stackable = (StackableItem)item; 
    stackable.stack() /* Now works without issue */ 
} 

Answer 2

Сделать каждый элемент Stackable - в настоящее время неустойчивые элементы должны иметь размер стека одного. Вы могли бы избавиться от стекируемого абстрактного класса и интерфейса, поместив эту функциональность в базовый класс.