Linq/Enumerable Any Vs содержит

Это вероятно потому, что вы не переопределили сравнения на равенство (Equals, GetHashCode, operator==) в своем классе Role. Поэтому он выполнял сравнение ссылок, что на самом деле не лучшая идея, как будто это не один и тот же объект, это заставляет его думать, что это разные объекты. Вам необходимо переопределить эти операторы равенства, чтобы обеспечить равенство значений.

Вы должны переопределить Equals (а затем всегда также GetHashCode), если хотите использовать Contains. В противном случае Equals будет просто сравнивать ссылки.

Так, например:

public class Role
{
    public string RoleName{ get; set; }
    public int RoleID{ get; set; }
    // ...

    public override bool Equals(object obj)
    {
        Role r2 = obj as Role;
        if (r2 == null) return false;
        return RoleID == r2.RoleID;
    }
    public override int GetHashCode()
    {
        return RoleID;
    }
    public override string ToString()
    {
        return RoleName;
    } 
}

Другой вариант — реализовать собственный IEqualityComparer<Role> для перегрузки Enumerable.Contains:

public class RoleComparer : IEqualityComparer<Role>
{
    public bool Equals(Role x, Role y)
    {
        return x.RoleID.Equals(y.RoleID);
    }

    public int GetHashCode(Role obj)
    {
        return obj.RoleID;
    }
}

Используйте его таким образом:

var comparer = new RoleComparer();
User commentUser = userRepository.GetUserById(comment.userId);
Role commentUserRole = context.Roles.Single(x=>x.Name == "admin");
if(commentUser.Roles.Contains(commentUserRole, comparer))
{
    // ...
}

При использовании метода Contains вы проверяете, содержит ли массив Roles пользовательского объекта объект, который вы предварительно извлекли из базы данных. Хотя массив содержит объект для роли «admin», он не содержит точного объекта, который вы извлекли ранее.

При использовании Any-метода вы проверяете, есть ли какая-либо роль с именем "admin" - и это дает ожидаемый результат.

Чтобы получить тот же результат с помощью метода Contains, реализуйте интерфейс IEquatable<Role> в классе ролей и сравните имя, чтобы проверить, действительно ли два экземпляра имеют одно и то же значение.

Это будет ваше сравнение равенства для роли.

Объект в commentUserRole отличается от того, который вы ищете commentUser.Roles.

Ваш контекстный объект создаст новый объект, когда вы выберете из него и заполните свое свойство Roles коллекцией новых ролей. Если ваш контекст не отслеживает объекты, чтобы вернуть тот же объект, когда запрашивается вторая копия, это будет другой объект, даже если все свойства могут быть одинаковыми. Следовательно, отказ Содержит

Ваше предложение Any явно проверяет свойство Name, поэтому оно работает

Попробуйте реализовать роль IEquatable<Role>

public class Role : IEquatable<Role> {
  public bool Equals(Role compare) {
    return compare != null && this.Name == compare.Name;
  }
}

Хотя MSDN показывает, что это нужно только для List<T>, на самом деле вам может понадобиться переопределите Equals и GetHashCode, чтобы это работало

в таком случае:

public class Role : IEquatable<Role> {
  public bool Equals(Role compare) {
    return compare != null && this.Name == compare.Name;
  }

  public override bool Equals(object compare) {
     return this.Equals(compare as Role); // this will call the above equals method
  }

  public override int GetHashCode() {
     return this.Name == null ? 0 : this.Name.GetHashCode();
  }
}