Javaストリームが副作用でマップされ、結果リストを収集またはforeachして移入する

Question

私はこのようなコードを持っています。

私はこれに関して2つの矛盾する（？） "ルール"を読んでいます。 http://files.zeroturnaround.com/pdf/zt_java8_streams_cheat_sheet.pdfで述べたように.mapは副作用

.foreachが 更新変更可能な変数はならないことを（私は、その後、foreachのを使用して 結果リストを移入するリファクタリング場合はそのことを壊すこと）を持つべきではないこと

ストリームを使用してもリストを返すようにするにはどうすればできますか？ストリームをスキップするだけですか？

@Transactional 
public Collection<Thing> save(Collection<Thing> things) { 
    return things.stream().map(this::save).collect(Collectors.toList()); 
} 

@Transactional 
public Thing save(Thing thing) { 
    // org.springframework.data.repository.CrudRepository.save 
    // Saves a given entity. Use the returned instance for further operations as the save operation might have changed the entity instance completely. 
    Thing saved = thingRepo.save(thing); 
    return saved; 
} 

Answer 1

可変状態とは言わないのですか？あなたの場合、メソッドの中でリストを宣言してからforEachを使用すれば、すべて問題ありません。 2番目の答えhereは、あなたがしようとしていることを正確に示しています。

Answer 2

あなたがすべてでそれを変異させていない場合は、全く新しいListを収集する理由にはほとんどがあります。あなたのユースケースは、基本的にコレクション内のすべての要素を繰り返し処理しており、単にfor-eachを使用することで実現できます。

hingRepo.save(thing)がオブジェクトを変異させても、同じコレクションを返すことができますが、この時点では、hingRepo.save(thing)はそれを示唆していない隠れた変異があります。