Herencia de Java y evitar el uso constante de instanceof

Question

Tengo tres clases, un usuario abstracto y dos específicos: NormalUser que contiene una ArrayList de uno o más objetos de dirección que pueden ser diferentes (doméstico, internacional, personalizado, etc.) y luego el Clase de administrador que tiene un método que devuelve verdadero. Ambos contienen más métodos que no están relacionados entre sí.

abstract class User{ 
    public User(String username, String pw){ 
... 

} 

public class NormalUser extends User{ 
... 
    private ArrayList<Address> addresses; 

... 

    public void addAdress(ArrayList<Address> address){ 
     addresses.addAll(address); 
} 

public class Admin extends User{ 

... 
    public boolean getIsAdmin(){ 
     return true; 
    } 
} 

Ahora en otro ramo, si hago 4 objetos de usuario así por ejemplo:

ArrayList<User> users; 

    users.add(new NormalUser("1", "pw"); 
    users.add(new NormalUser("2", "pw"); 
    users.add(new NormalUser("3", "pw"); 
    users.add(new NormalUser("4", "pw"); 
    users.add(new Admin("5", "pw")); 
    users.add(new NormalUser("6", "pw"); 

Y digo que quiero utilizar el método addAddress en NormalUser, entonces tengo que abatido el usuario specfic en usuarios a NormalUser, antes de que pueda utilizar el método addAddress en NormalUser así:

 if (user instanceof NormalUser){ 
     NormalUser normal = (NormalUser) user; 
     normal.addAddress(...) 
     } 

la razón por la que me gustaría tanto NormalUser y de administración para ser un usuario es para que pueda procesarlos juntos al iniciar sesión.

Pensé en agregar addEmail a la clase User y luego anularla en la clase NormalUser, pero tendría que hacer eso para cada método en la clase NormalUser, además Admin lo heredaría de la Usuario también, cuando no necesita esa funcionalidad.

Pregunta 1: ¿Hay una manera mejor de hacer esto que he escuchado usando instanceof es algo malo? y tendría que usar instanceof cada vez que utilizo un método que es específico de la clase NormalUser.

Quesiton 2: es una ArrayList de objeto ¿Direcciones de la mejor manera de vincular el Usuario Regular a direcciones específicas/(Objetos)?

No hay ninguna base de datos involucrada en este momento.

Así, por ejemplo, un usuario tiene 2 direcciones uno interno y otro internacional, y el usuario B solo tiene una dirección doméstica, el usuario C tiene una interna y una dirección personalizada etc.

Gracias.

PS. He buscado las publicaciones anteriores extensamente pero no he encontrado una solución. En ambos libros de Java, ambos muestran ejemplos de uso de instanceof, pero no se menciona que sea una mala práctica.

Answer 1

Puede usar el Visitor pattern - un poco torpe y poco legible, pero probablemente la mejor solución para su problema.

En realidad, su solución con empujar addEmail a la clase base no es tan malo. Simplemente proporcione una implementación vacía en la base User y anule en RegularUser. Si desea verificar si la instancia dada de User admite la adición de correos electrónicos, proporcione otro método como supportsAddEmail que devuelva false de manera predeterminada y true al reemplazar addEmail.

Answer 2

Creo que la solución más fácil es crear una lista de usuarios de clase que contendría una lista de Usuario Normal y una lista de Admin. Una instancia de la UserList de clase reemplazaría la lista original.El UserList clase podría proporcionar algunos métodos tales como:

• usuario getUser (índice i) // implementado con las dos listas

• removeuser de usuario (índice i) // implementado con las dos listas

• NormalUser getNormalUser (índice i) // implementado con la lista normal de usuario
• NormalUser removeNormalUser (índice i) // implementado con la lista de usuarios normales
• de administración GetAdmin (índice i) // implementado con la lista de usuario de administración
• administración removeAdmin (índice i) // en práctica con la lista de usuario admin
• ....

Todo el código para el manejo de las listas apropiadas se encapsula en la clase UserList . Podría tener métodos que usen ambas listas o solo una, según lo que necesite hacer con los usuarios. Las clases que interactúan con UserList no sabrían si hay solo una o dos listas dentro de UserList.