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.
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.