Избегайте использования примера:

Question

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

Моя классная структура выглядит следующим образом.

Game Controller - Ответственный за нерестовые монеты и отслеживание состояния игры

Сетка - класс, который хранит объекты, которые в настоящее время находятся на сетке, и связанные с ними методы несколько сетки.

GridObject - абстрактный класс, представляющий объект на сетке. например игрок, монстр или монета.

Игрок, Монстр, Монета - Все виды GridObject.

Каков самый способ ООП для обработки столкновений? Желаемый результат: Игрок ударяет по монстру - конец игры, Игрок ударяет монеты - увеличивает счет, Монстр хиты Монета - ничего.

В настоящее время, когда объект GridObject перемещается, он уведомляет объект платы, который он хочет переместить; если это вызовет столкновение, я вызываю обработчик на GameController (через шаблон прослушивателя) для обработки столкновения. В этом обработчике, который принимает два GridObjects в качестве параметров, я использую множество команд «instanceof», чтобы отличать вышеуказанные случаи.

Каков наилучший способ избежать использования всего этого «экземпляра», который я прочитал, как правило, означает плохой дизайн?

Answer 1

насчет введения некоторых интерфейсов:

interface Interactable { 

    InteractionResult interact(WorldEntity entity); 

} 

interface WorldEntity { 

    boolean isMonster(); 

    boolean isPlayer(); 

    boolean isPowerUp(); 

} 

Теперь мы можем сказать, что, когда игрок взаимодействует с другими вещами, мы можем сделать:

public InteractionResult interact(WorldEntity entity) { 
     if(entity.isMonster()) { 
      return new DeathInteraction(this); 
     } 
     ... 
    } 

Вы не устранить все условные, но, по крайней мере, вы начинаете организовывать абстракции и отвязать ваш классы.

Answer 2

Вместо использования instanceof вы можете использовать перечисление в GridObject для определения типа. Таким образом, вы просто сравниваете перечисление, чтобы проверить, какие объекты сталкиваются. Например, (предупреждение, синтаксис может быть не совсем правильно)

public enum ObjectType { 
    Player, Monster, Coin 
} 

private final ObjectType type; 

public GridObject(ObjectType type) { 
    this.type = type; 
} 

public ObjectType getType() {return type;} 

, а затем в ваших подклассов, вы должны сделать следующее:

public Player(your parameters) { 
    super(ObjectType.Player); 
} 

А затем для проверки типов , вы можете просто сделать:

switch(gridObject.getType()) { 
    case ObjectType.Player: 
     // do something 
     break; 
    case ObjectType.Monster: 
     // do something else 
     break; 
    case ObjectType.Coin: 
     // do something else 
     break; 
}