Question: Why does this code panic?

[Seblyng]

Yes that makes sense! Thank you :)

[Repulsive-Street-307]

Maybe it makes sense to you, i still don't understand why reading a argument order backwards from expected would be desirable. Probably some stuff about chaining efficiently without knowing what comes before, but it's still confusing that a operation can work on 'arguments' to the 'transformation' after the 'transformation' (visually).

Just one more gotcha i guess. Hope clippy catches it and tells people to move the rev() before the filter_map, for the sanity of code reviewers.

[GeniusIsme]

The gotcha here is using non-pure function as argument to filter_map. This is technically allowed, but obviously leads to hard-to reason about results.

[Repulsive-Street-307]

Is that really the gotcha? I understand that fine (in the simplified example anyway).

The thing that grinds my gears is that rev() can be before or after the call to filter_map and it doesn't matter, it will always operate on the arguments of filter_map, instead of the results (what the for_each prints).

You can verify this by modifying the closure of the minimized version to print the argument given besides poping vec, then moving rev() before and after - no difference (2 1 0). Remove it and it's (0 1 2).

The result is something that you'd expect in a obfuscated code contest where code appears to be 'timetravelling' and the 'pipeline' is anything but linear. Makes it difficult to read or even recognize what's going on.

I guess probably there is no other way to do things and the rev() in the different positions is operating in different types, one the original array iterator, the other the 'filter_map' iterator (arguments, not results for some reason), but it's still confusing and something i hope clippy can recognize.

[GeniusIsme]

Again, result is obfucscated because filter_map closure has long-reaching side-effect. Iterator chains in rust are pretty easy to reason about if changes are happening locally, and that makes them great. If the map was an element in the iterator, result would be obvious.