Java "instanceof", чтобы добавить несколько списков

Question

В настоящее время я пытаюсь реорганизовать свой код после того, как прочитал, что реализация предпочтительнее расширений. В настоящее время я пытаюсь создать функцию, которая добавляет объект в сцену. Для того, чтобы лучше определить, что каждый объект, есть несколько списков, таких как список для обновления, визуализации и т.д.

private List<Updatable> updatables; 
private List<Removable> removables; 
private List<Renderable> renderables; 
private List<Collidable> collidables; 

Я хочу сделать функцию в моем классе сцены, как так:

public void add(Object o) { 
    if(o instanceof Updatable) 
     updatables.add((Updatable) o); 
    if(o instanceof Removable) 
     removables.add((Removable) o); 
    if(o instanceof Renderable) 
     renderables.add((Renderable) o); 
    if(o instanceof Collidable) 
     collidables.add((Collidable) o); 
} 

Аналогичным образом я бы создал аналогичную функцию удаления. Мой вопрос в том, что это плохая практика кодирования, поскольку я слышал о подводных камнях instanceof, а во-вторых, если это так, есть ли способ об этом/реструктурировать мой код, чтобы упростить добавление экземпляров? Я предпочитаю не добавлять экземпляр в каждый список и определять, к какому из них он относится каждый раз.

Answer 1

Я хотел бы сделать подобный с фабричной моделью с использованием Map

static Map<String, List> maps = new HashMap<>(); 
static { 
    maps.put(Updatable.class.getName(), new ArrayList<Updatable>()); 
    maps.put(Removable.class.getName(), new ArrayList<Removable>()); 
    maps.put(Renderable.class.getName(), new ArrayList<Renderable>()); 
    maps.put(Collidable.class.getName(), new ArrayList<Collidable>()); 
} 
public static void add(Object obj){ 
    maps.get(obj.getClass().getName()).add(obj); 
} 

Answer 2

Просто используйте перегруженный: add(Updateable u), add(Removable r), и т.д.

Answer 3

Как упоминался в комментариях, ваш случай использования несколько особенный. Updatable, Removable, Renderable и Collidable все интерфейсы. У вас есть классы, которые реализуют один или несколько из этих интерфейсов. В конечном счете, вы хотите, чтобы ваш метод add добавлял объект в каждый список интерфейса, который он реализует. (Поправьте меня, если это не так.)

Если перегрузка будет слишком многословным, вы можете сделать это с отражением, например, так:

private List<Updatable> updatables; 
private List<Removable> removables; 
private List<Renderable> renderables; 
private List<Collidable> collidables; 

@SuppressWarnings("unchecked") 
public void add(Object o) { 
    try { 
     for (Class c : o.getClass().getInterfaces()) { 
      // Changes "Updatable" to "updatables", "Removable" to "removables", etc. 
      String listName = c.getSimpleName().toLowerCase() + "s"; 

      // Adds o to the list named by listName. 
      ((List) getClass().getDeclaredField(listName).get(this)).add(o); 
     } 
    } catch (IllegalAccessException e) { 
     // TODO Handle it 
    } catch (NoSuchFieldException e) { 
     // TODO Handle it 
    } 
} 

Но если вы собираетесь использовать этот подход, обратите внимание на эти оговорки:

1. Отражение, по своей природе, очень подвержено ошибкам во время выполнения.
2. Вы можете столкнуться с проблемами, если ваша иерархия сложнее. Например, если у вас есть Foo implements Updatable, Removable, а также Bar extends Foo, то вызов getClass().getInterfaces() на Bar будет возвращать пустой массив. Если это похоже на ваш прецедент, вы можете вместо этого использовать ClassUtils.getAllInterfaces от Apache Commons Lang.

Answer 4

То, что я бы ожидать, что у вас есть отдельный add метод для каждого из этих списков, например, так:

public void addUpdatable(Updatable updatable) { 
    updatables.add(updatable); 
} 

public void addRemovable(Removable removable) { 
    removables.add(removable); 
} 

// and so on 

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

Например, недоразумение (в комментариях к комментариям EJP) о том, что происходит, когда добавлен Updatable, который также добавлен Removable, показывает это отсутствие ясности. Хотя вы используете аналогичные термины как для ваших типов, так и для своего состояния, что делает его излишним, вы все равно должны указать, к какому списку (или части состояния) добавлен элемент.

В любом случае, похоже, что вы боретесь с проблемой в ее больших масштабах.Я думаю, что было бы неплохо потратить больше времени на понимание более серьезной проблемы.

Answer 5

От @ Jerry06, лучший способ добавить экземпляр в несколько списков разных типов для меня - использовать магический приемник.

private Map<Class, List> maps = new HashMap<>(); 

«maps» используется, чтобы найти список, который вы ищите.

private <T> List<T> get(Class<T> c) { 
    return maps.get(c); 
} 

вместо использования maps.get, я просто использую get (SomeClass.class). Функция выше автоматически преобразует ее в список с правильным типом элемента. Я тогда был в состоянии сделать:

get(Updatable.class).stream().filter(Updatable::isAwake).forEach(Updatable::update); 

вместо:

updatables.stream().filter(Updatable::isAwake).forEach(Updatable::update); 

Я в основном хотел структуру, как это так, все списки могут быть обернуты красиво, а во-вторых, чтобы я мог просто сказать карты. добавьте (SomeNewClass.class, новый ArrayList()), а затем вызовите его, когда захотите.