As I just posted to his blog:<p>How about instead you use proper names? &quot;Results&quot; is a terrible name. So is &quot;i&quot;. That is why you can&#x27;t understand that code. Your &#x27;fix&#x27; 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.<p>This is completely readable, IMO:<p><pre><code>    for (auto employee : sorted_employees) {
        StoreEmpoyeeInFoo(employee);
    }</code></pre>