Here's a Linq version, it returns a MembershipUserCollection
just like the similar Membership methods (FindUsersByEmail
, FindUsersByName
). It's not very pretty since it relies on the ForEach side effect:
public static MembershipUserCollection FindUsersByRole(string[] roles)
{
MembershipUserCollection msc = new MembershipUserCollection();
roles.Select(role => Roles.GetUsersInRole(role))
.Aggregate((a, b) => a.Union(b).ToArray())
.Distinct()
.Select( user => Membership.GetUser(user))
.ToList().ForEach( user => msc.Add(user));
return msc;
}
Alternatively if a list of MembershipUser will do:
public static List<MembershipUser> FindUsersByRole(string[] roles)
{
var userList = roles.Select(role => Roles.GetUsersInRole(role))
.Aggregate((a, b) => a.Union(b).ToArray())
.Distinct()
.Select( user => Membership.GetUser(user))
.ToList();
return userList;
}
And finally if you just need the user names you can skip one select:
public static List<string> FindUsersByRole(string[] roles)
{
var userList = roles.Select(role => Roles.GetUsersInRole(role))
.Aggregate((a, b) => a.Union(b).ToArray())
.Distinct()
.ToList();
return userList;
}