2

Есть несколько классов, далее идет проверка классов через instanceof, если передали такой класс тогда выполнить такие то действия. Какой паттерн можно использовать.

getParams(SomeClass some) {

if (some instanceof SomeNoth) {
... 
... 
... 
... }
if (some instanceof SomeNoth2) {
... 
... 
... 
... }
if (some instanceof SomeNoth3) {
... 
... 
... 
... }
if (some instanceof SomeNoth4) {
... 
... 
... 
... }
if (some instanceof SomeNoth5) {
... 
... 
... 
... }
6
  • странное решение... но, если вам оно действительно надо... - чтобы не писать вот такое вот полотно, я бы посоветовал завести интерфейс, который хранил бы два метода void run(Object obj) и boolean isTypeof(Object obj). Далее, простое решение - делайте кучу классов-хандлеров, наследуете каждым классом этот интерфейс, в каждом методе isTypeof(Object obj) возвращаете ваш obj instanceof ..., в методе run - делаете что надо. Создать инициализацию коллекции экземпляров таких классов, например, singleton-ом, вы и сами сможете) В цикле останется перебрать, и проверить, что надо запускать, а что не надо
    – test123
    26 окт 2017 в 6:34
  • Если других, более интересных, вариантов никто не предложит, покажу пример
    – test123
    26 окт 2017 в 6:36
  • Действия какого плана? Если вызов каких-то методов у some, то тогда стоит создать интерфейс SomeInterface, сделать чтобы все нужные классы его реализовывали, в метод передавать SomeInterface some, после чего вызывать нужный метод (или методы) у some без всяких if-ов.
    – Regent
    26 окт 2017 в 6:41
  • switch(some.getClass().getSimpleName()) в виде костыля.
    – post_zeew
    26 окт 2017 в 6:44
  • @test123 покажите пожалуйста пример
    – J Mas
    26 окт 2017 в 6:45

5 ответов 5

6

Можно сделать изящно и красиво - использовать фабрику. А именно:

Map<Class, Runnable> actions = new HashMap<>();  
....  
void registrationAction(Class<?> cl, Runnable action){
   actions.put(cl, action);
}

void actionOn(SomeClass cl){
   actions.get(cl.getClass()).run();
}
5
  • Не является ли это костылем?
    – J Mas
    26 окт 2017 в 7:22
  • 1
    @JNat, нет, не является. Хм, причём, довольно простое решение))
    – test123
    26 окт 2017 в 7:38
  • Вообще это общепринято. Если у вас вдруг появится новый класс, вам не надо будет дописывать свой код. Вам просто надо будет зарегистрировать новый обработчик (особенно это удобно если у вас будут плагины). Более того - мы у себя используем автоматическую регистрацию обработчиков - просто аннотируем класс, а сканер его укладывает в подобную в ответе структуру. 26 окт 2017 в 7:39
  • 1
    @JNat, С огромной долей вероятности, карта будет быстрее. + Её скрость можно менять реализацией, а instanceof - изменять не получится... Рефлексия, как ни крути, медленная штука.
    – test123
    26 окт 2017 в 8:52
  • HasMap будет быстрее. Он просто возьмет имя класса (строка) и за константную сложность О(1) выполнит любой запрос. 26 окт 2017 в 9:01
5

Начиная с Java 7 можно использовать String в switch, следовательно можно хакнуть так:

getParams(SomeClass some) {
   String clzName=some.getClass().getSimpleName();
   switch(clzName) {
      case "SomeNoth":
        break;
      case "SomeNoth2":
        break;
      //blah-blah
   }
}

Update

Для поборников чистоты можно сделать так:

interface Doable {
    public void doSome();
}

class ExSomeClass extends SomeClass implements Doable {
    public void doSome() {
        //blah-blah
    }
}

class ExSomeNoth extends SomeNoth implements Doable {
    public void doSome() {
        //blah-blah
    }
}

class ExSomeNoth2 extends SomeNoth2 implements Doable {
    public void doSome() {
        //blah-blah
    }
}

В итоге ваш метод будет работать так:

getParams(Doable some) {
   some.doSome();
}

Никаких кастов, instanceof и упаси боже модификации исходных классов.

5
  • А есть ли в этом смысл? От полотна кода вы не избавитесь.
    – test123
    26 окт 2017 в 7:01
  • Поскольку классы библиотечные, то внедрение интерфейса не сработает, ну разве что творить обертки над ними - но тогда кода будет еще больше. switch всяко разно лучше лесенки if/else if
    – Barmaley
    26 окт 2017 в 7:04
  • Так лучше) Но, опять же, автор пояснил, что он не сможет отнаследовать ваш интерфейс в нужных ему классах, потому что они - части библиотеки, которая, как я понимаю, для него - чёрный ящик, ну, или, просто недоступна для правки.
    – test123
    26 окт 2017 в 11:17
  • Причем тут это? Классы он создает сам, так что может как угодно отнаследовать
    – Barmaley
    26 окт 2017 в 11:23
  • если всё как вы говорите, то у ТС неверно поставлен вопрос. Это в корне всё меняет.
    – test123
    26 окт 2017 в 11:50
4

Так как вы настаиваете на примере, приведу простой для понимания. Наши тестовые классы:

public class Test1 {
    public void method1() { System.out.print("Test1.method1"); }
}

public class Test2 {
    public void method2() { System.out.print("Test2.method2"); }
}

Далее, небольшая архитектура:

public interface IHandler {

    void run(Object obj);

    boolean isTypeof(Object obj);

}

Под неё делаем две реализации хандлеров:

public class Test1Handler implements IHandler {

    @Override
    public void run(Object obj){
        Test1 test = (Test1)obj;
        // Здесь любой код...
        test.method1(); // для примера
    }

    @Override
    public boolean isTypeof(Object obj){
        return obj instanceof Test1;
    }

}

public class Test2Handler implements IHandler {

    @Override
    public void run(Object obj){
        Test2 test = (Test2)obj;
        // Здесь любой код...
        test.method2(); // для примера
    }

    @Override
    public boolean isTypeof(Object obj){
        return obj instanceof Test2;
    }

}

Всё, архитектура готова, реализация есть, теперь можно всё свернуть в один универсальный метод:

public class ServiceChecker {

    private static ServiceChecker instance;

    private List<IHandler> handlers = new ArrayList<IHandler>();

    public ServiceChecker(){
        handlers.add(new Test1Handler());
        handlers.add(new Test2Handler());
    }

    public static ServiceChecker getInstance(){
        if(instance==null){
            instance = new ServiceChecker();
        }
        return instance;
    }

    public void check(Object some){
        for(IHandler handler : handlers){
            if(handler.isTypeof(some)){
                handler.run(some);
                break;
            }
        }
    }

}

Теперь можно смело использовать:

ServiceChecker.getInstance().check(new Test2());
ServiceChecker.getInstance().check(new Test1());

Что вообще этот подход вам даст:

  1. Вы разделите ваше полотно на кучу классов, что в разы удобнее содержать и дополнять.
  2. Вы сможете составить огромную "библиотеку" хандлеров. Лично я такое использовал для преобразования типов в Unity3d, своего рода универсальный парсер. Универсальный он потому, что добавлять и удалять хандлеры не составляет труда, как вы заметили.

Является ли это паттерном? Честно скажу - я не знаю. Скорее всего, нечто такое уже придумали.

3
  • При поддержке будет достаточно сложно понять, что речь шла о getParams
    – hardsky
    26 окт 2017 в 7:41
  • @hardsky, да и в целом, решение довольно громоздкое. ИМХО, самое отличное решение показал Александр Мартынцев, надо будет мне самому взять на вооружение :D
    – test123
    26 окт 2017 в 9:03
  • @test123 Да реализовать свой класс для каждого класса лучше, плюс прямо в классе описываю какой instanceof подходит, имхо меньше шансов ошибиться. а HashMap применять если здесь проблема с производительностью
    – J Mas
    27 окт 2017 в 6:05
2

Что бы действовать подобным образом, надо иметь очень веские на то причины. Стандартным решением будет объявить интерфейс с методом getParams и имплиментировать его для всех ваших классов.

public interface SomeNothInterface{
    public Params getParams();
}

public class SomeNoth implements SomeNothInterface{
....
    public Params getParams(){
        //логика
    }

И вызывать так

public Params getParams(SomeNothInterface someClass) {
    return someClass.getParams();
}
8
  • так не получится сделать потому что классы из библиотеки
    – J Mas
    26 окт 2017 в 6:43
  • 2
    @JNat вот было бы здорово, если бы в тексте вопросов не пропускали столь важные моменты...
    – Regent
    26 окт 2017 в 6:44
  • @JNat между ними есть что-то общее или прямо совсем разные классы? Посмотрите, возможно они и так интерфейс один реализуют
    – rjhdby
    26 окт 2017 в 6:44
  • Либо врапперы над классами с необходимой доп логикой
    – hardsky
    26 окт 2017 в 6:45
  • @JNat я вам рекомендую задать новый вопрос с описанием глобальной задачи. Велика вероятность, что вы пытаетесь закручивать гвозди отверткой
    – rjhdby
    26 окт 2017 в 6:46
2

У вас тут самый настоящий антипаттерн. Правильный объектно-ориентированный код работает с максимально узкой абстракцией и ему не требуется проверять конкретный тип. Если такая необходимость вдруг появилась, значит вы нарушаете принципы SOLID. К тому же, instanceof ещё и медленный оператор.

5
  • Что досталось, это либа
    – J Mas
    26 окт 2017 в 7:23
  • 2
    Обожаю такие ответы :) - выглядит как возмущение автомобилиста - что на дорогах не должно быть ям - но они то ведь есть и будут - мир то ведь неидеален...
    – Barmaley
    26 окт 2017 в 9:45
  • @Barmaley не отрицаю, что иногда без проверки типа не обойтись, но надо стремиться избегать таких ситуаций. И воспитывать это стремление в начинающих. 26 окт 2017 в 10:53
  • Посмотрите на апдейт моего ответа - без антипаттерна :)
    – Barmaley
    26 окт 2017 в 12:04
  • @Barmaley я его даже плюсанул. Как и многие другие ваши ответы :) 26 окт 2017 в 12:08

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.