ReSQL.ru
     
powered by simpleCommunicator - 2.0.61     © 2026 Programmizd 02
Мобильная версия    Контакт    Правила    FAQ    Помощь
Целевая тема:
Создать новую тему:
Автор:
Закрыть
Цитировать
Доб. в избранное | Игнор. тему | Прикреп. тему | Пометить прочит. / непрочит. | Фильтр
Форумы / Java [игнор отключен] [закрыт для гостей] / Какие альтернативы есть у instanceOf
25 сообщений из 60, страница 2 из 3
  2  все
Какие альтернативы есть у instanceOf
    #38050251
Фотография Blazkowicz
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Blazkowicz
Участник
grasoff.netочень нужно, взял блокнот
Негодяй. :)
Конечно иногда мы не может сделать рефакторинг, так как не владеем класами. Например это коллекции или обертки примитивов.
Поэтому далее предполагаем что мы проверяем собственные классы.

В растущем проекте. (А мы всегда предполагаем что проект растет)
Код вида
Код: java
1.
2.
3.
if(o instanceof Clazz){
 ...
}

часто превращается в
Код: java
1.
2.
3.
4.
5.
6.
if(o instanceof Clazz){
 ...
}else if(o instanceof AnotherClazz)
 ...
{
}



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


Второе, это то что рафакторить-двигать методы по иерархи вверх-вниз достаточно просто. А то время как зарефакторить лесенку if...else труднее, особенно если кто-то туда домешал дополнительных условий. Код на полиморфизме проще модифицировать.

Ну, и третье, хотя, возможно, и не последнее это читаемость . С полиморфизмом в конкретном классе у нас есть небольшой конкретный метод. В случае лесенки instanceof приходится охватывать взглядом сразу несколько вариантов и искать нужный.
...
Рейтинг: 0 / 0
23.11.2012, 12:20:23
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050273
0FD
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
0FD
Участник
ivanra0FDРаботать не будет, doSomething(A value) будет всегда вызываться. Вот этого метода doSomething(A value) и не должно быть.

Согласен, написал чересчур коротко и как следствие неточно. Вот рабочий пример
Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
// тут желательно определить интерфейсы, пропущено для краткости
class A {
	void accept(Visitor visitor) {
		visitor.doSomething(this);
	}
}
class Visitor{
	void doSomething(A value) {
		//nothing
	}
	void doSomething(A1 value) {
		System.out.println("A1");
	}
}
class A1 extends A{
	void accept(Visitor visitor) {
		visitor.doSomething(this);
	}
}
class A2 extends A{}
public class TestCase {
	public static void main(String[] args) {
		Visitor visitor = new Visitor();
		A[] values = {
			new A1(),
			new A2()
		};
		for(A value:values) {
			value.accept(visitor);
		}
	}
}


Весь фокус в перегрузке методов в Visitor и переопределении accept в нужных классах

Повторюсь, РАБОТАТЬ НЕ БУДЕТ. ПОТОМУ ЧТО
...
Рейтинг: 0 / 0
23.11.2012, 12:28:11
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050278
0FD
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
0FD
Участник
0FD,

Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
class A {
	void accept(Visitor visitor) {
		visitor.doSomething(this);   ---> Visitor.doSomething(A value) { --->>//nothing
		//nothing
	}
}
class Visitor{
	void doSomething(A value) {
		//nothing
	}
...
Рейтинг: 0 / 0
23.11.2012, 12:29:27
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050290
Фотография Blazkowicz
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Blazkowicz
Участник
ivanra,
и какие тут бенефиты от визитора? в сравнении с обычным методом и его переопределением?
...
Рейтинг: 0 / 0
23.11.2012, 12:31:59
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050302
Фотография Blazkowicz
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Blazkowicz
Участник
Открываем хотя бы википедию и смотрим что даёт Visitor.
Его задача реализовать РАЗНЫЕ операции над разным элементами. Причем основное достоинство в том что много разных операций разделяются на разные визиторы с общим интерфейсом.
В данной же задаче есть единственный тип и, пока, единственная операция. Для каждого типа свой Visitor будет. 8)
...
Рейтинг: 0 / 0
23.11.2012, 12:37:41
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050328
ivanra
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
ivanra
Гость
Blazkowiczivanra,
и какие тут бенефиты от визитора? в сравнении с обычным методом и его переопределением?
Ну, фактически, визитор и есть завуалированное переопределение, только в самом классе метод переопределяется шаблоном, а все полезные действия выносятся в перегруженный метод визитора. В примере всё работает благодаря методу, переписанному под копирку
Код: java
1.
2.
3.
4.
5.
class A1 extends A{
	void accept(Visitor visitor) {
		visitor.doSomething(this);
	}
}


Фактически, это означает, что в рантайме класс A1 вызовет метод с сигнатурой visitor.doSomething(A1), вот и весь фокус.

0FD : пример рабочий, скопируйте и запустите
...
Рейтинг: 0 / 0
23.11.2012, 12:47:37
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050333
javapecker
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
javapecker
Участник
Не тратьте время на споры, еще страница и Petro123 все сведет к базе данных))
...
Рейтинг: 0 / 0
23.11.2012, 12:51:37
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050418
Фотография Petro123
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Petro123
Участник
javapeckerНе тратьте время на споры, еще страница и Petro123 все сведет к базе данных))
Угу))))
Или к вопросу- почему в java не любят RTTI ))
...
Рейтинг: 0 / 0
23.11.2012, 13:30:35
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050435
Фотография Blazkowicz
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Blazkowicz
Участник
http://www.javapractices.com/topic/TopicAction.do?Id=31
...
Рейтинг: 0 / 0
23.11.2012, 13:35:04
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050452
Фотография mayton
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
mayton
Участник
Blazkowicz +100

Но добавлю что даункастинги если они будут требуют обработки exception
что само по себе дорогостоящая штука. instanceof надо полагать работает
экономно т.к. генерирует одну инструкцию jvm "instanceof". Тоесть тут
как-бы надо подумать а на сколько "дорого" нам обработать неизвестный
Object который попал на вход нашего интерфейса.
...
Рейтинг: 0 / 0
23.11.2012, 13:39:38
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050561
Miha_S7
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Miha_S7
Гость
Blazkowicz http://www.javapractices.com/topic/TopicAction.do?Id=31

Думаю, по ссылке немного о другом речь. Там говорят, что логику метода нужно определять в самом методе, а не где-то в другом месте, выясняя сначала с помощью instanceof какую именно логику мы должны применить.
Думаю, автору вполне подойдёт это
 

Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
33.
34.
35.
36.
37.
38.
39.
40.
41.
42.
43.
44.
45.
public class XDevice
{
    public void doSomething()
    {
        System.out.println('Hello from XDevice');
    }
}

public class GoodDevice extends XDevice
{
    private static Boolean hold;
    
    public static void setHold( Boolean val ) 
    {
        hold = val;
    }
    
    @Override
    public void doSomething()
    {
        if ( !GoodDevice.hold )
        {
            System.out.println('Hello from GoodDevice');
        }
    }
}

public class BadDevice extends XDevice
{
    private static Boolean hold;
    
    public static void setHold( Boolean val ) 
    {
        hold = val;
    }
    
    @Override
    public void doSomething()
    {
        if ( !GoodDevice.hold )
        {
            System.out.println('Hello from BadDevice');
        }
    }
}


...
Рейтинг: 0 / 0
23.11.2012, 14:28:48
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050602
Фотография Blazkowicz
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Blazkowicz
Участник
Miha_S7Думаю, по ссылке немного о другом речь.

Именно об этом.

Miha_S7Там говорят, что логику метода нужно определять в самом методе, а не где-то в другом месте, выясняя сначала с помощью instanceof какую именно логику мы должны применить.

Автор темы именно это и делает. Выборку включаемых приборов выясняя применить к нему логику или нет. Опять же, соглашусь с тем что проект и задача сильно абстрактные и в контексте конкретного проекта могут быть другие решения.

Miha_S7Думаю, автору вполне подойдёт это

Это именно то что автору предложили ещё на первой странице.

Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
...
    private static Boolean hold;
    
    public static void setHold( Boolean val ) 
    {
        hold = val;
    }
...    
    private static Boolean hold;
    
    public static void setHold( Boolean val ) 
    {
        hold = val;
    }   
...


У меня в глазах двоится.
...
Рейтинг: 0 / 0
23.11.2012, 14:36:59
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050625
Фотография Blazkowicz
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Blazkowicz
Участник
Как вариант instanceof можно использовать с маркерным интерфейсом. Возможно этот вариант будет некотором компромисом между проверкой на конкретный тип и добавлением метода во всю иерархию. Да и интерфейс не обязательно делать маркерным. Можно в нем объявить метод "включить".

Код: java
1.
2.
3.
4.
5.
for (Device device : deviceList){
  if (device instanceof Activatable){
    ((Activatable)) device.activate();
  }
}
...
Рейтинг: 0 / 0
23.11.2012, 14:41:53
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050685
Фотография Blazkowicz
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Blazkowicz
Участник
Подумал ещё немного. Всё же я бы отказался от гетерогенного списка и унес логику фильтрации в саму коллецию.

Код: java
1.
2.
3.
4.
Collection<Activatable> activatables = devices.byType(Activatable.class);
for(Activatable activatable : activatables){
     activatable.activate();
}


Тогда у нас всегда будет запас оптимизировать коллекцию, если перебор станет слишком дорогим.
...
Рейтинг: 0 / 0
23.11.2012, 15:06:30
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050696
IDVsbruck
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
IDVsbruck
Участник
А если свести instanceof к работе с примитивами (или энумами)? К примеру,
Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
public enum DeviceType {A1, A2, A3}

class A1 {
    public DeviceType getType() {
        return DeviceType.A1;
    }
}

class A2 {
    public DeviceType getType() {
        return DeviceType.A2;
    }
}

class Controller {
    ...
    for (A device: devices)
        if (device.getType() == DeviceType.A1) {...}
    ...
}


То есть просто реализовываем аналогичную проверку на принадлежность потомка к нужному типу. Не так универсально, но чуть менее затратно. Хотя, в принципе, вариант с расширением абстрактного класса-контроллера или наследование контроллеров для однотипных девайсов от одного интерфейса выглядит заманчивее.
...
Рейтинг: 0 / 0
23.11.2012, 15:13:33
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050777
Miha_S7
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Miha_S7
Гость
BlazkowiczКак вариант instanceof можно использовать с маркерным интерфейсом. Возможно этот вариант будет некотором компромисом между проверкой на конкретный тип и добавлением метода во всю иерархию. Да и интерфейс не обязательно делать маркерным. Можно в нем объявить метод "включить".

Код: java
1.
2.
3.
4.
5.
for (Device device : deviceList){
  if (device instanceof Activatable){
    ((Activatable)) device.activate();
  }
}


Ну, к примеру
15:42 - мы хотим включить все жалюзи, а остальные девайсы не трогать. Если применять маркерный интерфейс, то мы должны вписать этот интерфейс в код класса. Ок, вписали, сделали, включили.
18:00 - хотим включить все световые приборы. Что делать? Для классов "световых" девайсов тоже интерфейс прописывать? Тогда они и в 15:42 сработают. Не? Или я где-то туплю?
...
Рейтинг: 0 / 0
23.11.2012, 15:45:39
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050826
Фотография Petro123
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Petro123
Участник
Miha_S7,
Причем тут тип класса в сабже и время?
...
Рейтинг: 0 / 0
23.11.2012, 16:03:31
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050870
Miha_S7
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Miha_S7
Гость
Petro123Miha_S7,
Причем тут тип класса в сабже и время?

Ну это же, чтобы проиллюстрировать, что классы мы определяем на этапе кодинга. А активировать/деактивировать функциональность метода какого-то класса нам может быть нужно в рантайме.
...
Рейтинг: 0 / 0
23.11.2012, 16:15:48
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050889
Фотография Petro123
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Petro123
Участник
Miha_S7Petro123Miha_S7,
Причем тут тип класса в сабже и время?

Ну это же, чтобы проиллюстрировать, что классы мы определяем на этапе кодинга. А активировать/деактивировать функциональность метода какого-то класса нам может быть нужно в рантайме.
Нет.
Время в минутах и RTTI т.е. сабж, совершенно разные вещи.
Просто вместо маркера от программиста уже есть маркер от компиляторы - пользуйся и не будь конфигурастом. ШУТКА.
...
Рейтинг: 0 / 0
23.11.2012, 16:24:10
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050892
javapecker
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
javapecker
Участник
Есть у меня похожие вопросы. Буду благодарен, если подскажете что-нить красивое.
Существует небольшое подобие анемичной модели. Объекты не содержат логики, только состояние. Различные действия с объектами выполняют соответствующие сервисы. В моем примере есть автомобиль и самолет, и соответствующие им сервисы. Так же там есть менеджер сервисов, который должен работать так: клиент( в смысле потребитель услуг моих сервисов) знает о классе объекта, с которым будет работать. Он хочет получить сервис, связанный с этим классом. Для этого он обращается к менеджеру сервисов, и передает ему класс объекта. А менеджер возвращает связанный с ним сервис. Не могу придумать, как избежать в менеджере перебора классов чтобы найти и отдать нужный сервис.
Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
33.
34.
35.
36.
37.
38.
39.
40.
41.
42.
43.
44.
45.
46.
47.
48.
49.
50.
51.
52.
53.
54.
public class Test {

    public static void main(String[] args) {
     
     IEntity  car = new ControllerManager().getController(Car.class).get();
     
    }   
    
   
}
interface IEntity{
    
}

class Car implements IEntity{
     
}

class Plane implements IEntity{
     
}

interface IEntityController{
    IEntity get();
}

class PlaneController implements IEntityController{

    @Override
    public Plane get() {
        return new Plane();
    }
    
}

class CarController implements IEntityController{

    @Override
    public Car get() {
        return new Car();
        
    }
    
}

class ControllerManager{
     IEntityController getController(Class<?> clazz){
        //тут хочу вернуть сервис для конкретной энтити
       //как-нибудь красиво 
        return null;
    }
  
     
}
...
Рейтинг: 0 / 0
23.11.2012, 16:25:09
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050909
Фотография Petro123
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Petro123
Участник
javapecker,
Ты просто перевернул с ног на голову.
Клиент не работает с объектами. Он вызывает Сервис - как хранимку- сделай то- то.
...
Рейтинг: 0 / 0
23.11.2012, 16:31:00
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050919
javapecker
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
javapecker
Участник
Petro123,
допустим перевернул, как все-таки быть для этого перевернутого случая?
...
Рейтинг: 0 / 0
23.11.2012, 16:33:26
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38050949
Фотография mayton
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
mayton
Участник
Если есть лексическая связь между entitiy и ее интерфейсом контроля то
можно привязаться к именам. Plane -> PlaceController, Car->CarController.
Ну и дальше через загрузчик классов.
...
Рейтинг: 0 / 0
23.11.2012, 16:43:15
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38051020
IDVsbruck
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
IDVsbruck
Участник
Можно и без имен: намертво привязать класс контроллера внутри класса сущности. И через вызов имплементированного метода сущности-наследника вызывать контроллер. Правда, эти способы не очень хорошо пахнут ...
...
Рейтинг: 0 / 0
23.11.2012, 17:09:45
| Ответить | Цитировать | Написать  
Какие альтернативы есть у instanceOf
    #38051033
javapecker
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
javapecker
Участник
Petro123,
Клиент не работает с объектами. Он вызывает Сервис - как хранимку- сделай то- то.
Ну пусть сделай то-то. Так вот сервис, который может сделать то-то, должен знать с чем это делать и с помощью чего. С чем у него есть, а с помощью чего - это мои сервисы самолетов и машин, и их надо как-то получить, как получают нормальные люди?
...
Рейтинг: 0 / 0
23.11.2012, 17:13:14
| Ответить | Цитировать | Написать  
25 сообщений из 60, страница 2 из 3
  2  все
Форумы / Java [игнор отключен] [закрыт для гостей] / Какие альтернативы есть у instanceOf
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение   Полит. конфиден.  
NoSQL.ru       Кролег: Проекты, Новости, Чат       РЕД ФОРУМ       Фотоальбом: Участники, Встречи
Закрыть
start [/forum/topic.php?fid=59&msg=38050278&tid=2129994]:
 0ms
get settings:
 17ms
get forum list:
 26ms
check forum access:
 7ms
check topic access:
 7ms
track hit:
 43ms
get topic data:
 19ms
get forum data:
 5ms
get page messages:
 116ms
get tp. blocked users:
 2ms
others: 296ms
total:  538ms
созд. / загр.: 538ms
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]