C# For-Schleife mit LISTE mit LINQ

Ich bin mit LINQ und die Rückgabe einer Liste, um meine Business-Logik-Schicht. Ich'mtrying ändern Sie einen der Werte in der Liste (die änderung der "star" - rating, um ein Bild mit der Anzahl der Sterne).

Obwohl der Zähler (i) scheint zu funktionieren, die FOR-Schleife nicht richtig funktioniert. Das erste mal durch, es Stoppt an der richtigen, WENN es dann aber öffnet sich bei der ELSE-Anweisung, die für alles und alle Werte am Ende mit "star0.png." Es scheint, als ob ich nicht mit dem Fahrrad durch die Liste??? Vielen Dank im Voraus!

    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";
        }
    }

Haben Sie versucht, mit einer foreach item in Liste-Schleife?
Die Schleife an sich sieht mir Recht sein. Was passiert, wenn Sie Schritt für Schritt durch diese in den debugger? Was sind die Werte der serviceCodes?
Haben Sie bestätigt den Inhalt ReviewList?
Klingt wie vielleicht eine gemeinsame Referenz.

InformationsquelleAutor Susan | 2011-08-18

c#for-loop linq list

2

Ich glaube nicht, dass Ihr eine Ausgabe der for loop richtig funktioniert... deine syntax ist gut und wie geschrieben, wird die Iteration ReviewList.Count # Male.

Ich würde Schritt für Schritt durch und überprüfen Sie den Inhalt der ReviewList ersten.

Lassen Sie mich wissen, was Sie finden
- Also hier ist was ich Tat. (1) geändert serviceCode einen Integer-Wert vergleicht ist der Identitätswert der Tabelle. Also, wenn SERVICe_ID = 1, REVIEWLIST[I].SERVICE.SERVICE_DESC = star1.png. es funktioniert perfekt. Da bin ich in der Lage, pull-DIENST.SERVICE_DESC im UI, ich verstehe nicht, warum es würde nicht funktionieren in meinem Bus Logic Layer. Irgendwelche Ideen??? Oh gut, Danke für die Hilfe!!!
- schwer zu sagen ohne den code anzuschauen, ganz... aber ich entnehme den Kommentaren, dass Sie Ihr Problem gelöst, das ist eine gute Sache 🙂
InformationsquelleAutor quashBox
3

Wenn alle Werte am Ende auf star0.png, dann sind Radfahren durch die Liste. Die Tatsache, dass die else - Anweisung wird der code nur ausgeführt wird, für jedes element, schlägt einen logischen Fehler -- hast du vielleicht meinst, so etwas zu tun?
```
string serviceCode = ReviewList[i].SERVICE.SERVICE_CODE;
```
- Das war meine Reaktion auch... "alle Werte" heißt Radfahren durch die Schleife funktioniert.
- Schöne Beobachtung!
- Der einzige Unterschied, den ich bemerken in Ihrem code, die Sie ersetzt haben SERVICE.SERVICE_DESC mit SERVICE.SERVICE_CODE. Letzteres ist eine ganze Zahl in meiner Tabelle und ist dem Identifizieren der Wert in dieser Tabelle.
- Ja, ich war darauf hindeutet, dass vielleicht Sie soll SERVICE_CODE und stattdessen SERVICE_DESC - basierend auf Ihren Kommentar, dies ist jedoch eindeutig nicht der Fall. Wenn Sie fügen Sie einen Haltepunkt in Ihrem else - Anweisung, welche die Werte von serviceCode?
- Vielen Dank für die Hilfe. Wenn Sie looke auf Antwort 0 ist, können Sie sehen, wie ich es endlich geschafft, die Arbeit war teilweise ein Ergebnis der Anregung, und die Anregung oben. Vielen Dank!
InformationsquelleAutor Donut
1

Wenn Sie wissen, jedes Element besteht aus einer Anzahl von Sternen, warum nicht diese?:
```
for (int i = 0; i < ReviewList.Count; i++)
{
    string serviceCode = ReviewList[i].SERVICE.SERVICE_DESC;
    ReviewList[i].SERVICE.SERVICE_DESC = "star" + serviceCode.Length + ".png";
}
```
- In der Theorie nicht berücksichtigt, für die else voll (auch wenn es vielleicht in der Praxis.)
- Sie müssten einige Sortieren oder prüfen Sie, dass jedes element die Zeichenfolge ein * - symbol, und überprüfen Sie die Länge um sicherzustellen, dass es arn nicht mehr als 4
InformationsquelleAutor StriplingWarrior

Schutz auf Doppel-pass und mit else-Bedingung

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";

   }
}

Alternative LINQ Ansatz

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

InformationsquelleAutor hungryMind

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.