 <p>If <strong>all values</strong> end up at <code>star0.png</code>, then you <em>are</em> cycling through the list. The fact that the <code>else</code> statement is the only code being executed for each element suggests a logical error -- did you perhaps mean to do something like this?</p> <pre><code>string serviceCode = ReviewList[i].SERVICE.SERVICE_CODE; </code></pre>