Why does iterating over entry set not work?

Question

I try to iterate over an EntrySet like this:

 for (Entry<A, List<B>> list : service.entrySet()) {
                if (list.getKey() == typ1) {
                    for (B current : list.getValue()) {                            
                      // do sth
                    }
                }
               
                } else {
                    PrintHelper.printOut("not implemented case"
                            + list.getKey());
                }
            }
       }

Even though I have that part if (list.getKey() == typ1) I still get the printed case not implemented case typ1.

Why is that the case? What am I doing wrong with the iteration/ the if case?

Are `typ1` and the return value of `list.getKey()` strings, or any kind of object? If so, compare them with `equals()`, not with ' ==' . — JustAnotherDeveloper, Sep 12 '20 at 16:44

score 1 · Accepted Answer · answered Sep 12 '20 at 16:44

1

The key of a Map (or map entry) is an object - you need to compare it with equals, not ==:

if (list.getKey().equals(typ1)) {

answered Sep 12 '20 at 16:44

Mureinik

297,002
52
306
350

Why does iterating over entry set not work?

1 Answers1