java 8リストのインスタンスを使用してマップするリストを変更する

Question

Collectors.toMapコールを使用してリストをマップに変換しようとしています。リストはActivityReconcileオブジェクトで構成されています。リスト内のすべてのエントリのインスタンスをtoMap呼び出しに渡したいとします。

コードは以下のとおりです。インスタンスが必要な場所は??で示されます。

final List<ActivityReconcile> activePostedList = loader.loadActivePosted(accessToken); 
Map<AccountTransactionKey, ActivityReconcile> postedActiveMap = 
activePostedList.stream().collect(
Collectors.toMap(
AccountTransactionKey.createNewAccountTransactionKeyFromActivityReconcileRecord(??),??)); 

Answer 1

私が正しくあなたを理解している場合、あなたは

Map<AccountTransactionKey, ActivityReconcile> result = choices 
       .stream() 
       .collect(Collectors.toMap(
         AccountTransactionKey::generate, 
         Function.identity())); 

ようなものが必要になり、（AccountTransactionKeyクラスの）メソッドは、私が交換してきた

public static AccountTransactionKey generate(ActivityReconcile reconcile) {...} 

---

のようになります。 createNewAccountTransactionKeyFromActivityReconcileRecによってgenerate答えをより読みやすく分かりやすくするためです。

activePostedList.stream().collect(Collectors.toMap(
    ar -> AccountTransactionKey.createNewAccountTransactionKeyFromActivityReconcileRecord(ar)), 
    o -> o)); 

またはメソッド参照を使用します：

Answer 2

ラムダパラメータを追加、少なくとも変更であなたのコードを「修正」するためにところで

activePostedList.stream().collect(Collectors.toMap(
    AccountTransactionKey::createNewAccountTransactionKeyFromActivityReconcileRecord, o -> o)); 

を、私は見て今までに思い出すことができませんメソッド名がcreateNewAccountTransactionKeyFromActivityReconcileRecordの場合 - 可読性のため、戻り値の型とパラメータの型が他のファクトリメソッドと区別するのに十分であるため、それを単にcreate()に減らすことを検討してください。