結果が無視されているため、forEach に置換できるmap の呼び出しを報告します。
このように map を使用するのは誤解を招きます。map はコレクションを変換し、結果を返すことを意図しているためです。
その戻り値が使用されない場合、その呼び出しは不要なメモリ割り当てを発生させ、コードの可読性を低下させます。
このような場合には、副作用を伴う反復処理を明確に示す forEach を代わりに使用すべきです。
クイックフィックスを使用すると、map の呼び出しが forEach の呼び出しに置換されます。
例:
fun foo(list: List<String>) {
list.map { print(it) } // 報告されます
val x = list.map { it.length } // 報告されません: 結果は使用されます
}
クイックフィックス適用後:
fun foo(list: List<String>) {
list.forEach { print(it) }
val x = list.map { it.length } // 報告されません: 結果は使用されます
}