Java - Удаление объекта из массива объектов

Question

Я пишу большую программу, которая включает в себя объект под названием Player. Определение игрока заключается в следующем:

public class Player 
{ 
    public static String name; 
    public static Item inventory[] = new Item[10]; 
    public static int items; 

    /* ... */ 

    public void addItem(String itemName, int itemType) 
    { 
      if ((items + 1) <= 10) { 
        inventory[items] = new Item(itemName, itemType); 
        items++; 
      } 
    } 

    public void removeItem(int x) 
    { 
      for (int i = x; i < items; i++) 
        inventory[i] = inventory[i+1]; 
    } 
} 

Я добавляю обработку инвентаря теперь, потому что это гораздо проще, чем добавить его позже, но inventory не будет не будет использоваться гораздо позже в процессе развития. У меня нет способа узнать, работает ли removeItem. Я изменил функцию, которую я написал, назвал strstrip, чтобы получить это ... Будет ли removeItem работать? Если нет, то почему?

Answer 1

Создайте модульные тесты для своих классов, особенно если вы собираетесь построить «большую и сложную программу». Это гарантирует вам, что написанный код будет работать позже, и если вы измените код, сбой модульных тестов должен указывать на проблему. Единичный тест также дает вам возможность проверить, работает ли ваш метод, как ожидалось.

Как и в других комментариях, рассмотрите возможность использования интерфейса List вместо массива, если у вас нет определенного требования (я не могу себе представить). И определенно, если поля public static в вашем классе выглядят подозрительными.

EDIT

Просто, чтобы показать, как код может выглядеть и как вызывать методы основного метода.

public class Player { 

    private String name; 
    private List<Item> inventory; 
    private int items; 

    public Player() { 
     this.inventory = new ArrayList(); 
    } 

    public void addItem(String itemName, int itemType) { 
     this.inventory.add(new Item(itemName, itemType)); 
    } 

    public void removeItem(int x) { 
     Item itemToRemove = this.inventory.get(x); 
     if (itemToRemove != null) { 
      this.inventory.remove(itemToRemove); 
     } 
    } 

    public static void main(String[] args) { 
     // create a new instance 
     Player player = new Player(); 
     // call a method on the instance 
     player.addItem("bla", 0); 
    } 
}