java inheritance передовая практика?

Question

У меня есть классы ниже.

public class BaseRequest { 
    protected List<Emp> names; // names property is required in many classes 
} 

public class FirstRequest extends BaseRequest { 

    public FirstRequest() { 
     names = new ArrayList<Emp>(); 
    } 

    public setNames(List<Emp> names){ 
     super.names = names; 
    } 

    public List<Emp> getNames(){ 
     return super.names; 
    } 
} 

public class ServiceClass { 

    public void someMethod(List<Emp> emps) { 
     FirstRequest request = new FirstRequest(); 
     request.setNames(emps); 
     //Some Logic 
    } 
} 

Я делаю наследование в правильном направлении? как я могу улучшить его дальше?

Спасибо!

Answer 1

Как я могу я улучшить его дальше?

Перенесите методы, связанные с names в базовый класс, и сделать namesprivate.

public class BaseRequest { 
    private List<Emp> names = new ArrayList<Emp>(); 

    public setNames(List<Emp> names){ 
     this.names = names; 
    } 

    public List<Emp> getNames(){ 
     return names; 
    } 
} 

public class FirstRequest extends BaseRequest { 
    // the rest of your stuff 
} 

Вы должны избегать names защищено: общественный поглотитель должен быть достаточно хорош для производных классов и для других пользователей класса.

Answer 2

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

FirstRequest request = new FirstRequest(); 

к

BaseRequest request = new FirstRequest(); 

и использовать геттеры/сеттер в суперклассе.

Кроме того, в конструкторе FirstRequest, вы должны поговорить с супер класс (сделать в ArrayList инициализации там, например)

public FirstRequest(){ 
super(); 
} 

и в супер классе

public BaseRequest() 
{ 
// initialisation 
} 

Answer 3

Вы можете конвертировать BaseRequest в интерфейс и предоставлять только подписи методов, которые будут использоваться клиентами. Клиент должен интересоваться только предоставленными методами (api), а не деталями реализации. Затем вы можете реализовать их во всех классах, реализующих интерфейс. Он является более гибким интерфейсом:

public interface BaseRequest { 
    List<Emp> getEmps(); 
    void setEmps(List<Emp> list); 
    .... 
} 


public class FirstRequest implements BaseRequest{ 

    List<Emp> getEmps(){ 
     return ...; 
    } 

    .... 
}