Generally I agree with the sentiment, for one major reason: debugability/dev loops. Inevitably you're going to end up having to debug that each of those steps

Generally I agree with the sentiment, for one major reason: debugability/dev loops. Inevitably you're going to end up having to debug that each of those steps is correct, for that you'll find it a lot easier to break it out.. and the next person who has to do it will as well. I do think the example is somewhat loaded though, rename "result" to "top5ActiveUserNames" would do a lot there.