Java継承とinstanceofの定数使用を避ける

Question

私は3つのクラス、抽象的なユーザと2つの特定のものを持っています：異なる（国内、国際、カスタムなど）可能な1つ以上のAddressオブジェクトのArrayListを保持するNormalUser trueを返すメソッドを持つAdminクラス。彼らはどちらもお互いに無関係なより多くのメソッドを含んでいます。私は、例えば次のように4ユーザーオブジェクトを作成する場合は、別のクラスの今

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; 
    } 
} 

：

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メソッドを使用したいと言う、私はにspecficユーザーをダウンキャストする必要がありますNormalUserへのユーザー、私はこのようなNormalUserでaddAddressメソッドを使用する前に：

私はNormalUserと管理者の両方がユーザーになりたい理由は、私はそれらを処理することができそうです

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

一緒にログインするとき。

UserクラスにaddEmailを追加し、それをNormalUserクラスで上書きすることを考えましたが、NormalUserクラスのすべてのメソッドに対してこれを行う必要があります。さらに、Adminはそれをユーザーがその機能を必要としないときも同様です。

質問1：instanceofの使用が悪いと聞いたので、これを行うには良い方法がありますか？ NormalUserクラスに固有のメソッドを使用するたびにinstanceofを使用する必要があります。

Quesiton 2：オブジェクトアドレスのArrayListは、RegularUserを特定のアドレス/（オブジェクト）にリンクする最適な方法ですか？

現在データベースはありません。ユーザCは、国内およびなど

おかげカスタムアドレスを持っている、

ですから、例えば、ユーザAが2つのアドレス国内1および国際的なものを持っており、ユーザBは、単に国内のアドレスを持っています。

PS。以前の投稿を広範囲に検索しましたが、解決策が見つかりませんでした。どちらのJavaの本でも、どちらもinstanceofの使用例を示していますが、悪いことは言及されていません。

Answer 1

Visitor patternを使用することができます。ちょっとぎこちなく、少し読めないかもしれませんが、おそらくあなたの問題に最適な解決策です。

実際、addEmailを基底クラスにプッシュするソリューションはそれほど悪くありません。ベースUserの空の実装を提供し、RegularUserをオーバーライドするだけです。 Userインスタンスで電子メールの追加がサポートされているかどうかを確認する場合は、supportsAddEmailのような別の方法を指定してください。addEmailを無効にすると、falseが返され、trueが返されます。

Answer 2

私は最も簡単な解決策は、NormalUserのリストとAdminのリストを含むクラスのUserListを作成することだと思います。 UserListクラスのインスタンスは、元のリストを置き換えます。クラスのUserListはいくつかの方法のような提供することができます：

• ユーザーのgetUser（インデックスi）//二つのリストで実装

• ユーザーはremoveUser（インデックスi）//二つのリストを用いて実装を

• NormalUser getNormalUser（インデックスi）//通常のユーザーリスト
• NormalUser removeNormalUser（インデックスi）//通常のユーザーリスト
• 管理GETADMIN（インデックスi）で実装して実装//実装adminユーザーリスト
• 管理removeAdmin（インデックスi）で// adminユーザーリスト
• ....

適切なリストを処理するためのすべてのコードはのUserListクラスにカプセル化されるだろうと実装。ユーザーが何をする必要があるかに応じて、両方のリストまたは1つのリストのみを使用するメソッドを持つことができます。 UserListと対話するクラスは、UserList内に1つまたは2つのリストしかないかどうかを知りません。