結果が無視され、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 }  // 報告されない: 結果が使用される
}