我有三个类,一个抽象用户和两个特定类:NormalUser 包含一个或多个地址对象的 ArrayList,这些对象可能不同(国内、国际、自定义等),然后是 Admin 类,它有一个返回方法真的 。它们都包含更多彼此无关的方法。
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;
}
}
现在在另一个类中,如果我像这样创建 4 个用户对象:
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");
然后说我想在 NormalUser 中使用 addAddress 方法,然后我必须将 users 中的特定用户向下转换为 NormalUser,然后才能像这样在 NormalUser 中使用 addAddress 方法:
if (user instanceof NormalUser){
NormalUser normal = (NormalUser) user;
normal.addAddress(...)
}
我之所以希望 NormalUser 和 Admin 都成为 User,是因为我可以在登录时将它们一起处理。
我想将 addEmail 添加到 User 类,然后在 NormalUser 类中覆盖它,但我必须对 NormalUser 类中的每个方法都这样做,而且 Admin 也会从 User 继承它,当它不需要该功能。
问题 1:我听说使用 instanceof 是一件坏事,有没有更好的方法来做到这一点?每次使用特定于 NormalUser 类的方法时,我都必须使用 instanceof。
问题 2:对象地址的 ArrayList 是将 RegularUser 链接到特定地址/(对象)的最佳方式吗?
目前不涉及数据库。
例如,用户 a 有 2 个地址,一个国内地址,一个国际地址,用户 b 只有一个国内地址,用户 c 有一个国内地址和一个自定义地址等。
谢谢。
附言。我已经广泛搜索了以前的帖子,但还没有找到解决方案。在我的两本 Java 书中,它们都展示了使用 instanceof 的示例,但没有提到这是一种不好的做法。
最佳答案
您可以使用 Visitor pattern - 有点笨拙且有点不可读,但可能是解决您问题的最佳方法。
实际上,将 addEmail
推送到基类的解决方案并没有那么糟糕。只需在基础 User
中提供一个空的实现并在 RegularUser
中覆盖。如果您想检查给定的 User
实例是否支持添加电子邮件,请提供另一种方法,例如 supportsAddEmail
默认返回 false
和 true
覆盖 addEmail
时。
关于Java 继承和避免经常使用 instanceof,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/8376818/