ジェネリックを使用してinstanceofを回避する方法はありますか？

Question

私は複数の場所でエラーメッセージと成功メッセージを生成できるプロセスを持っているので、それらをすべて1つのクラスで追跡し、プロセスの最後に、さまざまなタイプのメッセージをグループ化したいと思います。 これらの行には、何か：

public class Message { 
    String message; 
    public Message(String message) { 
     this.message = message; 
    } 
} 

public class ErrorMessage extends Message {}; 
public class FileErrorMessage extends ErrorMessage {}; 
public class OkMessage extends Message {}; 

などがあります。 （わかりやすくするために派生クラスのコンストラクタをスキップしています）。私はinstanceofオペレータ、しかしを使用してリストの中で区別することができます。ジェネリックスはよりエレガントだと思いますが、どのようにリスト内でそれらを区別しますか？

Message<ErrorMessage> eMsg = new Message<ErrorMessage>("invalid user"); 
Message<FileErrorMessage> feMsg = new Message<FileErrorMessage>("file not found"); 

私は別のクラスの列挙型

enum MessagType { ERROR, FILE_ERROR, OK } 

を使用するのではと思ったが、私は解決策を考え出すことができませんでした。ありがとう。

Answer 1

このような状況で通常できることは、make Messageがサブクラスによってオーバーライドできる関数を持つことです。次に、各サブクラスは、その関数が呼び出されるときに独自の動作を持つことができます。あるいは、Messagesのデフォルト動作を保持することもできます。デフォルトが意味を持たない場合、Messageを抽象クラスにすることができます。

その後することができますリストをループしてのようなものを実行します。これはあなたが探していたされているもの

for (Message m : messages) { 
    m.function(); 
} 

願っています！

編集：以下のコメントを受けて、あなたは（グアバと）このような何かを行うことができます。

SetMultimap<Class, Message> messagesByType = HashMultimap.create(); 
for (Message m : messages) { 
    messagesByType.add(m.getClass(), m); 
} 

その後、一度にそれぞれ異なるタイプのものを処理するためにmessagesByTypeて、ネストされたループを行うことができます。それが言われている、私は実際にこれを行う必要はありませんが、私の最初の応答を与えて、しかしこれはあなたの質問に答えることです。

Answer 2

この場合、ジェネリックスを使用することは私には意味がありません。異なるタイプのメッセージが異なる動作をしている場合、継承を使用してメッセージのタイプによって区別することは意味がありますが、これは当てはまりません。あなたがKISS principleを参照して、それをシンプルに保つ必要があり、一般的には

enum MessageType { ERROR, FILE_ERROR, OK } 

public class Message { 

    private final String message; 

    private final MessageType type; 

    public Message(String message, MessageType type) { 
     this.message = message; 
     this.type = type; 
    } 

    public MessageType getType() { 
     return this.type; 
    } 

    // getter for message 
} 

：

だから、私はちょうどMessageクラスの属性として、メッセージの種類を持っているでしょう。

EDIT：あなたはタイプ別にグループメッセージにしたい場合は、次のように

、あなたがそれを行うことができます。

Map<MessageType, Message> messagesByType = messages.stream() 
    .collect(Collectors.groupingBy(Message::getType)); 

messagesは、すべてのメッセージを含むListです。