Предотвращение instanceof в Java

Как подчеркивается в комментариях, шаблон посетителя был бы хорошим выбором. Но без прямого контроля над целью/акцептором/посетителем вы не сможете реализовать этот паттерн. Вот один из способов использования паттерна посетителя здесь, даже если у вас нет прямого контроля над подклассами, с помощью оберток (на примере Integer):

public class IntegerWrapper {
    private Integer integer;
    public IntegerWrapper(Integer anInteger){
        integer = anInteger;
    }
    //Access the integer directly such as
    public Integer getInteger() { return integer; }
    //or method passthrough...
    public int intValue() { return integer.intValue(); }
    //then implement your visitor:
    public void accept(NumericVisitor visitor) {
        visitor.visit(this);
    }
}

Конечно, обертка конечного класса может считаться запахом сама по себе, но, возможно, она хорошо сочетается с вашими подклассами. Лично я не думаю, что instanceof здесь так уж плохо пахнет, особенно если он ограничен одним методом, и я бы с удовольствием использовал его (возможно, вместо моего собственного предложения выше). Как вы говорите, он вполне читабелен, безопасен для типов и удобен в обслуживании. Как всегда, сохраняйте простоту.

Вместо огромного if, вы можете поместить обрабатываемые экземпляры в карту (ключ: класс, значение: обработчик).

Если поиск по ключу возвращает null, вызовите специальный метод обработчика, который пытается найти подходящий обработчик (например, вызывая isInstance() для каждого ключа в карте).

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

Это делает общий случай быстрым и простым и позволяет обрабатывать наследование.

Вы можете рассмотреть шаблон «Цепочка ответственности» . Для вашего первого примера что-то вроде:

public abstract class StuffHandler {
   private StuffHandler next;

   public final boolean handle(Object o) {
      boolean handled = doHandle(o);
      if (handled) { return true; }
      else if (next == null) { return false; }
      else { return next.handle(o); }
   }

   public void setNext(StuffHandler next) { this.next = next; }

   protected abstract boolean doHandle(Object o);
}

public class IntegerHandler extends StuffHandler {
   @Override
   protected boolean doHandle(Object o) {
      if (!o instanceof Integer) {
         return false;
      }
      NumberHandler.handle((Integer) o);
      return true;
   }
}

, а затем то же самое для других ваших обработчиков. Затем это случай объединения StuffHandlers в последовательность (от наиболее специфичного до наименее специфичного, с последним «резервным» обработчиком), и ваш код отправителя будет просто firstHandler.handle (o); .

(Вместо использования цепочки можно просто создать List в своем классе диспетчера и прокручивать список до тех пор, пока не будет handle () возвращает истину).