Readability/subjective. I think it's easier to read as/is with the subs separated. In terms of cost, this is something that happens once per run, so we're talking about iterating twice over a list that's usually less than 10 items (very minor cost, probably not even bench-markable). That said, I don't feel strongly enough about it not to change it.