0
List<ObjectComposition> objComposition = objectsComposition.stream()
                .sorted((oc1, oc2) -> {
                    return  Integer.parseInt(oc1.getNumberAccordingToPlan().replaceAll("\\D+", "")) -
                            Integer.parseInt(oc2.getNumberAccordingToPlan().replaceAll("\\D+", ""));
                })
                .sorted((oc1, oc2) -> {
                    return  Integer.parseInt(oc1.getGroupNumber().replaceAll("\\D+", "")) -
                            Integer.parseInt(oc2.getGroupNumber().replaceAll("\\D+", ""));
                })
                .collect(Collectors.toList());  

Ну должно же отсортировать нормально - по номеру и по группе. Что бы в группе были значения в порядке возрастания. Получается:
введите сюда описание изображения
Не понимаю, что я делаю не так?

1
  • Проблема воспроизводиться при сортировке только по numberAccordingToPlan? На двух объектах? Если да, то можете подготовить воспроизводимый пример? 25 окт 2017 в 12:31

1 ответ 1

6

Ваш stream до этого места включительно будет отсортирован по numberAccordingToPlan

objectsComposition.stream()
            .sorted((oc1, oc2) -> {
                return  Integer.parseInt(oc1.getNumberAccordingToPlan().replaceAll("\\D+", "")) -
                        Integer.parseInt(oc2.getNumberAccordingToPlan().replaceAll("\\D+", ""));
            }) 
...
...

а с этого:

...
...
.sorted((oc1, oc2) -> {
                return  Integer.parseInt(oc1.getGroupNumber().replaceAll("\\D+", "")) -
                        Integer.parseInt(oc2.getGroupNumber().replaceAll("\\D+", ""));
            })

он будет отсортирован уже только по groupNumber

Чтобы добиться того, чего вам нужно, нужно использовать один компаратор:

List<ObjectComposition> objComposition = objectsComposition.stream()
            .sorted((oc1, oc2) -> {
                int cmp = Integer.parseInt(oc1.getNumberAccordingToPlan().replaceAll("\\D+", "")) -
                        Integer.parseInt(oc2.getNumberAccordingToPlan().replaceAll("\\D+", ""));
                if(cmp == 0){
                    cmp = Integer.parseInt(oc1.getGroupNumber().replaceAll("\\D+", "")) -
                            Integer.parseInt(oc2.getGroupNumber().replaceAll("\\D+", ""));
                }
                return cmp;
            })
            .collect(Collectors.toList());

Если вам нужно чтобы первым ключом сортировки было поле groupNumber, поменяйте порядок сравнения в этом компараторе.

6
  • Не помогает. Проблема та же, что на картинке - 1 оказывается в конце группы В-1... 25 окт 2017 в 12:52
  • 1
    @Артём, не может быть такого. Вы что-то не так сделали. Почему у вас вообще в выводе groundType, а в коде groupNumber? Это одно и то же?
    – iksuy
    25 окт 2017 в 12:57
  • 1
    Рукалицо. Моя вина. Не понимаю как я это не заметил. Да, это разные поля. Спасибо. З.Ы. Сортировка .sorted((oc1, oc2) -> {...}).sorted((oc1, oc2) -> {...}) сортирует правильно. 25 окт 2017 в 13:06
  • 1
    @Артём-.....-...-.-, если ответ вам помог, то вы можете отметить его верным)
    – ЮрийСПб
    25 окт 2017 в 13:15
  • 1
    @Артём, да, действительно всё корректно работает, я уже понял :)
    – iksuy
    25 окт 2017 в 14:09

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.