Why Iterator cannot be converted to Iterator

Question

I have the sample code below attempting to construct an immutable map out of an iterable:

ImmutableList> list = ImmutableList.of(
    new AbstractMap.SimpleEntry(4, 2d),
    new AbstractMap.SimpleEntry(16, 4d));
ImmutableMap.builder().putAll(list); // No error
ImmutableMap.builder().putAll(() -> list.stream().iterator()); // Error - cannot convert type...

I am getting the error Bad return type in lambda expression: Iterator> cannot be converted to Iterator>. From my understanding this should be valid because Integer is a valid upper bound of Integer, and Double is a valid upper bound of Double. What is the problem here and how can I fix specifically the iterator code?

I need to fix the iterator code in particular since I am trying to transform another collection into a map like below. I am using guava on android so do not have access to the default guava collect() methods (though I think I can implement them manually if there is really no other way to do this).

ImmutableList numberList = ImmutableList.of(2,4,8,12,16,24,32);
ImmutableMap.builder().putAll(() -> 
   numberList.stream().map(num -> 
       (Entry) new AbstractMap.SimpleEntry(num, Math.sqrt(num))).iterator());

user_1076640 · Answer

Oh boy, this one is ugly.

You're right that Entry is a "subtype" of Entry (I think it's not technically called a subtype, but is effectively one). But, as you may know, Iterator is not a subtype of Iterator. Instead you have to do Iterator.

So you'd need to add just one more ? extends in there:

Iterator>
         ^^^^^^^^^

I'm not sure offhand what the best way to do that is, though.

user_16814390 · Answer

Integer is not a valid upper bound of Integer, you can't extend class A from class A. For example you can't write this: Public class Actor extends Actor{ } So to solve your problem make Number instead Integer like this Iterator>

Why Iterator<Entry<Integer, Double> cannot be converted to Iterator<Entry<? extends Integer, ? extends Double>>

2 Answers