Избегайте instanceof при проверке типа сообщения

Простой способ избежать тестирования instanceof - это полиморфная диспетчеризация; например

public class Client {
    void messageReceived(IMessage message) {
        message.doOperations(this);
    }
}

где каждый класс сообщений определяет соответствующий doOperations(Client client) метод.

РЕДАКТИРОВАТЬ: второе решение, которое лучше соответствует требованиям.

Альтернативой, заменяющей последовательность тестов instanceof оператором switch, является:

public class Client {
    void messageReceived(IMessage message) {
        switch (message.getMessageType()) {
        case TYPE_A:
           // process type A
           break;
        case TYPE_B:
           ...
        }
    }
}

Каждый класс IMessage должен определять int getMessageType() метод для возврата соответствующего кода. Перечисления работают так же хорошо, как и int, и более элегантны, IMO.

Один из вариантов - это цепочка обработчиков. У вас есть цепочка обработчиков, каждый из которых может обрабатывать сообщение (если применимо), а затем потреблять его, что означает, что оно не будет передано дальше по цепочке. Сначала вы определяете интерфейс Handler:

public interface Handler {
    void handle(IMessage msg);
}

И тогда логика цепочки обработчиков выглядит так:

List<Handler> handlers = //...
for (Handler h : handlers) {
    if (!e.isConsumed()) h.handle(e);
}

Затем каждый обработчик может решить обработать / потребить событие:

public class MessageAHandler implements Handler {
    public void handle(IMessage msg) {
        if (msg instanceof MessageA) {
            //process message
            //consume event 
            msg.consume();
        }
    }
}

Конечно, это не избавляет от instanceofs - но это означает, что у вас нет огромного if-elseif-else-if-instanceof блока, который может быть нечитаемым.

Какой тип системы сообщений вы используете?

У многих есть варианты добавления фильтра к обработчикам на основе заголовка или содержимого сообщения. Если это поддерживается, вы просто создаете обработчик с фильтром на основе типа сообщения, тогда ваш код будет красивым и чистым без необходимости instanceof или проверки типа (поскольку система обмена сообщениями уже проверила его для вас).

Я знаю, что вы можете сделать это в JMS или в службе событий OSGi.

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

  String filterMsg1 = "JMSType='messageType1'";
  String filterMsg2 = "JMSType='messageType2'";

  // Create a receiver using this filter
  Receiver receiverType1 = session.createReceiver(queue, filterMsg1);
  Receiver receiverType2 = session.createReceiver(queue, filterMsg2);

  receiverType1.setMessageHandler(messageType1Handler);
  receiverType2.setMessageHandler(messageType2Handler);

Теперь каждый обработчик будет получать только определенный тип сообщения (без instanceof или if-then), конечно, при условии, что отправитель устанавливает тип с помощью вызовов setJMSType () в исходящем сообщении.

Этот метод встроен в сообщение, но вы, конечно, можете создать собственное свойство заголовка и отфильтровать его.

Решение Java 8, использующее двойную отправку. Не избавляется от instanceof полностью, но требует только одной проверки для каждого сообщения вместо цепочки if-elseif.

public interface Message extends Consumer<Consumer<Message>> {};

public interface MessageA extends Message {

   @Override
   default void accept(Consumer<Message> consumer) {
      if(consumer instanceof MessageAReceiver){
        ((MessageAReceiver)consumer).accept(this);
      } else {
        Message.super.accept(this);
      }
   }
}

public interface MessageAReceiver extends Consumer<Message>{
   void accept(MessageA message);
}

С JMS 2.0 вы можете использовать:

consumer.receiveBody(String.class)

Для получения дополнительной информации см. здесь: