набор не содержит элемент, равный одному из его членов? - PullRequest
Новая
       35

набор не содержит элемент, равный одному из его членов?

0 голосов
/ 19 декабря 2018

Таким образом, в соответствии с Java API , Set::contains

возвращает true, если и только если этот набор содержит такой элемент e, что (o == null? E == null: o.equals (e))

Итак ... почему этот метод возвращает false, даже если набор содержит ровно один элемент, равный creds? 

private boolean validate(Credentials creds){
    Set<Credentials> acceptedUsers = getAcceptedUsers();
    return acceptedUsers.contains(creds);
}

Более конкретно, я вставил немного printlns 

private boolean validate(Credentials creds){
    Set<Credentials> acceptedUsers = getAcceptedUsers();

    System.out.print("accepted users: ");
    System.out.println(acceptedUsers);
    System.out.print("accessing user: ");
    System.out.println(creds);
    System.out.println("items are equal: " + acceptedUsers.stream().map(c -> c.equals(creds)).collect(Collectors.toSet()));
    System.out.println("access ok: " + (acceptedUsers.contains(creds) ? "YES" : "NO"));

    return acceptedUsers.contains(creds);
}

, и это то, что он должен был сказать: 

accepted users: [[
    foo
    FCDE2B2EDBA56BF408601FB721FE9B5C338D10EE429EA04FAE5511B68FBF8FB9
]]
accessing user: [
    foo
    FCDE2B2EDBA56BF408601FB721FE9B5C338D10EE429EA04FAE5511B68FBF8FB9
]
items are equal: [true]
access ok: NO

getAcceptedUsers в настоящее время возвращает фиктивный набор 

private Set<Credentials> getAcceptedUsers(){
    return new HashSet<Credentials>(){{add(new Credentials("foo","bar", false));}};
}

и Credentials реализованы как 

class Credentials{
    final String username;
    final String password;

    public Credentials(String username, String password, boolean isPasswordHashed) {
        this.username = username;

        if(isPasswordHashed) this.password = password;
        else {
            MessageDigest md;
            try {
                md = MessageDigest.getInstance("SHA-256");
            } catch (NoSuchAlgorithmException e) {
                throw new IllegalStateException(e);
            }

            md.update(password.getBytes());
            byte[] hash = md.digest();

            this.password = (new HexBinaryAdapter()).marshal(hash);
        }
    }

    @Override
    public boolean equals(Object obj) {
        if(obj == null) return false;
        if(!(obj instanceof Credentials)) return false;
        Credentials other = (Credentials)obj;
        return this.username.equals(other.username) && this.password.equals(other.password);
    }

    @Override
    public String toString() {
        return String.format("[\n\t%s\n\t%s\n]", username,password);
    }
}

Ответы [ 2 ]

0 голосов
/ 19 декабря 2018

Если вы хотите использовать HashMap или HashSet, вам нужно переопределить метод hashCode(), чтобы равные объекты имели одинаковый хэш-код.Например: 

@Override
public int hashCode() {
    return Objects.hash(username, password);
}
0 голосов
/ 19 декабря 2018

Реализация equals метода НЕ достаточна, вы должны также реализовать hashCode.

Как вы можете прочитать здесь :

Возвращает хешкодовое значение для этого набора.Хеш-код набора определяется как сумма хеш-кодов элементов в наборе, где хэш-код нулевого элемента определяется как ноль.Это гарантирует, что s1.equals (s2) подразумевает, что s1.hashCode () == s2.hashCode () для любых двух наборов s1 и s2, как того требует общий контракт Object.hashCode ().

...