Question

I’m using LINQ and returning a list to my Business Logic Layer. I’mtrying to change one of the values in the list (changing the ‘star’ rating to an image with the number of stars).

Although the counter (i) appears to be working, the FOR loop is not working correctly. The first time through it stops at the correct IF but then it pops out at the ELSE statement for everything and all values end up with “star0.png.” It appears as though I’m not cycling through the list??? Thanks in advance!

    for (int i = 0; i < ReviewList.Count; i++)
    {
        string serviceCode = ReviewList[i].SERVICE.SERVICE_DESC;
        if (serviceCode == "*")
        {
            ReviewList[i].SERVICE.SERVICE_DESC = "star1.png";
        }
        else if (serviceCode == "**")
        {
            ReviewList[i].SERVICE.SERVICE_DESC = "star2.png";
        }
        else if (serviceCode == "***")
        {
            ReviewList[i].SERVICE.SERVICE_DESC = "star3.png";
        }
        else if (serviceCode == "****")
        {
            ReviewList[i].SERVICE.SERVICE_DESC = "star4.png";
        }
        else
        {
            ReviewList[i].SERVICE.SERVICE_DESC = "star0.png";
        }
    }

Answer 1

I dont think its an issue of the for loop working properly… your syntax is good and as written will iterate ReviewList.Count # of times.

I would step through and verify the contents of ReviewList first.

Let me know what you find