Manière élégante de surmonter le type d'argument improbable pour equals (): Stream semble être sans rapport avec String

Premièrement, ce que vous pourriez avoir l'intention de faire pourrait être:

// here the 's.getId' is a String while 'map.entrySet()...map()' provides 'Stream<String>'
.filter(s -> s.getId().equals(map.entrySet().stream().map(Map.Entry::getKey)))

Deuxièmement, la comparaison utilisant est égal à dans votre code est incorrecte car elle est entre des objets de deux différents types de String et Stream.

System.out.println(stulist.stream()
        .filter(s -> map.keySet().contains(s.getId()))
        .count());

Vous pouvez utiliser map.containsKey pour éviter d'exécuter un flux sur l'ensemble d'entrées pour chaque entrée d'élève:

long count = stulist.stream()
                .filter(s -> map.entrySet()
                                .stream()
                                .map(Map.Entry::getKey)
                                .anyMatch(s.getId()::equals))
                .count();

Vous recevez l'avertissement car il est a détecté que vous testez String.equals (Stream ) , ce qui est bien sûr susceptible d'être un bogue (dans votre exemple, c'est certainement le cas).

Si vous deviez utiliser votre logique actuelle, la vérification appropriée devrait être:

long count = stulist.stream().map(student::getId).filter(map::containsKey).count();

En gros, pour comprendre pourquoi le compilateur ne donne pas d'erreur, vous devriez regarder dans String.equals () implémentation de la méthode

System.out.println(
  stulist.stream()
         .map(Student::getId)
         .filter(map::containsKey)
         .count());

Revenons maintenant à cette ligne:

s.getId().equals(map.entrySet().stream().map(Map.Entry::getKey))

Comme nous le savons, s.getId () est de type String et map.entrySet (). stream (). map (Map.Entry :: getKey) est de type Stream.

Depuis Stream n'est pas instanceof String , il est clair que la méthode String.equals () renverra false à chaque fois s.getId () est comparé à map.entrySet (). stream (). map (Map.Entry :: getKey) (d'où, 0 compter à la fin). Et le compilateur n'émet pas d'erreur, car rien d'illégal ne s'est réellement passé (en tenant compte de l'implémentation de String.equals()).

Aussi, probablement , le moyen le plus propre de trouver count sans avertissement serait:

    public boolean equals(Object anObject) {
        if (this == anObject) {
            return true;
        }
        if (anObject instanceof String) {
            String anotherString = (String)anObject;
            int n = value.length;
            if (n == anotherString.value.length) {
                char v1[] = value;
                char v2[] = anotherString.value;
                int i = 0;
                while (n-- != 0) {
                    if (v1[i] != v2[i])
                        return false;
                    i++;
                }
                return true;
            }
        }
        return false;
    }