Java 8 streams unexpected behaviour

Asked May 14 '16 at 17:48

Active May 14 '16 at 18:29

Viewed 49 times

This code returns the length of the longest String in a list of Strings. I have two solutions of which one works and the other doesn't

Correct

System.out.println(listOfConsecutiveOneStrings.stream()
    .max(Comparator.comparing(String::length))
    .max(Comparator.comparing(String::length))
    .map(onesSequence -> onesSequence.length())
    .get());

Wrong

System.out.println(listOfConsecutiveOneStrings.stream()
    .max(Comparator.comparing(String::length))
    .map(onesSequence -> onesSequence.length())
    .max(Integer::max)
    .get());

Why is the first snippet correct and the second one incorrect?

edited May 14 '16 at 18:29

randers

5,031
5
37
64

asked May 14 '16 at 17:48

AdHominem

1,204
3
13
32

3

both of methods look not correctly – Andrew Tobilko May 14 '16 at 17:52
I think you should map the length, then get the max of the integers. I'm not sure why you seem to be sorting the list first – OneCricketeer May 14 '16 at 17:54
`max()` doesn't have the such signature – Andrew Tobilko May 14 '16 at 17:55

Java 8 streams unexpected behaviour

Correct

Wrong

0 Answers0