タイプ曖昧さのためのデザイン

Question

下記のシナリオでリファクタリングの提案を探しています。
私たちはDataMap<header,RecordHolder>です。
RecordHolderクラスの構造は、次のとおりです。
RecordA、RecordBは共通基底型から継承しています。

RecordHolder{ 
    RecordA 
    RecordB 
    RecordC 
    RecordD; 

    setRecordA(); 
    setRecordB(); 
    .... 
} 

パーサユーティリティは、ファイルの各行を読み取り、対応するレコード・タイプに変換：解析から戻った

Map<key,Record> parse(String line){....} 

、キーに対応するRecordHolderインスタンスがフェッチされるべきであり - そして解析されたレコードを更新するには、適切なsetRecordXXXメソッドを呼び出す必要があります。で

RecordHolder d = DataMap.get(key); 
if(d==null){ ==========> (A) 
    d = new RecorHolder(); 
    DataMap.put(key,d); 
} 
d.setRecordXXX(Record);=========>(B) 

質問：
（A）をいただきました！マップの取得、他の-入れ-とリターンのための最高のイディオム。
私はレコード（RECORDA、RecordBなどなど）の明示的な型チェックせずにこれを達成することができますどのように（B）

Answer 1

（A）をいただきました！マップの取得、他の-入れ-とリターンのための最高のイディオム。

次のコードのようにあなたは、Map#computeIfAbsent()方法を利用することができます：

DataMap.computeIfAbsent(key, k -> new RecordHolder()).setRecordXXX(Record); 

computeIfAbsentは、最初の引数として鍵を受け取り、第二引数Function<KeyType, RecordHolder>として。したがって、キーの値が存在しないとき。その後、関数が呼び出され、値として設定されます。

public V computeIfAbsent(K key, Function<K,V> mappingFunction){ 
    V value = this.get(key); 
    if(value == null){ 
     value = mappingFunction.apply(key); 
     this.put(key, value); 
    } 
    return value; 
} 

タイプKとVはジェネリックであり、それぞれkeyvalueの種類を表す：

は、マップ内の可能な実装は、（ただし、JDKから実際のない）、次のことができます。ご覧のように、メソッドcomputeIfAbsentはvalueを返します。これはマップ内にあるか、後で作成されたものです。その上でsetRecordXXXメソッドを呼び出すことができます。

私はあなたがif-elseif-else-statementを使用していますRecordHolderでメソッドを作成する必要があるだろうというのレコード（RECORDA、RecordBなどなど）

の明示的な型チェックせずにこれを達成することができますどのように（B）どのレコードを設定すべきかを決定する。

public void setRecordXXX(Record record){ 
    if(record instanceof RecordA){ 
     setRecordA((RecordA) record); 
    } else if(record instanceof RecordB){ 
     setRecordB((RecordB) record);   
    } else if(...){ 
     // other records 
    } else { 
     // throw when unknown record implementation 
     throw new UnsupportedOperation("RecordType not supported: " + record); 
    } 
}