Comment by RogerL

As I just posted to his blog:

How about instead you use proper names? "Results" is a terrible name. So is "i". That is why you can't understand that code. Your 'fix' is still unreadable - the struct is rarely near the for loop, so you are going to have to go searching anyway to figure out what the hell is going on.

This is completely readable, IMO:

    for (auto employee : sorted_employees) {
        StoreEmpoyeeInFoo(employee);
    }