Mehrere wo Bedingungen in EF mit Lambda-Ausdrücke

Question

Ich benutze Entity Framework und Linq Query, ich möchte einige Daten mit einer where-Klausel mehrere auswählen.

In meinem DB habe ich in einem Feld mehrere IDs FieldOfInterestID, getrennt durch ";" (Ich weiß, ich weiß, aber es ist zu spät, etwas zu tun), oder nur eine ID, das bedeutet, dass das Komma nicht vorhanden sein wird

Ich möchte die Zeichenfolge mit dem ";" Trennzeichen und als all diese IDs, um sie in meiner WHERE-Klausel zu verwenden.

Mein Code sieht in etwa wie folgt aus:

await ctx.Customer.AsNoTracking() 
    .Where(e => e.UserId == userId) 
    .Select(e => new UserDTO { 
     FieldsOfStudy = ctx.Terms.Where(t => { 
      if (!e.FieldOfInterestID.Contains(";") && t.TermId.ToString() == e.FieldOfInterestID) 
       return true; 
      else if (e.FieldOfInterestID.Contains(";") 

      { 
       string fieldOfInterestIds = e.FieldOfInterestID.Split(";"); 
       foreach (string fieldOfInterestID in fieldOfInterestIds) 
       { 
        if (t.TermId.ToString() == e.FieldOfInterestID) 
         return true; 
        else 
         return false; 
       } 
      } 
      else 
       return false; 
     } 
    }) 
    .ToListAsync().ConfigureAwait(false); 

Mein aktueller "Fehler" ist not all code paths return a value...

Wie kann ich string [] in meiner where-Klausel besser nutzen?

Answer 1

Der aktuelle Fehler ist wegen dieser Zweig:

foreach (string fieldOfInterestID in fieldOfInterestIds) 
{ 
    if (t.TermId.ToString() == e.FieldOfInterestID) 
     return true; 
    else 
     return false; 
} 

Vom Compiler Sicht den Körper des foreach darf nicht eingegeben werden, daher not all code paths return a value....

Sie können das Problem beheben, indem Sie den return false; außerhalb des Körpers zu bewegen:

foreach (string fieldOfInterestID in fieldOfInterestIds) 
{ 
    if (t.TermId.ToString() == e.FieldOfInterestID) 
     return true; 
} 
return false; 

Dadurch wird der Compiler-Fehler beheben, aber das Problem nicht lösen. Sobald Sie es ausführen, werden Sie feststellen, dass LINQ to Entities keine Lambda-Ausdrücke mit der Methode body (=> { ... }) und string.Split unterstützt.

Die wirkliche Lösung erfordert ein anderes Kriterium - anstelle von nicht unterstützten

e.FieldOfInterestID.Split(";").Contains(t.TermId.ToString()) 

das Gegenteil aber unterstützt (verwendet String-Verkettung und string.Contains)

(";" + e.FieldOfInterestID + ";").Contains(";" + t.TermId + ";") 

beide Strings mit ; Enclosing benötigte korrekt zu handhaben Erste, Zwischen- und letzte Token.

Die letzte Abfrage wie folgt sein könnte:

var query = ctx.Customer.AsNoTracking() 
    .Where(e => e.UserId == userId) 
    .Select(e => new UserDTO 
    { 
     FieldsOfStudy = ctx.Terms 
      .Where(t => (";" + e.FieldOfInterestID + ";").Contains(";" + t.TermId + ";")) 
    }); 

Answer 2

Diese Linie scheint mir falsch:

string fieldOfInterestIds = e.FieldOfInterestID.Split(";");

ich es ändern würde:

string[] fieldOfInterestIds = e.FieldOfInterestID.Split(";");

Auch ich würde vereinfachen den Code ein wenig:

ctx.Terms.Where(t => return e.FieldOfInterestID.Split(";").Contains(t.TermId.ToString()));

Wenn die Zeichenfolge enthält nicht die ; Zeichen es sich um eine Zeichenfolge zurückgibt [] mit einem Element