Java 继承和避免经常使用 instanceof

Question

我有三个类，一个抽象用户和两个特定类: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 的示例，但没有提到这是一种不好的做法。

Answer 1

您可以使用 Visitor pattern - 有点笨拙且有点不可读，但可能是解决您问题的最佳方法。

实际上，将 addEmail 推送到基类的解决方案并没有那么糟糕。只需在基础 User 中提供一个空的实现并在 RegularUser 中覆盖。如果您想检查给定的 User 实例是否支持添加电子邮件，请提供另一种方法，例如 supportsAddEmail 默认返回 false 和 true 覆盖 addEmail 时。