MapToForEach

Reports map calls whose results are ignored and can be replaced with forEach.

Using map in this way is misleading because map is intended to transform a collection and return the result. When its return value is not used, the call creates unnecessary allocations and reduces code readability. In such cases, forEach should be used instead, as it clearly expresses iteration with side effects.

The quick-fix replaces the map call with a forEach call.

Example:

fun foo(list: List<String>) {
  list.map { print(it) } // reported

  val x = list.map { it.length } // not reported: the result is used
}

After the quick-fix is applied:

fun foo(list: List<String>) {
  list.forEach { print(it) }

  val x = list.map { it.length } // not reported: the result is used
}