ストリームでCollections.toMap()を使用するときに、リストの反復順序を維持するにはどうすればよいですか?


82

私は次のようにMapからを作成してListいます:

List<String> strings = Arrays.asList("a", "bb", "ccc");

Map<String, Integer> map = strings.stream()
    .collect(Collectors.toMap(Function.identity(), String::length));

と同じ反復順序を維持したいと思いListます。メソッドLinkedHashMapを使用してを作成するにはどうすればよいCollectors.toMap()ですか?


1
以下の私の答えを確認してください。これは、カスタムを使用してコードの唯一の4行であるSupplierAccumulatorCombinerのためにcollectあなたの方法stream:)
hzitoun

1
質問はすでに回答済みです。この質問に対する回答を見つけるためのパスをレイアウトしたいと思います。(1)マップで順序を設定する場合は、LinkedHashMapを使用する必要があります。(2)Collectors.toMap()には多くの実装があり、そのうちの1つはマップを要求します。したがって、Mapが必要な場所ではLinkedHashMapを使用してください。
SatyendraKumar19年

回答:


116

2パラメータバージョンはCollectors.toMap()HashMap使用します

public static <T, K, U> Collector<T, ?, Map<K,U>> toMap(
    Function<? super T, ? extends K> keyMapper, 
    Function<? super T, ? extends U> valueMapper) 
{
    return toMap(keyMapper, valueMapper, throwingMerger(), HashMap::new);
}

4パラメータバージョンを使用するには、次のものを置き換えることができます。

Collectors.toMap(Function.identity(), String::length)

と:

Collectors.toMap(
    Function.identity(), 
    String::length, 
    (u, v) -> {
        throw new IllegalStateException(String.format("Duplicate key %s", u));
    }, 
    LinkedHashMap::new
)

または、少しクリーンにするために、新しいtoLinkedMap()メソッドを作成して使用します。

public class MoreCollectors
{
    public static <T, K, U> Collector<T, ?, Map<K,U>> toLinkedMap(
        Function<? super T, ? extends K> keyMapper,
        Function<? super T, ? extends U> valueMapper)
    {
        return Collectors.toMap(
            keyMapper,
            valueMapper, 
            (u, v) -> {
                throw new IllegalStateException(String.format("Duplicate key %s", u));
            },
            LinkedHashMap::new
        );
    }
}

2
なぜそのような複雑さ?あなたは簡単に以下の私の答えをチェック、それを行うことができます
hzitoun

1
mergeFunctionことは、4パラメータバージョンでCollectors.toMap()キーが両方とも、マージされているかについての手掛かりがないuv値ですが。したがって、IllegalStateExceptionに関するメッセージは正しくありません。
MoonFruit

1
@hzitounを使用するだけではMap::put、同じキーに対して(おそらく)異なる値になるためです。を使用することによりMap::put、最初に出くわした値が正しくないことを間接的に選択しました。それはエンドユーザーが望んでいることですか?本気ですか?はいの場合は、確かに、を使用してくださいMap::put。それ以外の場合は、確信が持てず、決定できません。ストリームが異なる値を持つ2つの同一のキーにマップされていることをユーザーに知らせます。私はあなた自身の答えにコメントしたでしょうが、それは現在ロックされています。
オリヴィエグレゴワール

68

あなた自身を提供しSupplierAccumulatorそしてCombiner

    List<String> myList = Arrays.asList("a", "bb", "ccc"); 
    // or since java 9 List.of("a", "bb", "ccc");
    
    LinkedHashMap<String, Integer> mapInOrder = myList
        .stream()
        .collect(
            LinkedHashMap::new,                                   // Supplier
            (map, item) -> map.put(item, item.length()),          // Accumulator
            Map::putAll);                                         // Combiner

    System.out.println(mapInOrder);  // {a=1, bb=2, ccc=3}

1
@Sushil承認された回答により、ロジックを再利用できます
シリンダー

1
ストリームは、map.put(..)どちらが間違っているかなどの副作用に依存しています。
NikolasCharalambidis19年

何が速いか知っていますか?あなたのバージョンまたは受け入れられた答えを使用しますか?
nimo23

@ニコラス副作用とはどういう意味ですか?:私は実際に選択するかを決定する問題が持っているstackoverflow.com/questions/61479650/...
nimo23

0

Kotlinでは、toMap()順序を維持しています。

fun <K, V> Iterable<Pair<K, V>>.toMap(): Map<K, V>

指定されたペアのコレクションからのすべてのキーと値のペアを含む新しいマップを返します。

返されたマップは、元のコレクションのエントリの反復順序を保持します。2つのペアのいずれかが同じキーを持っている場合、最後のペアがマップに追加されます。

その実装は次のとおりです。

public fun <K, V> Iterable<Pair<K, V>>.toMap(): Map<K, V> {
    if (this is Collection) {
        return when (size) {
            0 -> emptyMap()
            1 -> mapOf(if (this is List) this[0] else iterator().next())
            else -> toMap(LinkedHashMap<K, V>(mapCapacity(size)))
        }
    }
    return toMap(LinkedHashMap<K, V>()).optimizeReadOnlyMap()
}

使用法は単純です:

val strings = listOf("a", "bb", "ccc")
val map = strings.map { it to it.length }.toMap()

の基になるコレクションmapLinkedHashMap(挿入順)です。


0

いくつかのフィールドによってオブジェクトの配列をマップする単純な関数:

public static <T, E> Map<E, T> toLinkedHashMap(List<T> list, Function<T, E> someFunction) {
    return list.stream()
               .collect(Collectors.toMap(
                   someFunction, 
                   myObject -> myObject, 
                   (key1, key2) -> key1, 
                   LinkedHashMap::new)
               );
}


Map<String, MyObject> myObjectsByIdMap1 = toLinkedHashMap(
                listOfMyObjects, 
                MyObject::getSomeStringField()
);

Map<Integer, MyObject> myObjectsByIdMap2 = toLinkedHashMap(
                listOfMyObjects, 
                MyObject::getSomeIntegerField()
);
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.