How to force max to return ALL maximum values in a Java Stream?

后端 未结 7 848
陌清茗
陌清茗 2020-11-22 09:47

I\'ve tested a bit the max function on Java 8 lambdas and streams, and it seems that in case max is executed, even if more than one object compares

相关标签:
7条回答
  • 2020-11-22 09:59

    If you'd rather rely on a library than the other answers here, StreamEx has a collector to do this.

    Stream.of(1, 3, 5, 3, 2, 3, 5)
        .collect(MoreCollectors.maxAll())
        .forEach(System.out::println);
    

    There's a version which takes a Comparator too for streams of items which don't have a natural ordering (i.e. don't implement Comparable).

    0 讨论(0)
  • 2020-11-22 10:00

    If I understood well, you want the frequency of the max value in the Stream.

    One way to achieve that would be to store the results in a TreeMap<Integer, List<Integer> when you collect elements from the Stream. Then you grab the last key (or first depending on the comparator you give) to get the value which will contains the list of max values.

    List<Integer> maxValues = st.collect(toMap(i -> i,
                         Arrays::asList,
                         (l1, l2) -> Stream.concat(l1.stream(), l2.stream()).collect(toList()),
                         TreeMap::new))
                 .lastEntry()
                 .getValue();
    

    Collecting it from the Stream(4, 5, -2, 5, 5) will give you a List [5, 5, 5].

    Another approach in the same spirit would be to use a group by operation combined with the counting() collector:

    Entry<Integer, Long> maxValues = st.collect(groupingBy(i -> i,
                    TreeMap::new,
                    counting())).lastEntry(); //5=3 -> 5 appears 3 times
    

    Basically you firstly get a Map<Integer, List<Integer>>. Then the downstream counting() collector will return the number of elements in each list mapped by its key resulting in a Map. From there you grab the max entry.

    The first approaches require to store all the elements from the stream. The second one is better (see Holger's comment) as the intermediate List is not built. In both approached, the result is computed in a single pass.

    If you get the source from a collection, you may want to use Collections.max one time to find the maximum value followed by Collections.frequency to find how many times this value appears.

    It requires two passes but uses less memory as you don't have to build the data-structure.

    The stream equivalent would be coll.stream().max(...).get(...) followed by coll.stream().filter(...).count().

    0 讨论(0)
  • 2020-11-22 10:03

    I would group by value and store the values into a TreeMap in order to have my values sorted, then I would get the max value by getting the last entry as next:

    Stream.of(1, 3, 5, 3, 2, 3, 5)
        .collect(groupingBy(Function.identity(), TreeMap::new, toList()))
        .lastEntry()
        .getValue()
        .forEach(System.out::println);
    

    Output:

    5
    5
    
    0 讨论(0)
  • 2020-11-22 10:05

    I believe the OP is using a Comparator to partition the input into equivalence classes, and the desired result is a list of members of the equivalence class that is the maximum according to that Comparator.

    Unfortunately, using int values as a sample problem is a terrible example. All equal int values are fungible, so there is no notion of preserving the ordering of equivalent values. Perhaps a better example is using string lengths, where the desired result is to return a list of strings from an input that all have the longest length within that input.

    I don't know of any way to do this without storing at least partial results in a collection.

    Given an input collection, say

    List<String> list = ... ;
    

    ...it's simple enough to do this in two passes, the first to get the longest length, and the second to filter the strings that have that length:

    int longest = list.stream()
                      .mapToInt(String::length)
                      .max()
                      .orElse(-1);
    
    List<String> result = list.stream()
                              .filter(s -> s.length() == longest)
                              .collect(toList());
    

    If the input is a stream, which cannot be traversed more than once, it is possible to compute the result in only a single pass using a collector. Writing such a collector isn't difficult, but it is a bit tedious as there are several cases to be handled. A helper function that generates such a collector, given a Comparator, is as follows:

    static <T> Collector<T,?,List<T>> maxList(Comparator<? super T> comp) {
        return Collector.of(
            ArrayList::new,
            (list, t) -> {
                int c;
                if (list.isEmpty() || (c = comp.compare(t, list.get(0))) == 0) {
                    list.add(t);
                } else if (c > 0) {
                    list.clear();
                    list.add(t);
                }
            },
            (list1, list2) -> {
                if (list1.isEmpty()) {
                    return list2;
                } 
                if (list2.isEmpty()) {
                    return list1;
                }
                int r = comp.compare(list1.get(0), list2.get(0));
                if (r < 0) {
                    return list2;
                } else if (r > 0) {
                    return list1;
                } else {
                    list1.addAll(list2);
                    return list1;
                }
            });
    }
    

    This stores intermediate results in an ArrayList. The invariant is that all elements within any such list are equivalent in terms of the Comparator. When adding an element, if it's less than the elements in the list, it's ignored; if it's equal, it's added; and if it's greater, the list is emptied and the new element is added. Merging isn't too difficult either: the list with the greater elements is returned, but if their elements are equal the lists are appended.

    Given an input stream, this is pretty easy to use:

    Stream<String> input = ... ;
    
    List<String> result = input.collect(maxList(comparing(String::length)));
    
    0 讨论(0)
  • 2020-11-22 10:09

    I'm not really sure whether you are trying to

    • (a) find the number of occurrences of the maximum item, or
    • (b) Find all the maximum values in the case of a Comparator that is not consistent with equals.

    An example of (a) would be [1, 5, 4, 5, 1, 1] -> [5, 5].

    An example of (b) would be:

    Stream.of("Bar", "FOO", "foo", "BAR", "Foo")
          .max((s, t) -> s.toLowerCase().compareTo(t.toLowerCase()));
    

    which you want to give [Foo, foo, Foo], rather than just FOO or Optional[FOO].

    In both cases, there are clever ways to do it in just one pass. But these approaches are of dubious value because you would need to keep track of unnecessary information along the way. For example, if you start with [2, 0, 2, 2, 1, 6, 2], it would only be when you reach 6 that you would realise it was not necessary to track all the 2s.

    I think the best approach is the obvious one; use max, and then iterate the items again putting all the ties into a collection of your choice. This will work for both (a) and (b).

    0 讨论(0)
  • 2020-11-22 10:12
    System.out.println(
      Stream.of(1, 3, 5, 3, 2, 3, 5)
        .map(a->new Integer[]{a})
        .reduce((a,b)-> 
            a[0]==b[0]?
                Stream.concat(Stream.of(a),Stream.of(b)).toArray() :
                a[0]>b[0]? a:b
        ).get()
    )
    
    0 讨论(0)
提交回复
热议问题