Linq Любой() против Еогеасп

Question

Просто интересно, какой подход быстрее и лучше использовать, или которые вы предпочитаете

bool userHavePermission = user.Permissions.Any(x => x.UpperName == "ADMINISTRATOR"); 

или

foreach (Permission p in _Permissions) 
{ 
    if (p.UpperName == name.ToUpper()) 
     return true; 
} 
return false; 

Благодаря

Answer 1

Это почти тот же код, то только с тем, что со вторым фрагментом кода вы получите NullReferenceException во время выполнения, если переменная name равна null, потому что вы будете звонить .ToUpper() метод на нулевом экземпляре. Первый выглядит короче, безопаснее и читабельнее, это то, что я буду использовать. И для обеспечения того, чтобы не было никаких НЭО:

return user 
    .Permissions 
    .Any(x => string.Equals(x.UpperName, name, StringComparison.OrdinalIgnoreCase)); 

Answer 2

Использование Any - лучший подход, поскольку это одна строка. Он читает легче и занимает меньше места.

Кроме того, неясно, что такое объект Permissions, но если это объект somekind, представляющий таблицу базы данных, то Any определенно лучше, поскольку вы возвращаете результат запроса только там, где foreach собирается разрешить весь список разрешений до начала итерации.