Стоит ли использовать instanceof, если в JTree несколько разных объектов?

Question

[Rodgenk]

В моем JTree дереве существует несколько разных видов объектов (6 штук).
Для работы разных объектов нужны совершенно разные методы.
У каждого класса около десятка своих полей.

<code lang="java">
DefaultMutableTreeNode node = (DefaultMutableTreeNode) tree.getModel().getChild(currentNode, i);
    Object ob = node.getUserObject();
    if(ob instanceof DescriptionTemplate) {
        // todo...
    }
</code>

Насколько оправдано использование использование оператора instanceof в этом случае?
При использовании instanceof образуется большое количество ветвлений.

Можно ли решить эту проблему другим способом?

Answer 1

[Losted]

Как насчет общего интерфейса с методом getType()?

Comments

[Rodgenk]

Здравствуйте. Да, это избавит от instanceof, об этом не подумал, спасибо.
Подскажите, есть ли польза от данного решения, кроме как избавления от оператора?

[Андрей Вершинин]

@Rodgenk С точки зрения архитектуры это более красиво.

Answer 2

[paralainer]

Не стоит забывать про возможность использования стратегий.

Очень приблизительный код, но суть ясна:

private static final Map<Class, Action> strategies = new HashMap<>();
    
    private static interface Action {
         void doAction(Params params);
    }
    
    static {
        strategies.put(ANodeType.class, new Action() {
            public void doAction(Params params){
                 ...
            } 
        });

        strategies.put(BNodeType.class, new Action() {
            public void doAction(Params params){
                 ...
            } 
        });
    }
    
    ...
    
    public void handleTreeNode(Object node, Params params){
       strategies.get(node.getClass()).doAction(params);
    }

А вообще есть вариант реализовать всеми типами нод интерфейс Action (переименовав его, например, в JTreeNode) и выглядеть это будет примерно так:

public void handleTreeNode(JTreeNode node, Params params){
       node.doAction(params);
    }

Но тут нужно смотреть возможно ли изменить код классов, которые хранятся в дереве и насколько это будет архитектурно правильно. (Специальные ли это объекты для дерева или общие объекты системы)

Comments

[Rodgenk]

Спасибо за ваш ответ и потраченное время.
Главная проблема на самом деле в том, что объекты просто хранят разные значения из БД и проверяю я их на объекты только, чтобы понять какие данные мне нужно получить, например, такой кусок:

DefaultMutableTreeNode root = (DefaultMutableTreeNode) tree.getModel().getRoot();
        for (int i = 0; i < root.getChildCount(); i++) {
            DefaultMutableTreeNode event = (DefaultMutableTreeNode) root.getChildAt(i);
            for (int j = 0; j < event.getChildCount(); j++) {
                DefaultMutableTreeNode subEvent = (DefaultMutableTreeNode) event.getChildAt(j);
                for (int k = 0; k < subEvent.getChildCount(); k++) {
                    DefaultMutableTreeNode rec = (DefaultMutableTreeNode) subEvent.getChildAt(k);
                    Object ob = rec.getUserObject();

                    if (ob instanceof DescriptionResult) {
                        DescriptionResult temp = (DescriptionResult) ob;
                        allActiveTemplateRecords.add(temp);
                        DESCRIPTION_RESULT dt = new DESCRIPTION_RESULT(Global.conn, temp.getId(), true);
                        formPanel.addRow(temp.getName(), temp.getId(), dt.getCText());
                    }

                    if (ob instanceof DescriptionTemplate) {
                        DescriptionTemplate temp = (DescriptionTemplate) ob;
                        if(temp.getEditStatus()) {
                            allActiveTemplateRecords.add(temp);
                            DESCRIPTION_TEMPLATE dt = new DESCRIPTION_TEMPLATE(Global.conn, temp.getId(), true);
                            formPanel.addRow(temp.getText(), temp.getId(), dt.getCText());
                        }
                    }

                    if (ob instanceof Protocol) {
                        Protocol temp = (Protocol) ob;
                        formPanel.addProtocolFormRow(temp.getId(), visitId, temp.getFormId(), temp.getText());
                    }
                }
            }
        }

Поэтому выслушав все ответы, я решил, что оставлю как есть. Хотя хочется, конечно, сделать все красиво.