C# Linq Intersect

Question

В большинстве случаев я просто использую операторы foreach, потому что это намного проще для меня кодировать. Позже, когда мой код будет выполнен, я конвертирую foreach в операторы LINQ, где это имеет смысл.

Я хочу научиться писать хорошие читаемые заявления LINQ. Как вы, ребята, конвертируете этот код foreach в правильный оператор LINQ?

private List<QARoles> GetUserRoles(User user) 
{ 
    //TODO: Fix this with LINQ intersect. 

    var result = new List<QARoles>(); 
    foreach (var role in user.Roles) 
    { 
     foreach (QARoles qarole in System.Enum.GetValues(typeof(QARoles))) 
     { 
      if (role.Name.Equals(qarole.ToString())) 
      {       
       result.Add(qarole); 
      }      
     } 
    } 
    return result; 
} 

Answer 1

Это может быть упрощена:

var result = user.Roles 
       .Where(r => Enum.IsDefined(typeof(QARoles), r.Name)) 
       .Select(r => (QARoles)Enum.Parse(typeof(QARoles), r.Name)) 
       .ToList(); 

Answer 2

return (from role in user.Roles 
     from QARoles qarole in System.Enum.GetValues(typeof(QARoles)) 
     where role.Name.Equals(qarole.ToString()) 
     select qarole) 
.ToList(); 

или

return user.Roles.SelectMany(role => System.Enum.GetValues(typeof(QARoles)).Cast<QARoles>(), 
      (role, qarole) => new {role, qarole}) 
     .Where(t => t.role.Name.Equals(t.qarole.ToString())).Select(t => t.qarole).ToList(); 

Answer 3

Вы можете сделать это:

var roles=System.Enum.GetValues(typeof(QARoles)); 
return roles.Where(r=>user.Roles.Any(role=>role.Name.Equals(r.ToString())).ToList(); 

Answer 4

использованием ReSharp эр:

  return (from role in user.Roles 
       from QARoles qarole 
       in System.Enum.GetValues(typeof(QARoles)) 
       where role.Name.Equals(qarole.ToString()) 
       select qarole) 
       .ToList(); 

Answer 5

return System.Enum.GetValues(typeof(QARoles)).Where(role => 
user.Roles.Any(r => r.Name == role.ToString())) 

Answer 6

Я придумал это сам:

return user.Roles 
     .Select(r => r.Name).ToList<string>() 
     .Select(str => 
     { 
      QARoles qarole; 
      bool success = System.Enum.TryParse(str, out qarole); 
      return new { qarole, success }; 
     }) 
     .Where(pair => pair.success) 
     .Select(pair => pair.qarole) 
     .ToList(); 

Но я не думаю, что он стал лучше читаемым ;-)