更好的方法让用户不属于任何角色？

Question

我编写的以下内容确实有效，但想知道是否有更有效的方法来获取所有没有任何角色的用户。

using System.Collections.Generic;
using System.Linq;
using System.Web.Security;

public static IEnumerable<MembershipUser> GetUsersHavingNoRole() {
  var allUsers = Membership.GetAllUsers().Cast<MembershipUser>();
  foreach (var user in allUsers) {
    if (Roles.GetRolesForUser(user.UserName).Length.Equals(0)) {
      yield return user;
    }
  }
}

Answer 1

我预计用户通常比角色多 - 因此迭代角色可能是有意义的（由 Roles.GetAllRoles() 返回），建立一个任何这些角色中的用户列表（例如，通过创建 HashSet 并为 Roles.GetUsersInRole 返回的每个角色添加用户），然后查找差异在该用户和所有用户之间。因此，如果您使用 LINQ，则可以使用：

var usersInRolesQuery = Roles.GetAllRoles()
                             .SelectMany(role => Roles.GetUsersInRole(role));

var usersInRoles = new HashSet<string>(usersInRolesQuery);
return Membership.GetAllUsers()
                 .Cast<MembershipUser>()
                 .Where(user => !usersInRoles.Contains(user.UserName));

当然，这仍然处理相同数量的数据 - 但它可能意味着到所涉及的任何数据存储的往返次数更少。

您是否对应用程序进行了基准测试以了解当前方法的成本如何？

Answer 2

根据用户、角色和它们之间的关系的数量以及您需要此信息的次数，可能值得（有足够的免责声明了吗？）向 (自定义）角色提供者。

提供程序中所需的本机查询可能比您现在使用 LINQ 的方式高效得多。

很大程度上取决于这个答案第一行的因素，也取决于提供商。可维护性也可能是一个问题，因为您添加的查询将来可能不受支持（提供程序的下一版本）

Answer 3

如果我正确理解你的问题。这段代码应该有帮助！

MembershipUserCollection users = Membership.GetAllUsers();
MembershipUserCollection usersNoRoles = new MembershipUserCollection();

    foreach (MembershipUser user in users)
    {
        string[] roles = Roles.GetRolesForUser(user.UserName);

        // if roles empty
        if (roles.Count() == 0)
        {
            // Add User to a List for User with no Roles
            usersNoRoles.Add(user);
        }

   }