C# For Loop avec LIST utilisant LINQ

Question

J'utilise LINQ et renvoie une liste à ma couche logique métier. Je suis en train de changer l'une des valeurs de la liste (en changeant la note 'étoile' en une image avec le nombre d'étoiles).

Bien que le compteur (i) semble fonctionner, la boucle FOR ne fonctionne pas correctement. La première fois à travers elle s'arrête à la bonne IF, mais alors il sort à l'instruction ELSE pour tout et toutes les valeurs se terminent par "star0.png". Il semble que je ne suis pas à vélo dans la liste ??? Merci d'avance!

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

Je ne pense pas que son problème de for loop fonctionne correctement ... votre syntaxe est bonne et comme écrit itérera ReviewList.Count nombre de fois.

Je passerais en revue et vérifier le contenu de ReviewList en premier.

Laissez-moi savoir ce que vous trouvez

Answer 2

Si vous connaissez chaque élément sera composé d'un certain nombre d'étoiles, pourquoi ne pas le faire ?:

for (int i = 0; i < ReviewList.Count; i++) 
{ 
    string serviceCode = ReviewList[i].SERVICE.SERVICE_DESC; 
    ReviewList[i].SERVICE.SERVICE_DESC = "star" + serviceCode.Length + ".png"; 
} 

Answer 3

Si toutes les valeurs finissent par à star0.png, alors vous êtes à vélo par la liste. Le fait que l'instruction else soit le seul code exécuté pour chaque élément suggère une erreur logique - vouliez-vous peut-être faire quelque chose comme ça?

string serviceCode = ReviewList[i].SERVICE.SERVICE_CODE; 

Answer 4

Protection sur double passe et état autre

for (int i = 0; i < ReviewList.Count; i++) 
{ 
    string serviceCode = ReviewList[i].SERVICE.SERVICE_DESC; 
    if(!serviceCode.Contains(".png")) { // once name set should not be modified 
     if(serviceCode.Contains("*")) 
     ReviewList[i].SERVICE.SERVICE_DESC = "star" + serviceCode.Length + ".png"; 
     else 
     ReviewList[i].SERVICE.SERVICE_DESC = "star0.png"; 

    } 
} 

autre approche LINQ

 ReviewList.ForEach(rs=>if(!rs.SERVICE.SERVICE_DESC.Contains(".png")) 
{ rs.SERVICE.SERVICE_DESC = 
    "star" + rs.SERVICE.SERVICE_DESC.Length + ".png"});