Является примером хорошей практики?

Question

У меня есть список объектов, которые простираются от базового класса. Теперь я хочу применить определенную операцию только к одному экземпляру классов в списке.

Является ли использование instanceof хорошей практикой? Или я должен отличать объекты, например, пользовательские enum?

abstract class Base; 
class Foo extends Base; 
class Bar extends Base; 

List<Base> bases; 

for (Base base : bases) { 
    if (base instanceof Bar.class) { 
    //execute my custom operation on the base object 
    doSomething((Bar) base); 
    } 
} 

Если такой подход не так хорош в целом, как я мог бы сделать лучше?

Answer 1

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

abstract class Base{ 
    public void doSomething(){} 
} 

public class B0 extends Base{ 
    @Override 
    public void doSomething(){//actually do something} 
} 

public class B1 extends Base{} 

Пример использования это может быть что-то вроде:

public class SomeOtherClass{ 
    public void something(List<Base> bases){ 
     for(Base base:bases) 
      base.doSomething(); 
    } 
} 

Answer 2

abstract class Base;//abstract function doSomething() 
class Foo extends Base;//implements doSomething() 
class Bar extends Base;//dito 

List<Base> bases; 

for (Base base : bases) { 
    base.doSomething(); 
} 

Чтобы ответить на ваш вопрос: использовать экземпляр instanceof не рекомендуется.

Answer 3

экземпляра не является хорошей практикой здесь.

Правильное решение будет зависеть от того, что именно происходит внутри этого метода doSomething. Если вы делаете это по-своему, то, помимо других вещей, вы нарушаете Liskov Substitution Principle. Я предполагаю, что вы решили, что вам нужна эта иерархия в первую очередь из-за чего-то, и я также предполагаю, что подтипы имеют некоторое поведение, а не только метод doSomething. В этом случае то, что вы можете сделать, показано ниже. В основном только те типы, которые должны на самом деле сделать это, и остальные типы делают что-то вроде no operation. Таким образом, вы можете использовать эти объекты, не зная, какой тип они на самом деле.

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

public abstract class Base 
{ 
    public abstract void doSomething(); 

    public void someOtherMethod() 
    { 
     // which does stuff 
    } 
} 

public class SubTypeWhichCanDoSomething extends Base 
{ 
    @Override 
    public void doSomething() 
    { 
     // actually implement method and DO something 
    } 
} 

public class DoesNothing extends Base 
{ 
    @Override 
    public void doSomething() 
    { 
     // does nothing 
     return; 
    } 
} 

// then your code looks like these 
for(Base base : bases) 
{ 
    base.doSomething(); 
}