如何确保用户只添加到列表一次

Question

我必须通过自定义搜索列出用户列表，其中我从添加到共享点 Web 上的权限列表的所有组中获取所有用户。我的问题是用户可以位于多个组中，因此他们会多次添加到返回的列表中。我如何确保它们只添加一次？

C#

// keywords is the whatever value a user types into the search textbox
private static IEnumerable<SPUser> GetUsers(SPWeb web, string keywords)
    {
        var oList = new List<SPUser>();
        var oListUsers = web.Groups.Cast<SPGroup>().SelectMany(grp => grp.Users.Cast<SPUser>().Where(user => user.Name.Contains(keywords))).ToList();

        foreach (SPUser user in oListUsers)
        {
            // My attempt here is to check if the list already contains the current item
            // but it seems to ignore it. I've tried counting too, but same outcome.
            if (!oList.Contains(user))
                oList.Add(user);
        }

        return oList;
    }

Answer 1

试试这个（而不是包含）

 if (! oList.Any(u => u.Name == user.Name ))
 {
      oList.Add(user);
 }

Answer 2

看起来您的 SPUser 类需要实现 IEquatable 才能包含按您想要的方式工作。

Answer 3

您可以使用 Linq 获取唯一记录

var uniqueValues = oList.Distinct();

这将删除具有相同引用的 SPUser 对象。您还可以为自己的平等逻辑实现 IEqualityCompaprer

Answer 4

使用哈希集而不是列表。这样，您就不必检查是否包含，并且重复的项目将被忽略。

这也会更快，因为 HashSet 几乎可以轻松拒绝重复项，而 List<>.Contains() 的复杂度为 O(n)

 var oList = new HashSet<SPUser>();
    var oListUsers = web.Groups.Cast<SPGroup>().SelectMany(grp => grp.Users.Cast<SPUser> ().Where(user => user.Name.Contains(keywords))).ToList();

    foreach (SPUser user in oListUsers)
    {
      oList.Add(user);
    }

Answer 5

通过暴力，你可以使用 Distinct：

var oListUsers = web.Groups.Cast<SPGroup>().SelectMany(grp => grp.Users.Cast<SPUser>().Where(user => user.Name.Contains(keywords))).Distinct().ToList();

Answer 6

问题是您的 oListUsers 列表中有不同的对象代表同一用户，但具有不同的对象引用 - 由于 Contains() 使用对象引用进行检查，因此您赢了除非您在 SPUser 类上定义自定义比较器/平等，否则无法捕获这种情况。

或者，如果用户名是唯一的，您可以通过这种方式过滤掉重复项。