Yes, ForEach isn't idiomatic but he could use Select instead.
Side effects in a Select is not idiomatic either