結果が無視されているため、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 }  // 報告されません: 結果は使用されます
}