peek（）を使用してストリーム要素を変更するのはアンチパターンですか？

モノのストリームがあり、それらを途中で「強化」したい場合peek()、これを行うために使用できます。たとえば：

streamOfThings.peek(this::thingMutator).forEach(this::someConsumer);

コードのこの時点でthingMutatorモノを変更することは正しい動作であると仮定します。たとえば、メソッドは「lastProcessed」フィールドを現在の時間に設定できます。

ただし、peek()ほとんどのコンテキストでは、「見て、触れないでください」という意味です。

ストリーム要素の突然変異にアンチパターンを使用peek()していますか？

編集：

より一般的な代替アプローチは、消費者を変換することです。

private void thingMutator(Thing thing) {
    thing.setLastProcessed(System.currentTimeMillis());
}

パラメータを返す関数に：

private Thing thingMutator(Thing thing) {
    thing.setLastProcessed(currentTimeMillis());
    return thing;
}

map()代わりに使用します：

stream.map(this::thingMutator)...

しかし、それは機能的なコード（return）を導入しpeek()、同じオブジェクトを返すことを知っているので、それがより明確であるとは確信していませんがmap()、同じクラスのオブジェクトであることは一目でさえわかりません。

さらに、peek()あなたは突然変異するラムダを持つことができmap()ますが、列車の残骸を構築する必要があります。比較する：

stream.peek(t -> t.setLastProcessed(currentTimeMillis())).forEach(...)
stream.map(t -> {t.setLastProcessed(currentTimeMillis()); return t;}).forEach(...)

私は思うpeek()ので、何の「神秘的」副作用はありません、バージョンが明確で、かつラムダは明らかに変異されます。同様に、メソッド参照が使用され、メソッドの名前が明らかに突然変異を暗示している場合、それも明確で明白です。

個人的には、peek()突然変異に使うことをためらわない-非常に便利だと思う。

あなたは正しいです、英語の意味での「覗く」は「見て、触れないで」という意味です。

ただし、JavaDocの状態：

ピーク

ストリームピーク（コンシューマアクション）

このストリームの要素で構成されるストリームを返します。さらに、結果のストリームから要素が消費されると、各要素で指定されたアクションを実行します。

キーワード：「実行中...アクション」および「消費済み」。JavaDocにはpeek、ストリームを変更する機能があることを期待すべきであることは非常に明確です。

ただし、JavaDocには次のことも記載されています。

API注：

このメソッドは、主にデバッグをサポートするために存在します。ここでは、パイプラインの特定のポイントを通過する要素を表示します。

これは、ストリーム内の要素をログに記録するなど、監視することを目的としていることを示しています。

このすべてから私が取るのは、ストリーム内の要素を使用してアクションを実行できることですが、ストリーム内の要素を変更することは避けるべきです。たとえば、先に進んでオブジェクトのメソッドを呼び出しますが、オブジェクトの操作を変更しないようにします。

少なくとも、次の行に沿ってコードに短いコメントを追加します。

// Note: this peek() call will modify the Things in the stream.
streamOfThings.peek(this::thingMutator).forEach(this::someConsumer);

このようなコメントの有用性に関する意見は異なりますが、この場合はこのようなコメントを使用します。

簡単に誤解される可能性があるため、このような使用は避けます。おそらく最良のオプションは、ラムダ関数を使用して、必要な2つのアクションをforEach呼び出しにマージすることです。また、既存のオブジェクトを変更するのではなく、新しいオブジェクトを返すことを検討することもできます-効率はわずかに低下する可能性がありますが、コードが読みやすくなり、変更されたリストを誤って使用する可能性を減らす可能性がありますオリジナルを受け取りました。

APIノートは、メソッドが主にデバッグ/ロギング/発火統計などのアクションのために追加されたことを示しています。

@apiNoteこのメソッドは、主にデバッグをサポートするために存在します。パイプライン内の特定のポイントを通過するエレメントを表示する場合は、次のようにします。

       {@コード
     * Stream.of（ "one"、 "two"、 "three"、 "four"）
     * .filter（e-> e.length（）> 3）
     * .peek（e-> System.out.println（ "フィルターされた値：" + e））
     * .map（String :: toUpperCase）
     * .peek（e-> System.out.println（ "マップされた値：" + e））
     * .collect（Collectors.toList（））;
     *}